[giapeto] Permission system broken
Roel Gloudemans
roel at gloudemans.info
Wed Sep 1 23:11:27 PDT 2004
The fix in CVS works.
Thanks guys,
Roel
>> Uh, I think I found it. Jan, if you could pitch in with your
>> thoughts on this,
>> since you rearranged recently the Perms.php code. Giapeto works by extending
>> Perms to Giapeto_Perms so that some extra stuff can be set up, specific to
>> giapeto. However, now that we have the singleton()/factory() method and no
>> constructor function Perms(), giapeto obviously keeps ending up with
>> the null
>> driver when it calls in base.php:
>> $GLOBALS['giapeto_perms'] = &Giapeto_Perms::Perms();
>
> I think the best solution is going to be to have Giapeto_Perms store an
> instance
> of the Horde perms driver, and to pass calls through (in
> pattern-speak, either
> using Perms as a delegate, or acting as a decorator to Perms).
>
> I'll take a look at this.
>
More information about the giapeto
mailing list