Re: Is "return true" the sane fallback for hasAppPermission?
Michael M Slusarz <[email protected]>
| Newsgroups | gmane.comp.horde.devel |
|---|---|
| Message-ID | <20140721134809.Horde.yqcYTOnnXVzLkWRxu_4cSw1@bigworm.curecanti.org> |
Quoting Ralf Lang <[email protected]>: > On 21.07.2014 11:35, Jasper Olbrich wrote: >> Hi, >> >> I just stumbled upon the function hasAppPermission in Horde/Core/Perms.php: >> >> if ($this->_perms->exists($app . ':' . $permission)) { >> // determine permission >> } >> return true; >> >> Is this "return true;" intentional? In our case, this made everyone >> "Resource Manager" in Kronolith untill we created the corresponding >> permission and set it up appropriately. >> > > OTOH, it would deny all users all apps (including horde itself) before > permissions are configured. And permissions is an optional feature. michael ___________________________________ Michael Slusarz [[email protected]] -- dev mailing list Frequently Asked Questions: http://wiki.horde.org/FAQ To unsubscribe, mail: [email protected]