[dev] Re: [cvs] commit: horde/lib Perms.php

[Marko Djukic]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Marko Djukic <mdjukic at horde.org>:
>
> > yes, that sounds like a good idea. i was not entirely happy with the code
> asit
> > is now but does look better than lots of code repetition earlier. plus the
> > other goal was to get it to be more versatile where it can be "imported"
> into
> > an app so that there could be freedom to do more app-specific perms stuff
> > within the app.
>
> Alright, made the Perms_UI class and updated files to use it.
>
> > > And you should probably add an
> > > optional dependancy on the Horde_Tree package to the Perms package to
> cover
> > > that.
> >
> > k, sure.
>
> Added this, too.
>
> > > Would you object to my
> > > tweaking the code a bit, and then helping fix up the calling scripts?
> >
> > no probs.
>
> Alright, done so a bit. Lemme know what you think, or if I broke anything.

ok, code looks much better. nope, nothing broke. thanks.

m.