smrealms / smr

engine/Default/planet_kick_processing.php

Braces +4 added lines, -2 removed lines

@@ -1,12 +1,14 @@
 <?php
-if (!$player->isLandedOnPlanet())
+if (!$player->isLandedOnPlanet()) {
 	create_error('You are not on a planet!');
+}
 $planet = $player->getSectorPlanet();
 
 $planetPlayer = SmrPlayer::getPlayer($var['account_id'], $player->getGameID());
 $owner = $planet->getOwner();
-if ($owner->getAllianceID() != $player->getAllianceID())
+if ($owner->getAllianceID() != $player->getAllianceID()) {
 	create_error('You can not kick someone off a planet your alliance does not own!');
+}
 $message = 'You have been kicked from ' . $planet->getName() . ' in ' . Globals::getSectorBBLink($player->getSectorID());
 $player->sendMessage($planetPlayer->getAccountID(), MSG_PLAYER, $message, false);
 

engine/Default/galactic_post_delete.php

Braces +1 added lines, -2 removed lines

@@ -2,8 +2,7 @@
 $db2 = new SmrMySqlDatabase();
 if (isset($var['article'])) {
 	$db->query('DELETE FROM galactic_post_article WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND article_id = '.$db->escapeNumber($var['id']));
-}
-else {
+} else {
 	//we are deleting an entire paper
 	//find out which articles need to be deleted as well
 	$db->query('SELECT * FROM galactic_post_paper_content WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND paper_id = '.$db->escapeNumber($var['id']));

engine/Default/galactic_post_delete_confirm.php

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
 	$PHP_OUTPUT.=create_echo_form($container);
 	$PHP_OUTPUT.=create_submit('No');
 	$PHP_OUTPUT.=('</form>');
-}
-else {
+} else {
 	$db->query('SELECT * FROM galactic_post_paper WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND paper_id = '.$db->escapeNumber($var['id']));
 	$db->nextRecord();
 	$title = $db->getField('title');

engine/Default/galactic_post_view_article.php

Braces +2 added lines, -2 removed lines

@@ -14,9 +14,9 @@
 if ($db->getNumRows()) {
 	$PHP_OUTPUT.=('It is your responsibility to make sure ALL HTML tags are closed!<br />');
 	$PHP_OUTPUT.=('You have the following articles to view.<br /><br />');
-}
-else
+} else {
 	$PHP_OUTPUT.=('There are no articles to view');
+}
 
 while ($db->nextRecord()) {
 	$db2->query('SELECT * FROM galactic_post_paper_content WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND article_id = ' . $db->escapeNumber($db->getInt('article_id')));

engine/Default/game_join.php

Braces +14 added lines, -9 removed lines

@@ -7,9 +7,10 @@ 
 // do we need credits for this game?
 if ($game->getCreditsNeeded() > 0) {
 	// do we have enough
-	if ($account->getTotalSmrCredits() < $game->getCreditsNeeded())
-		create_error('Sorry you do not have enough SMR Credits to play this game.<br />To get SMR credits you need to donate to SMR.');
-}
+	if ($account->getTotalSmrCredits() < $game->getCreditsNeeded()) {
+			create_error('Sorry you do not have enough SMR Credits to play this game.<br />To get SMR credits you need to donate to SMR.');
+	}
+	}
 
 // is the game already full?
 if ($game->getTotalPlayers() >= $game->getMaxPlayers()) {
@@ -19,8 +20,9 @@ 
 //if (TIME < $game['StartDate'])
 //	create_error('You want to join a game that hasn\'t started yet?');
 
-if (TIME > $game->getEndDate())
+if (TIME > $game->getEndDate()) {
 	create_error('You want to join a game that is already over?');
+}
 
 $template->assign('PageTopic', 'Join Game: ' . $game->getDisplayName());
 
@@ -31,10 +33,10 @@ 
 		$raceDescriptions.=('\'' . str_replace('\'','\\\'"',$race['Description']) . '\'');
 		$first = false;
 
+	} else {
+			$raceDescriptions.=(', \'' . str_replace('\'','\\\'',$race['Description']) . '\'');
+	}
 	}
-	else
-		$raceDescriptions.=(', \'' . str_replace('\'','\\\'',$race['Description']) . '\'');
-}
 $template->assign('RaceDescriptions',$raceDescriptions);
 
 
@@ -42,8 +44,9 @@ 
 $container = array();
 $container['game_id'] = $var['game_id'];
 $container['url'] = 'game_join_processing.php';
-if (TIME >= $game->getStartDate())
+if (TIME >= $game->getStartDate()) {
 	$template->assign('JoinGameFormHref',SmrSession::getNewHREF($container));
+}
 
 $db2 = new SmrMySqlDatabase();
 //this prevents multiple races appearing when there is more than 1 game
@@ -62,7 +65,9 @@ 
 	$race_name = substr(stripslashes($db->getField('location_name')), 0, -13);
 
 	$curr_race_id = $db->getField('location_type_id') - 101;
-	if (in_array($curr_race_id, $only)) continue;
+	if (in_array($curr_race_id, $only)) {
+		continue;
+	}
 	$only[] = $curr_race_id;
 	// get number of traders in game
 	$db2->query('SELECT count(*) as number_of_race FROM player WHERE race_id = '.$db2->escapeNumber($curr_race_id).' AND game_id = ' . $db2->escapeNumber($var['game_id']));

engine/Default/bar_read_wall.php

Braces +1 added lines, -2 removed lines

@@ -7,8 +7,7 @@
 $db->query('SELECT message_id FROM bar_wall WHERE sector_id = ' . $db->escapeNumber($sector->getSectorID()) . ' AND game_id = ' . $db->escapeNumber($player->getGameID()) . ' ORDER BY message_id DESC');
 if ($db->nextRecord()) {
 	$amount = $db->getInt('message_id') + 1;
-}
-else {
+} else {
 	$amount = 1;
 }
 $wall = $_REQUEST['wall'];

engine/Default/feature_request_comments.php

Braces +5 added lines, -3 removed lines

@@ -1,6 +1,7 @@ 
 <?php
-if (!Globals::isFeatureRequestOpen())
+if (!Globals::isFeatureRequestOpen()) {
 	create_error('Feature requests are currently not being accepted.');
+}
 
 $template->assign('PageTopic','Feature Request Comments');
 
@@ -32,8 +33,9 @@ 
 								'Time' => date(DATE_FULL_SHORT,$db->getField('posting_time')),
 								'Anonymous' => $db->getBoolean('anonymous')
 		);
-		if($featureModerator || !$db->getBoolean('anonymous'))
-			$featureRequestComments[$commentID]['PosterAccount'] = SmrAccount::getAccount($db->getField('poster_id'));
+		if($featureModerator || !$db->getBoolean('anonymous')) {
+					$featureRequestComments[$commentID]['PosterAccount'] = SmrAccount::getAccount($db->getField('poster_id'));
+		}
 	}
 	$template->assign('Comments', $featureRequestComments);
 }

engine/Default/message_send_processing.php

Braces +9 added lines, -9 removed lines

@@ -4,18 +4,20 @@ 
 
 if($_REQUEST['action'] == 'Preview message') {
 	$container = create_container('skeleton.php');
-	if(isset($var['alliance_id']))
-		$container['body'] = 'alliance_broadcast.php';
-	else
-		$container['body'] = 'message_send.php';
+	if(isset($var['alliance_id'])) {
+			$container['body'] = 'alliance_broadcast.php';
+	} else {
+			$container['body'] = 'message_send.php';
+	}
 	transfer('receiver');
 	transfer('alliance_id');
 	$container['preview'] = $message;
 	forward($container);
 }
 
-if (empty($message))
+if (empty($message)) {
 	create_error('You have to enter a message to send!');
+}
 
 if(isset($var['alliance_id'])) {
 	$db->query('SELECT account_id FROM player
@@ -26,11 +28,9 @@ 
 		$player->sendMessage($db->getField('account_id'), MSG_ALLIANCE, $message, false);
 	}
 	$player->sendMessage($player->getAccountID(), MSG_ALLIANCE, $message, true, false);
-}
-else if (!empty($var['receiver'])) {
+} else if (!empty($var['receiver'])) {
 	$player->sendMessage($var['receiver'], MSG_PLAYER, $message);
-}
-else {
+} else {
 	$player->sendGlobalMessage($message);
 }
 

engine/Default/planet_ownership_processing.php

Braces +7 added lines, -7 removed lines

@@ -1,14 +1,16 @@ 
 <?php
-if (!$player->isLandedOnPlanet())
+if (!$player->isLandedOnPlanet()) {
 	create_error('You are not on a planet!');
+}
 // get a planet from the sector where the player is in
 $planet = $player->getSectorPlanet();
 $action = $_REQUEST['action'];
 $password = isset($_REQUEST['password'])?$_REQUEST['password']:'';
 
 if ($action == 'Take Ownership') {
-	if ($planet->hasOwner() && $planet->getPassword() != $password)
-		create_error('You are not allowed to take ownership!');
+	if ($planet->hasOwner() && $planet->getPassword() != $password) {
+			create_error('You are not allowed to take ownership!');
+	}
 
 	// delete all previous ownerships
 	$db->query('UPDATE planet SET owner_id = 0, password = NULL
@@ -20,16 +22,14 @@ 
 	$planet->removePassword();
 	$planet->update();
 	$account->log(LOG_TYPE_PLANETS, 'Player takes ownership of planet.', $player->getSectorID());
-}
-else if ($action == 'Rename') {
+} else if ($action == 'Rename') {
 	$name = $_REQUEST['name'];
 	// rename planet
 	$planet->setName($name);
 	$planet->update();
 	$account->log(LOG_TYPE_PLANETS, 'Player renames planet to '.$name.'.', $player->getSectorID());
 
-}
-else if ($action == 'Set Password') {
+} else if ($action == 'Set Password') {
 	// set password
 	$planet->setPassword($password);
 	$planet->update();