mdaniels5757 / waca

users.php

Braces +6 added lines, -12 removed lines

@@ -154,8 +154,7 @@ 
 		$smarty->display("usermanagement/changelevel-reason.tpl");
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	else {
+	} else {
 		$user->demote($_POST['reason']);
 
 		BootstrapSkin::displayAlertBox( 
@@ -198,8 +197,7 @@ 
 			false);
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	elseif (!isset($_POST['reason'])) {
+	} elseif (!isset($_POST['reason'])) {
 		global $smarty;
 		$smarty->assign("user", $user);
 		$smarty->assign("status", "Suspended");
@@ -207,8 +205,7 @@ 
 		$smarty->display("usermanagement/changelevel-reason.tpl");
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	else {
+	} else {
 		$user->suspend($_POST['reason']);
 
 		Notification::userSuspended($user, $_POST['reason']);
@@ -301,8 +298,7 @@ 
 		$smarty->display("usermanagement/changelevel-reason.tpl");
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	else {
+	} else {
 		$user->decline($_POST['reason']);
 
 		Notification::userDeclined($user, $_POST['reason']);
@@ -346,8 +342,7 @@ 
 		$smarty->display("usermanagement/renameuser.tpl");
 		BootstrapSkin::displayInternalFooter();
 		die();
-	}
-	else {
+	} else {
 		if (!isset($_POST['newname']) || trim($_POST['newname']) == "") {
 			BootstrapSkin::displayAlertBox("The new username cannot be empty.", "alert-error", "Error", true, false);
 			BootstrapSkin::displayInternalFooter();
@@ -429,8 +424,7 @@ 
 		global $smarty;
 		$smarty->assign("user", $user);
 		$smarty->display("usermanagement/edituser.tpl");
-	}
-	else {
+	} else {
 		$database = gGetDb();
 		if (!$database->beginTransaction()) {
 			BootstrapSkin::displayAlertBox(

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;
         
 	// TODO: check blocked
@@ -196,8 +194,7 @@ 
 					0, 
 					$ex);
 			}
-		}
-		else {
+		} else {
 			global $baseurl;
 			Notification::userNew($newUser);
 			header("Location: {$baseurl}/acc.php?action=registercomplete");
@@ -205,19 +202,16 @@ 
 	});
     
 	die();
-}
-elseif ($action == "register") {
+} elseif ($action == "register") {
 	global $useOauthSignup, $smarty;
 	$smarty->assign("useOauthSignup", $useOauthSignup);
 	$smarty->display("registration/register.tpl");
 	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'])) {
@@ -246,14 +240,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();
@@ -267,8 +259,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", 
@@ -294,8 +285,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", 
@@ -304,8 +294,7 @@ 
             
 			BootstrapSkin::displayInternalFooter();
 			die();
-		}
-		else {
+		} else {
 			$hash = $user->getForgottenPasswordHash();
                        
 			$smarty->assign("user", $user);
@@ -337,8 +326,7 @@ 
 
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "login") {
+} elseif ($action == "login") {
 	global $baseurl, $smarty;
     
 	if (!isset($_POST['username'])) {
@@ -369,8 +357,7 @@ 
             
 			reattachOAuthAccount($user);
 		}
-	}
-	else {
+	} else {
 		global $enforceOAuth;
         
 		if ($enforceOAuth) {
@@ -453,8 +440,7 @@ 
 	}
     
 	header("Location: $baseurl/acc.php");
-}
-elseif ($action == "messagemgmt") {
+} elseif ($action == "messagemgmt") {
 	global $smarty;
     
 	if (isset($_GET['view'])) {
@@ -538,8 +524,7 @@ 
    
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "templatemgmt") {
+} elseif ($action == "templatemgmt") {
 	global $baseurl, $smarty;
     
 	if (isset($_GET['view'])) {
@@ -585,15 +570,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 = '';
 			}
@@ -619,8 +602,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());
@@ -629,8 +611,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();
@@ -707,8 +688,7 @@ 
             
 			header("Location: $baseurl/acc.php?action=templatemgmt");
 			die();
-		}
-		else {
+		} else {
 			$smarty->assign("template", $template);
 			$smarty->display("welcometemplate/edit.tpl");
             
@@ -724,8 +704,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "sban") {
+} elseif ($action == "sban") {
 	global $smarty;
     
 	// Checks whether the current user is an admin.
@@ -753,21 +732,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();
 	}
     
@@ -847,8 +823,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "unban") {
+} elseif ($action == "unban") {
 	global $smarty;
     
 	if (!isset($_GET['id']) || $_GET['id'] == "") {
@@ -887,8 +862,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)
@@ -907,15 +881,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'])) {
@@ -935,8 +907,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']);
@@ -944,8 +915,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']);
@@ -953,8 +923,7 @@ 
 			$row = $statement->fetch(PDO::FETCH_ASSOC);
 			$target = $row['name'];
 			$type = "Name";
-		}
-		else {
+		} else {
 			BootstrapSkin::displayAlertBox("Unknown ban type.", "alert-error");
 			BootstrapSkin::displayInternalFooter();
 			die();    
@@ -969,8 +938,7 @@ 
 		$smarty->assign("bantype", $type);
 		$smarty->assign("bantarget", trim($target));
 		$smarty->display("bans/banform.tpl");
-	}
-	else {
+	} else {
 		$bans = Ban::getActiveBans();
   
 		$smarty->assign("activebans", $bans);
@@ -979,8 +947,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)) {
@@ -1061,14 +1028,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'])) {
@@ -1082,8 +1047,7 @@ 
             
 			if ($mailisvalid === false) {
 				BootstrapSkin::displayAlertBox("Invalid email address", "alert-error", "Error!");
-			}
-			else {
+			} else {
 				$user->setEmail(trim($_POST['email']));
 			}
 		}
@@ -1104,20 +1068,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;
 	}
     
@@ -1183,8 +1144,7 @@ 
 	$ue = unserialize($userexist);
 	if (!isset ($ue['query']['users']['0']['missing'])) {
 		$exists = true;
-	}
-	else {
+	} else {
 		$exists = false;
 	}
 
@@ -1226,8 +1186,7 @@ 
 				$preloadTitle = $template->getName();
 				$preloadText = $template->getText();
 				$preloadAction = $template->getDefaultAction();
-			}
-			else {
+			} else {
 				$preloadText = "";
 				$preloadTitle = "";
 				$preloadAction = "";
@@ -1265,8 +1224,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);
 		}
 
@@ -1279,8 +1237,7 @@ 
 			if ($_POST['action'] == EmailTemplate::CREATED) {
 				$gem  = 'custom-y';
 				$crea = "Custom, Created";
-			}
-			else {
+			} else {
 				$gem  = 'custom-n';
 				$crea = "Custom, Not Created";
 			}
@@ -1291,8 +1248,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);
@@ -1300,8 +1256,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']);
@@ -1314,8 +1269,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);
@@ -1334,8 +1288,7 @@ 
 		echo defaultpage();
 		BootstrapSkin::displayInternalFooter();
 		die();		
-	}
-	else {
+	} else {
 		// Not a custom close, just a normal close
 	    
 		$request->setStatus('Closed');
@@ -1348,8 +1301,7 @@ 
 		
 		if ($gem == '0') {
 			$crea = "Dropped";
-		}
-		else {
+		} else {
 			$template = EmailTemplate::getById($gem, gGetDb());
 			$crea = $template->getName();
 		}
@@ -1370,8 +1322,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();
@@ -1380,8 +1331,7 @@ 
     
 	if (isset($_GET['hash'])) {
 		$urlhash = $_GET['hash'];
-	}
-	else {
+	} else {
 		$urlhash = "";
 	}
 	echo zoomPage($_GET['id'], $urlhash);
@@ -1389,8 +1339,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;
@@ -1438,11 +1387,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);
 		}
 	}
@@ -1465,8 +1412,7 @@ 
 	
 	BootstrapSkin::displayInternalFooter($tailscript);
 	die();
-}
-elseif ($action == "reserve") {
+} elseif ($action == "reserve") {
 	$database = gGetDb();
     
 	$database->transactionally(function() use ($database)
@@ -1542,8 +1488,7 @@ 
 	});
 	    
 	die();	
-}
-elseif ($action == "breakreserve") {
+} elseif ($action == "breakreserve") {
 	global $smarty;
     
 	$database = gGetDb();
@@ -1585,20 +1530,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);
@@ -1615,8 +1557,7 @@ 
     
 	BootstrapSkin::displayInternalFooter();
 	die();		
-}
-elseif ($action == "comment") {
+} elseif ($action == "comment") {
 	global $smarty;
     
 	$request = Request::getById($_GET['id'], gGetDb());
@@ -1624,8 +1565,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());
@@ -1670,8 +1610,7 @@ 
     
 	if (isset($_GET['hash'])) {
 		$urlhash = urlencode(htmlentities($_GET['hash']));
-	}
-	else {
+	} else {
 		$urlhash = "";
 	}
 
@@ -1685,8 +1624,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);
@@ -1729,8 +1667,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);
@@ -1767,8 +1704,7 @@ 
 	BootstrapSkin::displayAlertBox("Password successfully changed!", "alert-success", "", false, false);
 	BootstrapSkin::displayInternalFooter();
 	die();
-}
-elseif ($action == "ec") {
+} elseif ($action == "ec") {
 	// edit comment
   
 	global $smarty, $baseurl;
@@ -1810,15 +1746,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();
@@ -1857,8 +1791,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
@@ -1935,8 +1868,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']));
@@ -1991,16 +1923,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();
@@ -2013,8 +1943,7 @@ 
 	$currentUser->detachAccount();
         
 	header("Location: {$baseurl}/acc.php?action=logout");
-}
-elseif ($action == "oauthattach") {
+} elseif ($action == "oauthattach") {
 	$database = gGetDb();
 	$database->transactionally(function() use ($database)
 	{
@@ -2041,8 +1970,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']])) {