[Fusionforge-commits] r8221 - trunk/gforge/www/admin

Roland Mas lolando at libremir.placard.fr.eu.org
Thu Oct 1 13:52:38 CEST 2009


Author: lolando
Date: 2009-10-01 13:52:38 +0200 (Thu, 01 Oct 2009)
New Revision: 8221

Modified:
   trunk/gforge/www/admin/approve-pending.php
Log:
Fixed confusing feedback on error when approving a group (#28)

Modified: trunk/gforge/www/admin/approve-pending.php
===================================================================
--- trunk/gforge/www/admin/approve-pending.php	2009-09-30 20:42:19 UTC (rev 8220)
+++ trunk/gforge/www/admin/approve-pending.php	2009-10-01 11:52:38 UTC (rev 8221)
@@ -50,9 +50,11 @@
 		return false;
 	}
 
-	$feedback .= sprintf(_('Approving Group: %1$s'), $group->getUnixName()).'<br />';
 
-	if (!$group->approve(session_get_user())) {
+	if ($group->approve(session_get_user())) {
+		$feedback .= sprintf(_('Approving Group: %1$s'), $group->getUnixName()).'<br />';
+	} else {
+		$feedback .= sprintf(_('Error when approving Group: %1$s'), $group->getUnixName()).'<br />';
 		$feedback .= $group->getErrorMessage().'<br />';
 		return false;
 	}




More information about the Fusionforge-commits mailing list