[dev] Re: [cvs] commit: horde/config conf.xml hooks.php.dist framework/Share Share.php

[Stuart K Bingë]

On Monday, 04 October 2004 18:30, Stuart K Bingë wrote:
> I'd actually prefer this method as well. Although, wouldn't you need to
> include hooks.php before testing for the function existence? Something
> like:
>
> include_once HORDE_BASE . '/config/hooks.php';
> if (function_exists('_horde_hook_share_add')) {
>  Horde::callHook('_horde_hook_share_add', ...);
> }
>
> Or is there some other place that these could (or should) be defined that
> gets seen before Share.php?

* Note to self: read code before posting *
I see that callHook() does the existence checking itself...

So, as far as I can tell, these new blocks can just be replaced by a single 
Horde::callHook() call (or am I still missing something?).

-- 
Stuart K Bingë
 - Freelance UNIX/C++, PHP, Perl developer.

Email: omicron at mighty.co.za