[Fusionforge-commits] r9090 - trunk/gforge/common/include
Alain Peyrat
aljeux at libremir.placard.fr.eu.org
Mon Mar 15 21:40:54 CET 2010
Author: aljeux
Date: 2010-03-15 21:40:54 +0100 (Mon, 15 Mar 2010)
New Revision: 9090
Modified:
trunk/gforge/common/include/session.php
Log:
Minor: use isset to check variable.
Modified: trunk/gforge/common/include/session.php
===================================================================
--- trunk/gforge/common/include/session.php 2010-03-15 20:40:48 UTC (rev 9089)
+++ trunk/gforge/common/include/session.php 2010-03-15 20:40:54 UTC (rev 9090)
@@ -371,9 +371,7 @@
exit_permission_denied($reason);
}
- //don't really like this, but as admin_flags is not mandatory
- //I add @ to remove the warning
- if (@$req['admin_flags']) {
+ if (isset($req['admin_flags']) && $req['admin_flags']) {
if (!$perm->isAdmin()) {
exit_permission_denied($reason);
}
More information about the Fusionforge-commits
mailing list