Re: [cgiapp] Re: Yet another callback system proposal

[Cees Hek <suppressed>]

On 5/20/05, Mark Stosberg <suppressed> wrote:
> Cees, do you want to take what you like best of your patch and the other
> refinements to it and craft them into a new proposal-as-code?

Well, I would like to hear from some of the others on the list as well
(especially Michael, since he has sparked all this up again and I
believe he wants to use the callbacks for one of his projects).  I
know that several people are already using the callbacks in locally
patched versions, and there have been several plugins built that
mention the callbacks.  So please speak up now if you have any input
(positive or negative)...

Here is what I see as a minimal first step implementation of the callbacks:

- callbacks can be registered at the class level or the object level
- ordering is on a first come first served basis
- callbacks are called starting from the object and then up the inheritance tree
    - so if we have a prerun method registered at the 
      object level, it will be called first, and then any prerun
      methods in the current class, and then upwards
      in order until we hit the CGI::Application class
    - this means the standard cgiapp_prerun method will be
      called last!
    - what about multiple inheritance?  Does that pose a problem?
- new hooks can be created at the class level or the object level
    - for example the tt_pre_process hook in the Template Toolkit plugin

Anything missing there?

If anyone has some time to hammer something out, I would appreciate
it, 'cause I have a potentially busy weekend ahead of me.  It may be
sufficient to take Michael's latest patch and just remove and rework
the order of execution.

Also if anyone can come up with some good tests for the test suite
that would be very helpful (maybe they are already floating around in
one of the patches).

Cheers,

Cees

---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/suppressed/
              http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed