Re: Is "return true" the sane fallback for hasAppPermission?

Ralf Lang <[email protected]>
Newsgroups gmane.comp.horde.devel
Message-ID <[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.

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

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: [email protected]
signature.asc (application/pgp-signature, 263 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlPNa30ACgkQCs1dsHJ/X7A9EACghbUkY+eLogF/UFfkE3DyOl7v
yccAn1KcIiAaea0AT0y6Txic03gzbxhI
=nfJU
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.