[dev] Proposal for _group_hook

Ben Klang ben at alkaloid.net
Fri Aug 18 09:44:36 PDT 2006


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/
-- 
Ben Klang
Alkaloid Networks
ben at alkaloid.net
404.475.4850
http://projects.alkaloid.net


More information about the dev mailing list