[dev] Is "return true" the sane fallback for hasAppPermission?

[Ralf Lang]

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.

-- 
Ralf Lang
Linux Consultant / Developer
Tel.: +49-170-6381563
Mail: lang at b1-systems.de
B1 Systems GmbH
Osterfeldstraße 7 / 85088 Vohburg / http://www.b1-systems.de
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt,HRB 3537

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.horde.org/archives/dev/attachments/20140721/8bcd3ffb/attachment.bin>