smrealms / smr

engine/Default/trader_search.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-$template->assign('PageTopic','Search Trader');
+$template->assign('PageTopic', 'Search Trader');
 $template->assign('TraderSearchHREF', SmrSession::getNewHREF(create_container('skeleton.php', 'trader_search_result.php')));
 
 if (isset($var['empty_result'])) {

engine/Default/mission_accept_processing.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 if (count($player->getMissions()) >= 3)
-	create_error('red','Error','You can only have 3 missions at a time.');
+	create_error('red', 'Error', 'You can only have 3 missions at a time.');
 
 
 $player->addMission($var['MissionID']);

engine/Default/buy_message_notifications.php

Spacing +4 added lines, -4 removed lines

@@ -1,18 +1,18 @@
 <?php
 require_once(get_file_loc('message.functions.inc'));
 
-if(isset($var['Message'])) {
-	$template->assign('Message',$var['Message']);
+if (isset($var['Message'])) {
+	$template->assign('Message', $var['Message']);
 }
 
-$template->assign('PageTopic','Message Notifications');
+$template->assign('PageTopic', 'Message Notifications');
 
 $container = create_container('buy_message_notifications_processing.php');
 
 // Presently only player messages are eligible for notifications
 $notifyTypeIDs = array(MSG_PLAYER);
 
-$messageBoxes = array ();
+$messageBoxes = array();
 foreach ($notifyTypeIDs as $messageTypeID) {
 	$messageBox = array();
 	$messageBox['Name'] = getMessageTypeNames($messageTypeID);

engine/Default/chat_sharing_processing.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	}
 
 	$gameId = (isset($_POST['all_games'])) ? '0' : $player->getGameID();
-	$db->query('INSERT INTO account_shares_info (to_account_id, from_account_id, game_id) VALUES ('. $db->escapeNumber($accountId) .','. $db->escapeNumber($player->getAccountID()) .','. $db->escapeNumber($gameId) .')');
+	$db->query('INSERT INTO account_shares_info (to_account_id, from_account_id, game_id) VALUES (' . $db->escapeNumber($accountId) . ',' . $db->escapeNumber($player->getAccountID()) . ',' . $db->escapeNumber($gameId) . ')');
 }
 
 // Process removing a "share to" account

engine/Default/chess.php

Spacing +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@ 
 <?php
 
-$chessGames =& ChessGame::getOngoingAccountGames($player->getAccountID());
+$chessGames = & ChessGame::getOngoingAccountGames($player->getAccountID());
 $template->assign('ChessGames', $chessGames);
 
 $playersChallenged = array($player->getAccountID() => true);
-foreach($chessGames as $chessGame) {
+foreach ($chessGames as $chessGame) {
 	$playersChallenged[$chessGame->getWhiteID()] = true;
 	$playersChallenged[$chessGame->getBlackID()] = true;
 }
@@ -14,13 +14,13 @@ 
 while ($db->nextRecord()) {
 	$players[$db->getInt('player_id')] = $db->getField('player_name');
 }
-$template->assign('PlayerList',$players);
+$template->assign('PlayerList', $players);
 
-if(ENABLE_NPCS_CHESS) {
+if (ENABLE_NPCS_CHESS) {
 	$npcs = array();
 	$db->query('SELECT player_id, player.player_name FROM player JOIN account USING(account_id) JOIN npc_logins USING(login) WHERE validated = ' . $db->escapeBoolean(true) . ' AND game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND account_id NOT IN (' . $db->escapeArray(array_keys($playersChallenged)) . ') ORDER BY player_name');
 	while ($db->nextRecord()) {
 		$npcs[$db->getInt('player_id')] = $db->getField('player_name');
 	}
-	$template->assign('NPCList',$npcs);
+	$template->assign('NPCList', $npcs);
 }

engine/Default/alliance_forces.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 if (!isset($var['alliance_id'])) {
-	SmrSession::updateVar('alliance_id',$player->getAllianceID());
+	SmrSession::updateVar('alliance_id', $player->getAllianceID());
 }
 
 $alliance = SmrAlliance::getAlliance($var['alliance_id'], $player->getGameID());

engine/Default/port_attack.php

Spacing +9 added lines, -9 removed lines

@@ -1,16 +1,16 @@
 <?php
 
-if(isset($var['results'])) {
+if (isset($var['results'])) {
 	$results = unserialize($var['results']);
-	$template->assign('FullPortCombatResults',$results);
-	$template->assign('AlreadyDestroyed',false);
+	$template->assign('FullPortCombatResults', $results);
+	$template->assign('AlreadyDestroyed', false);
 }
 else
-	$template->assign('AlreadyDestroyed',true);
-$template->assign('MinimalDisplay',false);
+	$template->assign('AlreadyDestroyed', true);
+$template->assign('MinimalDisplay', false);
 	
-if(isset($var['override_death']))
-	$template->assign('OverrideDeath',true);
+if (isset($var['override_death']))
+	$template->assign('OverrideDeath', true);
 else
-	$template->assign('OverrideDeath',false);
-$template->assign('Port',$sector->getPort());
+	$template->assign('OverrideDeath', false);
+$template->assign('Port', $sector->getPort());

engine/Default/bug_report_processing.php

Spacing +7 added lines, -7 removed lines

@@ -4,15 +4,15 @@
 $subject = $_REQUEST['subject'];
 $error_msg = $_REQUEST['error_msg'];
 $description = $_REQUEST['description'];
-$new_sub = '[Bug] '.$subject;
+$new_sub = '[Bug] ' . $subject;
 
-$message = 'Login: '.$account->getLogin().EOL.EOL.'-----------'.EOL.EOL.
-	'Account ID: '.$account->getAccountID().EOL.EOL.'-----------'.EOL.EOL.
-	'Description: '.$description.EOL.EOL.'-----------'.EOL.EOL.
-	'Steps to repeat: '.$steps.EOL.EOL.'-----------'.EOL.EOL.
-	'Error Message: '.$error_msg;
+$message = 'Login: ' . $account->getLogin() . EOL . EOL . '-----------' . EOL . EOL .
+	'Account ID: ' . $account->getAccountID() . EOL . EOL . '-----------' . EOL . EOL .
+	'Description: ' . $description . EOL . EOL . '-----------' . EOL . EOL .
+	'Steps to repeat: ' . $steps . EOL . EOL . '-----------' . EOL . EOL .
+	'Error Message: ' . $error_msg;
 
-if(is_object($player)) {
+if (is_object($player)) {
 	$player->sendMessageToBox(BOX_BUGS_REPORTED, $message);
 }
 else {

engine/Default/port_attack_processing.php

Spacing +18 added lines, -18 removed lines

@@ -4,23 +4,23 @@ 
 	create_error('You are under newbie protection!');
 if ($player->hasFederalProtection())
 	create_error('You are under federal protection!');
-if($player->isLandedOnPlanet())
+if ($player->isLandedOnPlanet())
 	create_error('You cannot attack ports whilst on a planet!');
 if ($player->getTurns() < TURNS_TO_SHOOT_PORT)
 	create_error('You do not have enough turns to attack this port!');
-if(!$ship->hasWeapons() && !$ship->hasCDs())
+if (!$ship->hasWeapons() && !$ship->hasCDs())
 	create_error('What are you going to do? Insult it to death?');
-if(!$player->canFight())
+if (!$player->canFight())
 	create_error('You are not allowed to fight!');
 
 $port = $sector->getPort();
 
-if(!$port->exists())
+if (!$port->exists())
 	create_error('This port does not exist.');
 
 
 if ($port->isDestroyed()) {
-	forward(create_container('skeleton.php','port_attack.php'));
+	forward(create_container('skeleton.php', 'port_attack.php'));
 }
 
 
@@ -36,41 +36,41 @@ 
 
 $attackers = $sector->getFightingTradersAgainstPort($player, $port);
 
-$port->attackedBy($player,$attackers);
+$port->attackedBy($player, $attackers);
 
 // take the turns and decloak all attackers
-foreach($attackers as $attacker) {
-	$attacker->takeTurns(TURNS_TO_SHOOT_PORT,0);
+foreach ($attackers as $attacker) {
+	$attacker->takeTurns(TURNS_TO_SHOOT_PORT, 0);
 	$attacker->getShip()->decloak();
 }
 
 foreach ($attackers as $attacker) {
-	$playerResults =& $attacker->shootPort($port);
-	$results['Attackers']['Traders'][$attacker->getAccountID()]  =& $playerResults;
+	$playerResults = & $attacker->shootPort($port);
+	$results['Attackers']['Traders'][$attacker->getAccountID()] = & $playerResults;
 	$results['Attackers']['TotalDamage'] += $playerResults['TotalDamage'];
 }
 $results['Attackers']['Downgrades'] = $port->checkForDowngrade($results['Attackers']['TotalDamage']);
-$results['Port'] =& $port->shootPlayers($attackers);
+$results['Port'] = & $port->shootPlayers($attackers);
 
-$account->log(LOG_TYPE_PORT_RAIDING, 'Player attacks port, the port does '.$results['Port']['TotalDamage'].', their team does ' . $results['Attackers']['TotalDamage'] .' and downgrades '.$results['Attackers']['Downgrades'].' levels.', $port->getSectorID());
+$account->log(LOG_TYPE_PORT_RAIDING, 'Player attacks port, the port does ' . $results['Port']['TotalDamage'] . ', their team does ' . $results['Attackers']['TotalDamage'] . ' and downgrades ' . $results['Attackers']['Downgrades'] . ' levels.', $port->getSectorID());
 
 $ship->removeUnderAttack(); //Don't show attacker the under attack message.
 $port->update();
 
 $serializedResults = serialize($results);
-$db->query('INSERT INTO combat_logs VALUES(\'\',' . $db->escapeNumber($player->getGameID()) . ',\'PORT\',' . $db->escapeNumber($port->getSectorID()) . ',' . $db->escapeNumber(TIME) . ',' . $db->escapeNumber($player->getAccountID()) . ',' . $db->escapeNumber($player->getAllianceID()) . ','.$db->escapeNumber(ACCOUNT_ID_PORT).',' . $db->escapeNumber(PORT_ALLIANCE_ID) . ',' . $db->escapeBinary(gzcompress($serializedResults)) . ')');
+$db->query('INSERT INTO combat_logs VALUES(\'\',' . $db->escapeNumber($player->getGameID()) . ',\'PORT\',' . $db->escapeNumber($port->getSectorID()) . ',' . $db->escapeNumber(TIME) . ',' . $db->escapeNumber($player->getAccountID()) . ',' . $db->escapeNumber($player->getAllianceID()) . ',' . $db->escapeNumber(ACCOUNT_ID_PORT) . ',' . $db->escapeNumber(PORT_ALLIANCE_ID) . ',' . $db->escapeBinary(gzcompress($serializedResults)) . ')');
 unserialize($serializedResults); //because of references we have to undo this.
-$logId = $db->escapeString('[ATTACK_RESULTS]'.$db->getInsertID());
+$logId = $db->escapeString('[ATTACK_RESULTS]' . $db->getInsertID());
 foreach ($attackers as $attacker) {
-	if(!$player->equals($attacker)) {
-		$db->query('REPLACE INTO sector_message VALUES(' . $db->escapeNumber($attacker->getAccountID()) . ',' . $db->escapeNumber($attacker->getGameID()) . ','.$logId.')');
+	if (!$player->equals($attacker)) {
+		$db->query('REPLACE INTO sector_message VALUES(' . $db->escapeNumber($attacker->getAccountID()) . ',' . $db->escapeNumber($attacker->getGameID()) . ',' . $logId . ')');
 	}
 }
 
-$container = create_container('skeleton.php','port_attack.php');
+$container = create_container('skeleton.php', 'port_attack.php');
 
 // If they died on the shot they get to see the results
-if($player->isDead()) {
+if ($player->isDead()) {
 	$container['override_death'] = TRUE;
 }