[Fusionforge-commits] r16099 - trunk/src/plugins/mediawiki/www

Olivier Berger olberger at fusionforge.org
Thu Aug 16 17:01:16 CEST 2012


Author: olberger
Date: 2012-08-16 17:01:16 +0200 (Thu, 16 Aug 2012)
New Revision: 16099

Modified:
   trunk/src/plugins/mediawiki/www/plugin_admin.php
Log:
Safer if/then sections with curly braces

Modified: trunk/src/plugins/mediawiki/www/plugin_admin.php
===================================================================
--- trunk/src/plugins/mediawiki/www/plugin_admin.php	2012-08-16 15:01:09 UTC (rev 16098)
+++ trunk/src/plugins/mediawiki/www/plugin_admin.php	2012-08-16 15:01:16 UTC (rev 16099)
@@ -62,14 +62,17 @@
 	exit_error("Invalid User", "Cannot process your request for this user.");
 
 $gid = getIntFromRequest("group_id", -1);
-if ($gid == -1)
+if ($gid == -1) {
 	$group = false;
-else
+} else {
 	$group = group_get_object($gid);
-if (!$group)
+}
+if (!$group) {
 	exit_error("Invalid Project", "Nonexistent Project");
-if (!$group->usesPlugin("mediawiki"))
+}
+if (!$group->usesPlugin("mediawiki")) {
 	exit_error("Invalid Project", "Project does not use MediaWiki Plugin");
+}
 
 $userperm = $group->getPermission();
 if (!$userperm->IsMember())
@@ -106,10 +109,11 @@
 	$feedback = "";
 
 	if (getIntFromRequest("logo_nuke") == 1) {
-		if (unlink($wgUploadDirectory . "/.wgLogo.png"))
+	  if (unlink($wgUploadDirectory . "/.wgLogo.png")) {
 			$feedback = _("File successfully removed");
-		else
+	  } else {
 			$feedback = _("File removal error");
+	  }
 	} else if ($userfile && is_uploaded_file($userfile['tmp_name']) &&
 	    util_is_valid_filename($userfile['name'])) {
 		$infile = $userfile['tmp_name'];




More information about the Fusionforge-commits mailing list