smrealms / smr

tools/discord/commands/seed.php

Braces +3 added lines, -1 removed lines

@@ -4,7 +4,9 @@
 
 $fn_seed = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 
 	$result = shared_channel_msg_seed($link->player);
 	$message->channel->sendMessage(join(EOL, $result));

tools/discord/commands/op.php

Braces +9 added lines, -3 removed lines

@@ -6,7 +6,9 @@ 
 
 $fn_op = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 	$player = $link->player;
 
 	// print info about the next op
@@ -16,7 +18,9 @@ 
 
 $fn_op_list = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 	$player = $link->player;
 
 	// print list of attendees
@@ -26,7 +30,9 @@ 
 
 $fn_op_turns = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 	$player = $link->player;
 
 	// print list of attendees

tools/discord/commands/game.php

Braces +3 added lines, -1 removed lines

@@ -2,7 +2,9 @@
 
 $fn_game = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 
 	$game = SmrGame::getGame($link->player->getGameID(), true);
 	$msg = "I am linked to game `" . $game->getDisplayName() . "` in this channel.";

tools/discord/commands/money.php

Braces +3 added lines, -1 removed lines

@@ -4,7 +4,9 @@
 
 $fn_money = function($message) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 
 	$result = shared_channel_msg_money($link->player);
 	if ($result) {

tools/discord/commands/forces.php

Braces +3 added lines, -1 removed lines

@@ -4,7 +4,9 @@
 
 $fn_forces = function($message, $params) {
 	$link = new GameLink($message->channel, $message->author);
-	if (!$link->valid) return;
+	if (!$link->valid) {
+		return;
+	}
 
 	// print the next expiring forces
 	$option = isset($params[0]) ? $params[0] : null;

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,8 +22,7 @@ 
 	$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 = trim($_REQUEST['name']);
 	if (empty($name)) {
 		create_error('You cannot leave your planet nameless!');
@@ -31,8 +32,7 @@ 
 	$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();

engine/Default/planet_financial_processing.php

Braces +15 added lines, -11 removed lines

@@ -1,33 +1,37 @@
 <?php
-if (!$player->isLandedOnPlanet())
+if (!$player->isLandedOnPlanet()) {
 	create_error('You are not on a planet!');
+}
 $planet = $player->getSectorPlanet();
 $action = $_REQUEST['action'];
 
 // Player has requested a planetary fund transaction
 if ($action == 'Deposit' || $action == 'Withdraw') {
 	$amount = $_REQUEST['amount'];
-	if (!is_numeric($amount))
-		create_error('Numbers only please!');
+	if (!is_numeric($amount)) {
+			create_error('Numbers only please!');
+	}
 
 	// only whole numbers allowed
 	$amount = floor($amount);
 
 	// no negative amounts are allowed
-	if ($amount <= 0)
-		create_error('You must actually enter an amount > 0!');
+	if ($amount <= 0) {
+			create_error('You must actually enter an amount > 0!');
+	}
 
 	if ($action == 'Deposit') {
-		if ($player->getCredits() < $amount)
-			create_error('You don\'t own that much money!');
+		if ($player->getCredits() < $amount) {
+					create_error('You don\'t own that much money!');
+		}
 
 		$player->decreaseCredits($amount);
 		$planet->increaseCredits($amount);
 		$account->log(LOG_TYPE_BANK, 'Player puts ' . $amount . ' credits on planet', $player->getSectorID());
-	}
-	elseif ($action == 'Withdraw') {
-		if ($planet->getCredits() < $amount)
-			create_error('There are not enough credits in the planetary account!');
+	} elseif ($action == 'Withdraw') {
+		if ($planet->getCredits() < $amount) {
+					create_error('There are not enough credits in the planetary account!');
+		}
 
 		$player->increaseCredits($amount);
 		$planet->decreaseCredits($amount);

engine/Default/bank_anon_detail_processing.php

Braces +6 added lines, -8 removed lines

@@ -21,8 +21,7 @@ 
 	$db->query('SELECT transaction_id FROM anon_bank_transactions WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND anon_id = ' . $db->escapeNumber($account_num) . ' ORDER BY transaction_id DESC LIMIT 1');
 	if ($db->nextRecord()) {
 		$trans_id = $db->getInt('transaction_id') + 1;
-	}
-	else {
+	} else {
 		$trans_id = 1;
 	}
 	$db->query('INSERT INTO anon_bank_transactions (account_id, game_id, anon_id, transaction_id, transaction, amount, time) ' .
@@ -32,17 +31,16 @@ 
 
 	// log action
 	$account->log(LOG_TYPE_BANK, 'Deposits ' . $amount . ' credits in anonymous account #' . $account_num, $player->getSectorID());
-}
-else {
+} else {
 	$db->query('SELECT * FROM anon_bank WHERE anon_id = ' . $db->escapeNumber($account_num) . ' AND game_id = ' . $db->escapeNumber($player->getGameID()));
 	$db->nextRecord();
-	if ($db->getField('amount') < $amount)
-		create_error('You don\'t have that much money on your account!');
+	if ($db->getField('amount') < $amount) {
+			create_error('You don\'t have that much money on your account!');
+	}
 	$db->query('SELECT transaction_id FROM anon_bank_transactions WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND anon_id = ' . $db->escapeNumber($account_num) . ' ORDER BY transaction_id DESC LIMIT 1');
 	if ($db->nextRecord()) {
 		$trans_id = $db->getField('transaction_id') + 1;
-	}
-	else {
+	} else {
 		$trans_id = 1;
 	}
 	$db->query('INSERT INTO anon_bank_transactions (account_id, game_id, anon_id, transaction_id, transaction, amount, time)

engine/Default/feature_request_vote_processing.php

Braces +10 added lines, -8 removed lines

@@ -10,20 +10,22 @@
 		}
 		$db->query(substr($query, 0, -1));
 	}
-	if (!empty($_REQUEST['favourite']) && is_numeric($_REQUEST['favourite']))
-		$db->query('REPLACE INTO account_votes_for_feature VALUES(' . $db->escapeNumber($account->getAccountID()) . ', ' . $db->escapeNumber($_REQUEST['favourite']) . ',\'FAVOURITE\')');
+	if (!empty($_REQUEST['favourite']) && is_numeric($_REQUEST['favourite'])) {
+			$db->query('REPLACE INTO account_votes_for_feature VALUES(' . $db->escapeNumber($account->getAccountID()) . ', ' . $db->escapeNumber($_REQUEST['favourite']) . ',\'FAVOURITE\')');
+	}
 
 	forward(create_container('skeleton.php', 'feature_request.php'));
-}
-else if ($_REQUEST['action'] == 'Set Status') {
+} else if ($_REQUEST['action'] == 'Set Status') {
 	if (empty($_REQUEST['status'])) {
 		create_error('You have to select a status to set');
 	}
 	$status = $_REQUEST['status'];
-	if (empty($_REQUEST['set_status_ids']))
-		create_error('You have to select a feature');
-	if (!$account->hasPermission(PERMISSION_MODERATE_FEATURE_REQUEST))
-		create_error('You do not have permission to do that');
+	if (empty($_REQUEST['set_status_ids'])) {
+			create_error('You have to select a feature');
+	}
+	if (!$account->hasPermission(PERMISSION_MODERATE_FEATURE_REQUEST)) {
+			create_error('You do not have permission to do that');
+	}
 
 	$db->query('UPDATE feature_request fr SET status = ' . $db->escapeString($status) . '
 			, fav = (