Re: Permission system broken

Roel Gloudemans <[email protected]> Thu, 2 Sep 2004 08:11:27 +0200
Newsgroups gmane.comp.horde.giapeto
Message-ID <[email protected]>
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.
>


-- 
giapeto mailing list - Join the hunt: http://horde.org/bounties/#giapeto
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: [email protected]