[prev in list] [next in list] [prev in thread] [next in thread] 

List:       horde-dev
Subject:    Re: [dev] Is "return true" the sane fallback for hasAppPermission?
From:       Michael M Slusarz <slusarz () horde ! org>
Date:       2014-07-21 19:48:09
Message-ID: 20140721134809.Horde.yqcYTOnnXVzLkWRxu_4cSw1 () bigworm ! curecanti ! org
[Download RAW message or body]

Quoting Ralf Lang <lang@b1-systems.de>:

> 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 [slusarz@horde.org]

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: dev-unsubscribe@lists.horde.org
[prev in list] [next in list] [prev in thread] [next in thread]