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

[Chuck Hagenbuch]

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.

> i'm a bit K.O. with my connections these days. not sure how long it will
> last. so go ahead and tweak as much as you want, i'll leave it for now and do
> stuff i can do locally and catch up when i get back online in a meaningful
> way.

Alright, good luck with that.

> the only other bit i didn't get a chance to commit before i lost the
> connection was the app's advertising their available perms through the api...
> but that can wait.
>
> plus the horde_form i started to change a bit to allow the sections to be
> presented either as vertical drop down sections (as they are now) or as
> horizontal tabs at the top. but again, can wait.

Both sound good - looking forward to them.

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
Born right the first time.