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

[Jan Schneider]

Zitat von Marko Djukic <mdjukic at horde.org>:

> Quoting Marko Djukic <tech at oblo.com>:
>
> > mdjukic     2003/12/18 15:06:41 PST
> >
> >   Modified files:
> >     lib                  Registry.php
> >   Log:
> >   - check for perms to an app when doing a pushApp(), return an error
> if not
> > allowed.
>
> this spawns a few questions:
> - as perms are becomming a quite central part of the apps and blocks,
> should we
> make $perms a global var and set it up in horde's base.php? rather than
> require_once and singleton() all over the place...

I have no strong opinion about this. But it probably makes sense at a
certain point.

> - does it make sense to keep using the registry allow_guests setting? it
> seems
> that it will start creating confusion. who gets priority? setting
> allow_guests
> in the registry or in the perms?

Yeah, this setting should be dropped in favour of guest perms.

> - callByPackage() should also be checking for perms, no?

Correct. But doesn't it imply a pushApp()?

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft