[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