[Fusionforge-commits] r12282 - branches/Branch_5_1/src/www/tracker/reporting

Alain Peyrat aljeux at libremir.placard.fr.eu.org
Fri Feb 11 21:50:54 CET 2011


Author: aljeux
Date: 2011-02-11 21:50:53 +0100 (Fri, 11 Feb 2011)
New Revision: 12282

Modified:
   branches/Branch_5_1/src/www/tracker/reporting/index.php
Log:
Check group_id before using it.

Modified: branches/Branch_5_1/src/www/tracker/reporting/index.php
===================================================================
--- branches/Branch_5_1/src/www/tracker/reporting/index.php	2011-02-11 20:50:48 UTC (rev 12281)
+++ branches/Branch_5_1/src/www/tracker/reporting/index.php	2011-02-11 20:50:53 UTC (rev 12282)
@@ -43,6 +43,18 @@
 	exit_error($report->getErrorMessage());
 }
 
+$group = group_get_object($group_id);
+if (!$group || !is_object($group)) {
+        exit_no_group();
+}
+if ($group->isError()) {
+        if($group->isPermissionDeniedError()) {
+                exit_permission_denied($group->getErrorMessage());
+        } else {
+                exit_error($group->getErrorMessage(), 'tracker');
+        }
+}
+
 /*
  * Set the start date to birth of the project.
  */
@@ -60,18 +72,6 @@
 }
 if ($end < $start) list($start, $end) = array($end, $start);
 
-$group = group_get_object($group_id);
-if (!$group || !is_object($group)) {
-        exit_no_group();
-}
-if ($group->isError()) {
-        if($group->isPermissionDeniedError()) {
-                exit_permission_denied($group->getErrorMessage());
-        } else {
-                exit_error($group->getErrorMessage(), 'tracker');
-        }
-}
-
 if (!session_loggedin()) {
 	exit_not_logged_in();
 }




More information about the Fusionforge-commits mailing list