[giapeto] Permission system broken

Chuck Hagenbuch chuck at horde.org
Wed Sep 1 18:54:24 PDT 2004


Quoting Marko Djukic <mdjukic at horde.org>:

> 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.

-chuck

--
"Regard my poor demoralized mule!" - Juan Valdez


More information about the giapeto mailing list