[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