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]