mdaniels5757 / waca

acc.php

Braces +72 added lines, -144 removed lines

@@ -77,8 +77,7 @@ 
 		echo showlogin();
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	else {
+	} else {
 		// A content block is created if the action is none of the above.
 		// This block would later be used to keep all the HTML except the header and footer.
 		$out = "<div id=\"content\">";
@@ -101,8 +100,7 @@ 
 	echo defaultpage();
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "sreg") {
+} elseif ($action == "sreg") {
 	global $useOauthSignup, $smarty, $allowRegistration;
 
 	if(!$allowRegistration) {
@@ -202,8 +200,7 @@ 
 					0, 
 					$ex);
 			}
-		}
-		else {
+		} else {
 			global $baseurl;
 			Notification::userNew($newUser);
 			header("Location: {$baseurl}/acc.php?action=registercomplete");
@@ -211,8 +208,7 @@ 
 	});
     
 	die();
-}
-elseif ($action == "register") {
+} elseif ($action == "register") {
 	global $useOauthSignup, $smarty, $allowRegistration;
 	if($allowRegistration) {
 		$smarty->assign("useOauthSignup", $useOauthSignup);
@@ -222,12 +218,10 @@ 
 	}
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "registercomplete") {
+} elseif ($action == "registercomplete") {
 	$smarty->display("registration/alert-registrationcomplete.tpl");
 	BootstrapSkin::displayInternalFooter();
-}
-elseif ($action == "forgotpw") {
+} elseif ($action == "forgotpw") {
 	global $baseurl, $smarty;
     
 	if (isset ($_GET['si']) && isset ($_GET['id'])) {
@@ -256,14 +250,12 @@ 
                     
 					BootstrapSkin::displayInternalFooter();
 					die();
-				}
-				else {
+				} else {
 					BootstrapSkin::displayAlertBox("Passwords did not match!", "alert-error", "Error", true, false);
 					BootstrapSkin::displayInternalFooter();
 					die();
 				}
-			}
-			else {
+			} else {
 				BootstrapSkin::displayAlertBox("Invalid request<!-- 1 -->", "alert-error", "Error", true, false);
 				BootstrapSkin::displayInternalFooter();
 				die();
@@ -277,8 +269,7 @@ 
 			$smarty->assign('si', $_GET['si']);
 			$smarty->assign('id', $_GET['id']);
 			$smarty->display('forgot-password/forgotpwreset.tpl');
-		}
-		else {
+		} else {
 			BootstrapSkin::displayAlertBox(
 				"The hash supplied in the link did not match the hash in the database!", 
 				"alert-error", 
@@ -304,8 +295,7 @@ 
             
 			BootstrapSkin::displayInternalFooter();
 			die();
-		}
-		elseif (strtolower($_POST['email']) != strtolower($user->getEmail())) {
+		} elseif (strtolower($_POST['email']) != strtolower($user->getEmail())) {
 			BootstrapSkin::displayAlertBox("Could not find user with that username and email address!", 
 				"alert-error", 
 				"Error", 
@@ -314,8 +304,7 @@ 
             
 			BootstrapSkin::displayInternalFooter();
 			die();
-		}
-		else {
+		} else {
 			$hash = $user->getForgottenPasswordHash();
                        
 			$smarty->assign("user", $user);
@@ -347,8 +336,7 @@ 
 
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "login") {
+} elseif ($action == "login") {
 	global $baseurl, $smarty;
     
 	if (!isset($_POST['username'])) {
@@ -379,8 +367,7 @@ 
             
 			reattachOAuthAccount($user);
 		}
-	}
-	else {
+	} else {
 		global $enforceOAuth;
         
 		if ($enforceOAuth) {
@@ -463,8 +450,7 @@ 
 	}
     
 	header("Location: $baseurl/acc.php");
-}
-elseif ($action == "messagemgmt") {
+} elseif ($action == "messagemgmt") {
 	global $smarty;
     
 	if (isset($_GET['view'])) {
@@ -548,8 +534,7 @@ 
    
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "templatemgmt") {
+} elseif ($action == "templatemgmt") {
 	global $baseurl, $smarty;
     
 	if (isset($_GET['view'])) {
@@ -595,15 +580,13 @@ 
 				SessionAlert::success("Template successfully created.");
 				header("Location: $baseurl/acc.php?action=templatemgmt");
 			});
-		}
-		else {
+		} else {
 			
 			if (isset($_POST['preview'])) {
 				$usercode = $_POST['usercode'];
 				$botcode = $_POST['botcode'];
 				echo displayPreview($usercode);
-			}
-			else {
+			} else {
 				$usercode = '';
 				$botcode = '';
 			}
@@ -629,8 +612,7 @@ 
 			SessionAlert::success("Disabled automatic user welcoming.");
 			header("Location: {$baseurl}/acc.php?action=templatemgmt");
 			die();
-		}
-		else {
+		} else {
 			$template = WelcomeTemplate::getById($_GET['select'], gGetDb());
 			if ($template !== false) {
 				$user->setWelcomeTemplate($template->getId());
@@ -639,8 +621,7 @@ 
 				SessionAlert::success("Updated selected welcome template for automatic welcoming.");
 				header("Location: {$baseurl}/acc.php?action=templatemgmt");
 				die();
-			}
-			else {
+			} else {
 				SessionAlert::error("Something went wrong, we can't find the template you asked for!");
 				header("Location: {$baseurl}/acc.php?action=templatemgmt");
 				die();
@@ -717,8 +698,7 @@ 
             
 			header("Location: $baseurl/acc.php?action=templatemgmt");
 			die();
-		}
-		else {
+		} else {
 			$smarty->assign("template", $template);
 			$smarty->display("welcometemplate/edit.tpl");
             
@@ -734,8 +714,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "sban") {
+} elseif ($action == "sban") {
 	global $smarty;
     
 	// Checks whether the current user is an admin.
@@ -763,21 +742,18 @@ 
     
 	if ($duration == "-1") {
 		$duration = -1;
-	}
-	elseif ($duration == "other") {
+	} elseif ($duration == "other") {
 		$duration = strtotime($_POST['otherduration']);
 		if (!$duration) {
 			BootstrapSkin::displayAlertBox("Invalid ban time", "alert-error", "", false, false);
 			BootstrapSkin::displayInternalFooter();
 			die();
-		}
-		elseif (time() > $duration) {
+		} elseif (time() > $duration) {
 			BootstrapSkin::displayAlertBox("Ban time has already expired!", "alert-error", "", false, false);
 			BootstrapSkin::displayInternalFooter();
 			die();
 		}
-	}
-	else {
+	} else {
 		$duration = $duration + time();
 	}
     
@@ -857,8 +833,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "unban") {
+} elseif ($action == "unban") {
 	global $smarty;
     
 	if (!isset($_GET['id']) || $_GET['id'] == "") {
@@ -897,8 +872,7 @@ 
 			BootstrapSkin::displayAlertBox("You must enter an unban reason!", "alert-error", "", false, false);
 			BootstrapSkin::displayInternalFooter();
 			die();
-		}
-		else {
+		} else {
 			$database = gGetDb();
             
 			$database->transactionally(function() use ($database, $ban)
@@ -917,15 +891,13 @@ 
 			Notification::unbanned($ban, $_POST['unbanreason']);
 			die();
 		}
-	}
-	else {
+	} else {
 		$smarty->assign("ban", $ban);
 		$smarty->display("bans/unban.tpl");
         
 		BootstrapSkin::displayInternalFooter();
 	}
-}
-elseif ($action == "ban") {
+} elseif ($action == "ban") {
 	global $smarty;
     
 	if (isset ($_GET['ip']) || isset ($_GET['email']) || isset ($_GET['name'])) {
@@ -945,8 +917,7 @@ 
 			$row = $statement->fetch(PDO::FETCH_ASSOC);
 			$target = getTrustedClientIP($row['ip'], $row['forwardedip']);
 			$type = "IP";
-		}
-		elseif (isset($_GET['email'])) {
+		} elseif (isset($_GET['email'])) {
 			$query = "SELECT email FROM request WHERE id = :ip;";
 			$statement = $database->prepare($query);
 			$statement->bindValue(":ip", $_GET['email']);
@@ -954,8 +925,7 @@ 
 			$row = $statement->fetch(PDO::FETCH_ASSOC);
 			$target = $row['email'];
 			$type = "EMail";
-		}
-		elseif (isset($_GET['name'])) {
+		} elseif (isset($_GET['name'])) {
 			$query = "SELECT name FROM request WHERE id = :ip;";
 			$statement = $database->prepare($query);
 			$statement->bindValue(":ip", $_GET['name']);
@@ -963,8 +933,7 @@ 
 			$row = $statement->fetch(PDO::FETCH_ASSOC);
 			$target = $row['name'];
 			$type = "Name";
-		}
-		else {
+		} else {
 			BootstrapSkin::displayAlertBox("Unknown ban type.", "alert-error");
 			BootstrapSkin::displayInternalFooter();
 			die();    
@@ -979,8 +948,7 @@ 
 		$smarty->assign("bantype", $type);
 		$smarty->assign("bantarget", trim($target));
 		$smarty->display("bans/banform.tpl");
-	}
-	else {
+	} else {
 		$bans = Ban::getActiveBans();
   
 		$smarty->assign("activebans", $bans);
@@ -989,8 +957,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "defer" && $_GET['id'] != "" && $_GET['sum'] != "") {
+} elseif ($action == "defer" && $_GET['id'] != "" && $_GET['sum'] != "") {
 	global $availableRequestStates;
 	
 	if (array_key_exists($_GET['target'], $availableRequestStates)) {
@@ -1071,14 +1038,12 @@ 
 		});
         
 		die();
-	}
-	else {
+	} else {
 		BootstrapSkin::displayAlertBox("Defer target not valid.", "alert-error", "Error", true, false);
 		BootstrapSkin::displayInternalFooter();
 		die();
 	}
-}
-elseif ($action == "prefs") {
+} elseif ($action == "prefs") {
 	global $smarty, $enforceOAuth;
     
 	if (isset ($_POST['sig'])) {
@@ -1092,8 +1057,7 @@ 
             
 			if ($mailisvalid === false) {
 				BootstrapSkin::displayAlertBox("Invalid email address", "alert-error", "Error!");
-			}
-			else {
+			} else {
 				$user->setEmail(trim($_POST['email']));
 			}
 		}
@@ -1114,20 +1078,17 @@ 
 	$smarty->display("prefs.tpl");
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "done" && $_GET['id'] != "") {
+} elseif ($action == "done" && $_GET['id'] != "") {
 	// check for valid close reasons
 	global $messages, $baseurl, $smarty;
 	
 	if (isset($_GET['email'])) {
 		if ($_GET['email'] == 0 || $_GET['email'] == "custom") {
 			$validEmail = true;
-		}
-		else {
+		} else {
 			$validEmail = EmailTemplate::getById($_GET['email'], gGetDb()) != false;
 		}
-	}
-	else {
+	} else {
 		$validEmail = false;
 	}
     
@@ -1193,8 +1154,7 @@ 
 	$ue = unserialize($userexist);
 	if (!isset ($ue['query']['users']['0']['missing'])) {
 		$exists = true;
-	}
-	else {
+	} else {
 		$exists = false;
 	}
 
@@ -1236,8 +1196,7 @@ 
 				$preloadTitle = $template->getName();
 				$preloadText = $template->getText();
 				$preloadAction = $template->getDefaultAction();
-			}
-			else {
+			} else {
 				$preloadText = "";
 				$preloadTitle = "";
 				$preloadAction = "";
@@ -1275,8 +1234,7 @@ 
 		if (User::getCurrent()->getEmailSig() != "") {
 			$emailsig = html_entity_decode(User::getCurrent()->getEmailSig(), ENT_QUOTES, "UTF-8");
 			mail($request->getEmail(), "RE: [ACC #{$request->getId()}] English Wikipedia Account Request", $_POST['msgbody'] . "\n\n" . $emailsig, $headers);
-		}
-		else {
+		} else {
 			mail($request->getEmail(), "RE: [ACC #{$request->getId()}] English Wikipedia Account Request", $_POST['msgbody'], $headers);
 		}
 
@@ -1289,8 +1247,7 @@ 
 			if ($_POST['action'] == EmailTemplate::CREATED) {
 				$gem  = 'custom-y';
 				$crea = "Custom, Created";
-			}
-			else {
+			} else {
 				$gem  = 'custom-n';
 				$crea = "Custom, Not Created";
 			}
@@ -1301,8 +1258,7 @@ 
 			BootstrapSkin::displayAlertBox(
 				"Request " . $request->getId() . " (" . htmlentities($request->getName(), ENT_COMPAT, 'UTF-8') . ") marked as '" . htmlentities($crea, ENT_COMPAT, 'UTF-8') . "'.", 
 				"alert-success");
-		}
-		else if ($_POST['action'] == "mail") {
+		} else if ($_POST['action'] == "mail") {
 			// no action other than send mail!
 			Logger::sentMail(gGetDb(), $request, $messageBody);
 			Logger::unreserve(gGetDb(), $request);
@@ -1310,8 +1266,7 @@ 
 			Notification::sentMail($request);
 			BootstrapSkin::displayAlertBox("Sent mail to Request {$request->getId()}", 
 				"alert-success");
-		}
-		else if (array_key_exists($_POST['action'], $availableRequestStates)) {
+		} else if (array_key_exists($_POST['action'], $availableRequestStates)) {
 			// Defer
 
 			$request->setStatus($_POST['action']);
@@ -1324,8 +1279,7 @@ 
 			Notification::requestDeferredWithMail($request);
 			BootstrapSkin::displayAlertBox("Request {$request->getId()} deferred to $deto, sending an email.", 
 				"alert-success");
-		}
-		else {
+		} else {
 			// hmm. not sure what happened. Log that we sent the mail anyway.
 			Logger::sentMail(gGetDb(), $request, $messageBody);
 			Logger::unreserve(gGetDb(), $request);
@@ -1344,8 +1298,7 @@ 
 		echo defaultpage();
 		BootstrapSkin::displayInternalFooter();
 		die();		
-	}
-	else {
+	} else {
 		// Not a custom close, just a normal close
 	    
 		$request->setStatus('Closed');
@@ -1358,8 +1311,7 @@ 
 		
 		if ($gem == '0') {
 			$crea = "Dropped";
-		}
-		else {
+		} else {
 			$template = EmailTemplate::getById($gem, gGetDb());
 			$crea = $template->getName();
 		}
@@ -1380,8 +1332,7 @@ 
 		BootstrapSkin::displayInternalFooter();
 		die();
 	}
-}
-elseif ($action == "zoom") {
+} elseif ($action == "zoom") {
 	if (!isset($_GET['id'])) {
 		BootstrapSkin::displayAlertBox("No request specified!", "alert-error", "Error!", true, false);
 		BootstrapSkin::displayInternalFooter();
@@ -1390,8 +1341,7 @@ 
     
 	if (isset($_GET['hash'])) {
 		$urlhash = $_GET['hash'];
-	}
-	else {
+	} else {
 		$urlhash = "";
 	}
 	echo zoomPage($_GET['id'], $urlhash);
@@ -1399,8 +1349,7 @@ 
 	$tailscript = getTypeaheadSource(User::getAllUsernames(gGetDb()));
 	BootstrapSkin::displayInternalFooter($tailscript);
 	die();
-}
-elseif ($action == "logs") {
+} elseif ($action == "logs") {
 	global $baseurl;
 	
 	$filterUser = isset($_GET['filterUser']) && $_GET['filterUser'] != "" ? $_GET['filterUser'] : false;
@@ -1448,11 +1397,9 @@ 
 	if ($pageCount < $pageLimit) {
 		if ($pageData['lowpage'] == 1 && $pageData['hipage'] == $pageData['maxpage']) {
 			// nothing to do, we're already at max range.	
-		}
-		elseif ($pageData['lowpage'] == 1 && $pageData['hipage'] < $pageData['maxpage']) {
+		} elseif ($pageData['lowpage'] == 1 && $pageData['hipage'] < $pageData['maxpage']) {
 			$pageData['hipage'] = min($pageLimit, $pageData['maxpage']);
-		}
-		elseif ($pageData['lowpage'] > 1 && $pageData['hipage'] == $pageData['maxpage']) {
+		} elseif ($pageData['lowpage'] > 1 && $pageData['hipage'] == $pageData['maxpage']) {
 			$pageData['lowpage'] = max(1, $pageData['maxpage'] - $pageLimit + 1);
 		}
 	}
@@ -1475,8 +1422,7 @@ 
 	
 	BootstrapSkin::displayInternalFooter($tailscript);
 	die();
-}
-elseif ($action == "reserve") {
+} elseif ($action == "reserve") {
 	$database = gGetDb();
     
 	$database->transactionally(function() use ($database)
@@ -1552,8 +1498,7 @@ 
 	});
 	    
 	die();	
-}
-elseif ($action == "breakreserve") {
+} elseif ($action == "breakreserve") {
 	global $smarty;
     
 	$database = gGetDb();
@@ -1595,20 +1540,17 @@ 
 				});
                 
 				die();
-			}
-			else {
+			} else {
 				global $baseurl;
 				$smarty->assign("reservedUser", $reservedUser);
 				$smarty->assign("request", $request);
                 
 				$smarty->display("confirmations/breakreserve.tpl");
 			}
-		}
-		else {
+		} else {
 			echo "You cannot break " . htmlentities($reservedUser->getUsername()) . "'s reservation";
 		}
-	}
-	else {
+	} else {
 		$database->transactionally(function() use ($database, $request)
 		{
 			$request->setReserved(0);
@@ -1625,8 +1567,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();		
-}
-elseif ($action == "comment") {
+} elseif ($action == "comment") {
 	global $smarty;
     
 	$request = Request::getById($_GET['id'], gGetDb());
@@ -1634,8 +1575,7 @@ 
 	$smarty->display("commentform.tpl");
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "comment-add") {
+} elseif ($action == "comment-add") {
 	global $baseurl, $smarty;
     
 	$request = Request::getById($_POST['id'], gGetDb());
@@ -1680,8 +1620,7 @@ 
     
 	if (isset($_GET['hash'])) {
 		$urlhash = urlencode(htmlentities($_GET['hash']));
-	}
-	else {
+	} else {
 		$urlhash = "";
 	}
 
@@ -1695,8 +1634,7 @@ 
         
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "comment-quick") {
+} elseif ($action == "comment-quick") {
 	$request = Request::getById($_POST['id'], gGetDb());
 	if ($request == false) {
 		BootstrapSkin::displayAlertBox("Could not find request!", "alert-error", "Error", true, false);
@@ -1739,8 +1677,7 @@ 
 	Notification::commentCreated($comment);
     
 	header("Location: acc.php?action=zoom&id=" . $request->getId());
-}
-elseif ($action == "changepassword") {
+} elseif ($action == "changepassword") {
 	if ((!isset($_POST['oldpassword'])) || $_POST['oldpassword'] == "") {
 		//Throw an error if old password is not specified.
 		BootstrapSkin::displayAlertBox("You did not enter your old password.", "alert-error", "Error", true, false);
@@ -1777,8 +1714,7 @@ 
 	BootstrapSkin::displayAlertBox("Password successfully changed!", "alert-success", "", false, false);
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "ec") {
+} elseif ($action == "ec") {
 	// edit comment
   
 	global $smarty, $baseurl;
@@ -1820,15 +1756,13 @@ 
 		});
         
 		die();    
-	}
-	else {
+	} else {
 		$smarty->assign("comment", $comment);
 		$smarty->display("edit-comment.tpl");
 		BootstrapSkin::displayInternalFooter();
 		die();
 	}
-}
-elseif ($action == "sendtouser") {
+} elseif ($action == "sendtouser") {
 	global $baseurl;
     
 	$database = gGetDb();
@@ -1867,8 +1801,7 @@ 
 	Notification::requestReservationSent($requestObject, $user);
 	SessionAlert::success("Reservation sent successfully");
 	header("Location: $baseurl/acc.php?action=zoom&id=$request");
-}
-elseif ($action == "emailmgmt") {
+} elseif ($action == "emailmgmt") {
 	global $smarty, $createdid, $availableRequestStates;
     
 	/* New page for managing Emails, since I would rather not be handling editing
@@ -1945,8 +1878,7 @@ 
 				$emailTemplate->setDefaultAction(EmailTemplate::CREATED);
 				$emailTemplate->setActive(1);
 				$emailTemplate->setPreloadOnly(0);
-			}
-			else {
+			} else {
 				$emailTemplate->setDefaultAction($_POST['defaultaction']);
 				$emailTemplate->setActive(isset($_POST['active']));
 				$emailTemplate->setPreloadOnly(isset($_POST['preloadonly']));
@@ -2001,16 +1933,14 @@ 
  
 	if (count($inactiverows) > 0) {
 		$smarty->assign('displayinactive', true);
-	}
-	else {
+	} else {
 		$smarty->assign('displayinactive', false);
 	}
     
 	$smarty->display("email-management/main.tpl");
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "oauthdetach") {
+} elseif ($action == "oauthdetach") {
 	if ($enforceOAuth) {
 		BootstrapSkin::displayAccessDenied();
 		BootstrapSkin::displayInternalFooter();
@@ -2023,8 +1953,7 @@ 
 	$currentUser->detachAccount();
         
 	header("Location: {$baseurl}/acc.php?action=logout");
-}
-elseif ($action == "oauthattach") {
+} elseif ($action == "oauthattach") {
 	$database = gGetDb();
 	$database->transactionally(function() use ($database)
 	{
@@ -2051,8 +1980,7 @@ 
 			throw new TransactionException($ex->getMessage(), "Connection to Wikipedia failed.", "alert-error", 0, $ex);
 		}
 	});
-}
-elseif ($action == "listall") {
+} elseif ($action == "listall") {
     global $availableRequestStates, $enableEmailConfirm;
 
 	if (isset($_GET['status']) && isset($availableRequestStates[$_GET['status']])) {