smrealms / smr

engine/Default/login_check_processing.php

Braces +2 added lines, -4 removed lines

@@ -3,8 +3,7 @@ 
 	// is account validated?
 	if (!$account->isValidated()) {
 		forward(create_container('skeleton.php', 'validate.php'));
-	}
-	else {
+	} else {
 		$var['CheckType'] = 'Announcements';
 	}
 }
@@ -17,8 +16,7 @@ 
 	// do we have announcements?
 	if ($db->nextRecord()) {
 		forward(create_container('skeleton.php', 'announcements.php'));
-	}
-	else {
+	} else {
 		$var['CheckType'] = 'Updates';
 	}
 }

engine/Default/alliance_list.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
 while ($db->nextRecord()) {
 	if ($db->getField('alliance_id') != $player->getAllianceID()) {
 		$container['body'] = 'alliance_roster.php';
-	}
-	else {
+	} else {
 		$container['body'] = 'alliance_mod.php';
 	}
 	$allianceID = $db->getInt('alliance_id');

engine/Default/military_payment_claim.php

Braces +3 added lines, -2 removed lines

@@ -3,10 +3,11 @@
 $template->assign('PageTopic','Military Payment Center');
 
 require_once(get_file_loc('menu_hq.inc'));
-if ($sector->hasHQ())
+if ($sector->hasHQ()) {
 	create_hq_menu();
-else
+} else {
 	create_ug_menu();
+}
 
 // We can only claim the payment once, so to prevent clobbering the message
 // upon AJAX refresh, we store it as a session variable when we first get it.

engine/Default/shop_goods.php

Braces +17 added lines, -15 removed lines

@@ -7,8 +7,9 @@ 
 $template->assign('Port', $port);
 
 $tradeable = checkPortTradeable($port,$player);
-if($tradeable!==true)
+if($tradeable!==true) {
 	create_error($tradeable);
+}
 
 // topic
 $template->assign('PageTopic','Port In Sector #'.$player->getSectorID());
@@ -23,17 +24,19 @@ 
 //The player is sent here after trading and sees this if his offer is accepted.
 if (!empty($var['trade_msg'])) {
 	$template->assign('TradeMsg', $var['trade_msg']);
-}
-elseif ($player->getLastPort() != $player->getSectorID()) {
+} elseif ($player->getLastPort() != $player->getSectorID()) {
 	// test if we are searched, but only if we hadn't a previous trade here
 
 	$base_chance = 15;
-	if ($port->hasGood(GOODS_SLAVES))
-		$base_chance -= 4;
-	if ($port->hasGood(GOODS_WEAPONS))
-		$base_chance -= 4;
-	if ($port->hasGood(GOODS_NARCOTICS))
-		$base_chance -= 4;
+	if ($port->hasGood(GOODS_SLAVES)) {
+			$base_chance -= 4;
+	}
+	if ($port->hasGood(GOODS_WEAPONS)) {
+			$base_chance -= 4;
+	}
+	if ($port->hasGood(GOODS_NARCOTICS)) {
+			$base_chance -= 4;
+	}
 
 	if ($ship->isUnderground()) {
 		$base_chance -= 4;
@@ -62,11 +65,11 @@ 
 					// because credits is 0 it will take money from bank
 					$player->decreaseBank(min($fine,$player->getBank()));
 					// leave insurance
-					if ($player->getBank() < 5000)
-						$player->setBank(5000);
+					if ($player->getBank() < 5000) {
+											$player->setBank(5000);
+					}
 				}
-			}
-			else {
+			} else {
 				$player->decreaseCredits($fine);
 			}
 
@@ -78,8 +81,7 @@ 
 			$ship->setCargo(GOODS_NARCOTICS, 0);
 			$account->log(LOG_TYPE_TRADING, 'Player gets caught with illegals', $player->getSectorID());
 
-		}
-		else {
+		} else {
 			$template->assign('IllegalsFound', false);
 			$player->increaseHOF(1,array('Trade','Search','Times Found Innocent'), HOF_PUBLIC);
 			$player->increaseAlignment(1);

engine/Default/message_blacklist.php

Braces +1 added lines, -2 removed lines

@@ -49,8 +49,7 @@
 	$PHP_OUTPUT.= '</table><br />';
 	$PHP_OUTPUT.= $form['submit'];
 	$PHP_OUTPUT.= '</form><br />';
-}
-else {
+} else {
 	$PHP_OUTPUT.= 'You are currently accepting all communications.<br />';
 }
 

engine/Default/alliance_set_op_processing.php

Braces +1 added lines, -2 removed lines

@@ -15,8 +15,7 @@
 
 	// NOTE: for simplicity we don't touch `player_has_unread_messages` here,
 	// so they may get an errant alliance message icon if logged in.
-}
-else {
+} else {
 	// schedule an op
 	if (empty($_POST['date'])) {
 		error_on_page('You must specify a date for the operation!');

engine/Default/trader_status.php

Braces +18 added lines, -6 removed lines

@@ -33,12 +33,24 @@
 $template->assign('HardwareHREF', SmrSession::getNewHREF($container));
 
 $hardware = [];
-if ($ship->canHaveScanner()) $hardware[] = 'Scanner';
-if ($ship->canHaveIllusion()) $hardware[] = 'Illusion Generator';
-if ($ship->canHaveCloak()) $hardware[] = 'Cloaking Device';
-if ($ship->canHaveJump()) $hardware[] = 'Jump Drive';
-if ($ship->canHaveDCS()) $hardware[] = 'Drone Scrambler';
-if (empty($hardware)) $hardware[] = 'none';
+if ($ship->canHaveScanner()) {
+	$hardware[] = 'Scanner';
+}
+if ($ship->canHaveIllusion()) {
+	$hardware[] = 'Illusion Generator';
+}
+if ($ship->canHaveCloak()) {
+	$hardware[] = 'Cloaking Device';
+}
+if ($ship->canHaveJump()) {
+	$hardware[] = 'Jump Drive';
+}
+if ($ship->canHaveDCS()) {
+	$hardware[] = 'Drone Scrambler';
+}
+if (empty($hardware)) {
+	$hardware[] = 'none';
+}
 $template->assign('Hardware', $hardware);
 
 $db->query('SELECT level_name,requirement FROM level WHERE requirement>' . $db->escapeNumber($player->getExperience()) . ' ORDER BY requirement ASC LIMIT 1');

engine/Default/contact_processing.php

Braces +1 added lines, -2 removed lines

@@ -18,8 +18,7 @@
 $container['url'] = 'skeleton.php';
 if (SmrSession::hasGame()) {
 	$container['body'] = 'current_sector.php';
-}
-else {
+} else {
 	$container['body'] = 'game_play.php';
 }
 

engine/Default/sector_move_processing.php

Braces +17 added lines, -13 removed lines

@@ -1,12 +1,14 @@ 
 <?php
 
-if ($var['target_sector'] == $player->getSectorID())
+if ($var['target_sector'] == $player->getSectorID()) {
 	forward(create_container('skeleton.php', $var['target_page']));
+}
 
-if($sector->getWarp() == $var['target_sector'])
+if($sector->getWarp() == $var['target_sector']) {
 	$turns = TURNS_PER_WARP;
-else
+} else {
 	$turns = TURNS_PER_SECTOR;
+}
 
 //allow hidden players (admins that don't play) to move without pinging, hitting mines, losing turns
 if (in_array($player->getAccountID(), Globals::getHiddenPlayers())) {
@@ -24,19 +26,23 @@ 
 $action = '';
 if(isset($_REQUEST['action'])) {
 	$action = $_REQUEST['action'];
-	if ($action == 'No')
-		forward(create_container('skeleton.php', $var['target_page']));
-}
+	if ($action == 'No') {
+			forward(create_container('skeleton.php', $var['target_page']));
+	}
+	}
 
 // you can't move while on planet
-if ($player->isLandedOnPlanet())
+if ($player->isLandedOnPlanet()) {
 	create_error('You can\'t activate your engine while you are on a planet!');
+}
 
-if ($player->getTurns() < $turns)
+if ($player->getTurns() < $turns) {
 	create_error('You don\'t have enough turns to move!');
+}
 
-if (!$sector->isLinked($var['target_sector']))
+if (!$sector->isLinked($var['target_sector'])) {
 	create_error('You cannot move to that sector!');
+}
 
 // If you bump into mines in the sector you are trying to leave...
 if ($player->getLastSectorID() != $var['target_sector']) {
@@ -59,8 +65,7 @@ 
 			$container = create_container('skeleton.php', 'current_sector.php');
 			$container['msg']= 'You have just flown past a sprinkle of mines.<br />Because of your newbie status you have been spared from the harsh reality of the forces.<br />It has cost you ' . $turns.' turn'.($turns==1?'':'s') . ' to navigate the minefield safely';
 			forward($container);
-		}
-		else {
+		} else {
 			$container = create_container('forces_attack_processing.php');
 			$container['action'] = 'bump';
 			$container['owner_id'] = $mine_owner_id;
@@ -121,8 +126,7 @@ 
 		$container = create_container('skeleton.php', 'current_sector.php');
 		$container['msg']= 'You have just flown past a sprinkle of mines.<br />Because of your newbie status you have been spared from the harsh reality of the forces.<br />It has cost you ' . $turns.' turn'.($turns==1?'':'s') . ' to navigate the minefield safely.';
 		forward($container);
-	}
-	else {
+	} else {
 		$container = create_container('forces_attack_processing.php');
 		$container['action'] = 'bump';
 		$container['owner_id'] = $mine_owner_id;