[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