[dev] api / hasMethod()

[Jan Schneider]

Zitat von Marko Djukic <marko at oblo.com>:

> Quoting Chuck Hagenbuch <chuck at horde.org>:
>
> > Quoting Marko Djukic <marko at oblo.com>:
> >
> > > in the Registry::hasMethod() does it make sense to be looping through
> all
> > > the apps listed in the registry to check for methods or should it
> really
> > > loop only for active apps?
> >
> > It should exclude inactive ones definitely, good call. notoolbar and
> hidden
> > ones probably should be allowed, though. hidden is a judgement call -
> > anyone?
>
> ok committed a fix, included "hidden" as well, because by logic it makes
> sense
> for a hidden app to allow its api to be used - it's not inactive and if
> there's
> perm issues that should be handled by Perms. but anyone feel free to
> argue
> otherwise...

Agreed.

Jan.

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