John Peacock said the following on 02/21/2006 04:05 PM:
> Robin Bowes wrote:
> 
>> Are you suggesting that I don't log success from there at all but
>> instead create a new plugin, which hooks hook_queue and write to the log
>> from there?
> 
> 
> The current design separates hooking logging from hooking any other
> plugin hook (because you normally want to have logging loaded before
> other plugins, so that you can log the plugins loading).  There isn't
> any reason you couldn't have a very small hook_queue plugin that works
> hand in hand with the logging plugin.  Just take out the
> hook_reset_transaction() from logging/adaptive and make it hook_queue
> instead and list it last.

OK, you've convinced me!

> If we ever get the hook_*_first, hook_*_middle, hook_*_last code working
> (something that I believe Robert has mentioned in the past), we could
> easily merge the two files again and just register the hook_queue_last
> as part of the logging plugin itself.

Sure.

Thanks,

R.

Reply via email to