[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