[dev] Proposal for _group_hook

[Ben Chavet]

Quoting Ben Klang <ben at alkaloid.net>:

> On Friday 18 August 2006 12:26, Chuck Hagenbuch wrote:
>> Quoting Mike Dorman <dorm at dorm.org>:
>> > Anyone have any thoughts on changing the method for doing group
>> > hooks to _group_hook($groupName, $userName) instead of
>> > _group_hook_groupName($userName)?
>> >
>> > This will allow for determining which groups have hooks
>> > programatically, rather than manually having to create the hook for
>> > each group.
>> >
>> > I'm not sure if there is much of a performance implication.  The
>> > function_exists() call would be for _group_hook, and if it exists
>> > would then call it.  So I don't think there would be any extra
>> > function calls.
>>
>> This does make sense to me. Ben, Jan, other people who use complicated
>> groups setups - any problems here?
>>
>> -chuck
> I concur, except for one question:  if we're using the group name we need to
> make sure the full group path is passed.  Otherwise it may be ambiguous.
>
> /BAK/

Or, use the group ID, like we do everywhere else.

--Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/dev/attachments/20060818/9874d10d/attachment.bin