Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Completed
Branch master (287493)
by Dan
06:08
created
engine/Default/game_join.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 $game = SmrGame::getGame($var['game_id']);
4 4
 
5
-$template->assign('Game',$game);
5
+$template->assign('Game', $game);
6 6
 
7 7
 // do we need credits for this game?
8 8
 if ($game->getCreditsNeeded() > 0) {
@@ -24,18 +24,18 @@  discard block
 block discarded – undo
24 24
 
25 25
 $template->assign('PageTopic', 'Join Game: ' . $game->getDisplayName());
26 26
 
27
-$raceDescriptions='';
27
+$raceDescriptions = '';
28 28
 $first = true;
29 29
 foreach (Globals::getRaces() as $race) {
30 30
 	if ($first) {
31
-		$raceDescriptions.=('\'' . str_replace('\'','\\\'"',$race['Description']) . '\'');
31
+		$raceDescriptions .= ('\'' . str_replace('\'', '\\\'"', $race['Description']) . '\'');
32 32
 		$first = false;
33 33
 
34 34
 	}
35 35
 	else
36
-		$raceDescriptions.=(', \'' . str_replace('\'','\\\'',$race['Description']) . '\'');
36
+		$raceDescriptions .= (', \'' . str_replace('\'', '\\\'', $race['Description']) . '\'');
37 37
 }
38
-$template->assign('RaceDescriptions',$raceDescriptions);
38
+$template->assign('RaceDescriptions', $raceDescriptions);
39 39
 
40 40
 
41 41
 // create a container that will hold next url and additional variables.
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 $container['game_id'] = $var['game_id'];
44 44
 $container['url'] = 'game_join_processing.php';
45 45
 if (TIME >= $game->getStartDate())
46
-	$template->assign('JoinGameFormHref',SmrSession::getNewHREF($container));
46
+	$template->assign('JoinGameFormHref', SmrSession::getNewHREF($container));
47 47
 
48 48
 $db2 = new SmrMySqlDatabase();
49 49
 //this prevents multiple races appearing when there is more than 1 game
@@ -51,8 +51,8 @@  discard block
 block discarded – undo
51 51
 // get all available hq's
52 52
 $db->query('SELECT location_name, location_type_id
53 53
 			FROM location JOIN location_type USING(location_type_id)
54
-			WHERE location_type_id > '.$db->escapeNumber(UNDERGROUND).'
55
-				AND location_type_id < '.$db->escapeNumber(FED).'
54
+			WHERE location_type_id > '.$db->escapeNumber(UNDERGROUND) . '
55
+				AND location_type_id < '.$db->escapeNumber(FED) . '
56 56
 				AND game_id = ' . $db->escapeNumber($var['game_id']) . '
57 57
 			ORDER BY location_type_id');
58 58
 $races = array();
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	if (in_array($curr_race_id, $only)) continue;
66 66
 	$only[] = $curr_race_id;
67 67
 	// get number of traders in game
68
-	$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']));
68
+	$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']));
69 69
 	$db2->nextRecord();
70 70
 
71 71
 	$races[$curr_race_id]['ID'] = $curr_race_id;
@@ -73,10 +73,10 @@  discard block
 block discarded – undo
73 73
 	$races[$curr_race_id]['NumberOfPlayers'] = $db2->getInt('number_of_race');
74 74
 	$races[$curr_race_id]['Selected'] = false;
75 75
 }
76
-if(count($races) > 1) {
76
+if (count($races) > 1) {
77 77
 	do {
78 78
 		$raceKey = array_rand($races);
79
-	} while($races[$raceKey]['ID'] == RACE_ALSKANT);
79
+	} while ($races[$raceKey]['ID'] == RACE_ALSKANT);
80 80
 	$races[$raceKey]['Selected'] = true;
81 81
 }
82
-$template->assign('Races',$races);
82
+$template->assign('Races', $races);
Please login to merge, or discard this patch.
engine/Default/bar_read_wall.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-$template->assign('PageTopic','Reading The Wall');
3
+$template->assign('PageTopic', 'Reading The Wall');
4 4
 
5 5
 Menu::bar();
6 6
 
@@ -17,28 +17,28 @@  discard block
 block discarded – undo
17 17
 }
18 18
 $db->query('SELECT * FROM bar_wall WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' AND sector_id = ' . $db->escapeNumber($player->getSectorID()) . ' ORDER BY time DESC');
19 19
 if ($db->getNumRows()) {
20
-	$PHP_OUTPUT.=('<table class="standard">');
21
-	$PHP_OUTPUT.=('<tr>');
22
-	$PHP_OUTPUT.=('<th align="center">Time written</th>');
23
-	$PHP_OUTPUT.=('<th align="center">Message</th>');
24
-	$PHP_OUTPUT.=('</tr>');
20
+	$PHP_OUTPUT .= ('<table class="standard">');
21
+	$PHP_OUTPUT .= ('<tr>');
22
+	$PHP_OUTPUT .= ('<th align="center">Time written</th>');
23
+	$PHP_OUTPUT .= ('<th align="center">Message</th>');
24
+	$PHP_OUTPUT .= ('</tr>');
25 25
 
26 26
 	while ($db->nextRecord()) {
27 27
 		$time = $db->getInt('time');
28 28
 		$message_on_wall = $db->getField('message');
29 29
 
30
-		$PHP_OUTPUT.=('<tr>');
31
-		$PHP_OUTPUT.=('<td align="center"><b> ' . date(DATE_FULL_SHORT, $time) . ' </b></td>');
32
-		$PHP_OUTPUT.=('<td align="center"><b>'.$message_on_wall.'</b></td>');
33
-		$PHP_OUTPUT.=('</tr>');
30
+		$PHP_OUTPUT .= ('<tr>');
31
+		$PHP_OUTPUT .= ('<td align="center"><b> ' . date(DATE_FULL_SHORT, $time) . ' </b></td>');
32
+		$PHP_OUTPUT .= ('<td align="center"><b>' . $message_on_wall . '</b></td>');
33
+		$PHP_OUTPUT .= ('</tr>');
34 34
 	}
35
-	$PHP_OUTPUT.=('</table>');
35
+	$PHP_OUTPUT .= ('</table>');
36 36
 }
37
-$template->assign('PageTopic','Write on the wall');
37
+$template->assign('PageTopic', 'Write on the wall');
38 38
 
39
-$PHP_OUTPUT.=('<br />');
39
+$PHP_OUTPUT .= ('<br />');
40 40
 
41
-$PHP_OUTPUT.=create_echo_form(create_container('skeleton.php', 'bar_read_wall.php'));
42
-$PHP_OUTPUT.=('<textarea spellcheck="true" name="wall" id="InputFieldsText"></textarea><br /><br />');
43
-$PHP_OUTPUT.=create_submit('Write it');
44
-$PHP_OUTPUT.=('</form>');
41
+$PHP_OUTPUT .= create_echo_form(create_container('skeleton.php', 'bar_read_wall.php'));
42
+$PHP_OUTPUT .= ('<textarea spellcheck="true" name="wall" id="InputFieldsText"></textarea><br /><br />');
43
+$PHP_OUTPUT .= create_submit('Write it');
44
+$PHP_OUTPUT .= ('</form>');
Please login to merge, or discard this patch.
engine/Default/planet_main.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 //echo the dump cargo message or other message.
6 6
 if (isset($var['errorMsg'])) {
7
-	$template->assign('ErrorMsg',$var['errorMsg']);
7
+	$template->assign('ErrorMsg', $var['errorMsg']);
8 8
 }
9 9
 if (isset($var['msg'])) {
10 10
 	$template->assign('Msg', bbifyMessage($var['msg']));
@@ -12,4 +12,4 @@  discard block
 block discarded – undo
12 12
 
13 13
 doTickerAssigns($template, $player, $db);
14 14
 
15
-$template->assign('LaunchFormLink',SmrSession::getNewHREF(create_container('planet_launch_processing.php', '')));
15
+$template->assign('LaunchFormLink', SmrSession::getNewHREF(create_container('planet_launch_processing.php', '')));
Please login to merge, or discard this patch.
engine/Default/feature_request_comments.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -2,11 +2,11 @@  discard block
 block discarded – undo
2 2
 if (!Globals::isFeatureRequestOpen())
3 3
 	create_error('Feature requests are currently not being accepted.');
4 4
 
5
-$template->assign('PageTopic','Feature Request Comments');
5
+$template->assign('PageTopic', 'Feature Request Comments');
6 6
 
7 7
 $container = $var;
8 8
 $container['body'] = 'feature_request.php';
9
-$template->assign('BackHref',SmrSession::getNewHREF($container));
9
+$template->assign('BackHref', SmrSession::getNewHREF($container));
10 10
 
11 11
 $db->query('SELECT *
12 12
 			FROM feature_request
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 			ORDER BY comment_id ASC');
16 16
 if ($db->getNumRows() > 0) {
17 17
 	$featureModerator = $account->hasPermission(PERMISSION_MODERATE_FEATURE_REQUEST);
18
-	$template->assign('FeatureModerator',$featureModerator);
18
+	$template->assign('FeatureModerator', $featureModerator);
19 19
 
20 20
 	// variables needed to set the status for this feature request
21 21
 	if ($featureModerator) {
@@ -29,10 +29,10 @@  discard block
 block discarded – undo
29 29
 		$featureRequestComments[$commentID] = array(
30 30
 								'CommentID' => $commentID,
31 31
 								'Message' => $db->getField('text'),
32
-								'Time' => date(DATE_FULL_SHORT,$db->getField('posting_time')),
32
+								'Time' => date(DATE_FULL_SHORT, $db->getField('posting_time')),
33 33
 								'Anonymous' => $db->getBoolean('anonymous')
34 34
 		);
35
-		if($featureModerator || !$db->getBoolean('anonymous'))
35
+		if ($featureModerator || !$db->getBoolean('anonymous'))
36 36
 			$featureRequestComments[$commentID]['PosterAccount'] = SmrAccount::getAccount($db->getField('poster_id'));
37 37
 	}
38 38
 	$template->assign('Comments', $featureRequestComments);
@@ -41,4 +41,4 @@  discard block
 block discarded – undo
41 41
 $container = $var;
42 42
 $container['url'] = 'feature_request_comment_processing.php';
43 43
 unset($container['body']);
44
-$template->assign('FeatureRequestCommentFormHREF',SmrSession::getNewHREF($container));
44
+$template->assign('FeatureRequestCommentFormHREF', SmrSession::getNewHREF($container));
Please login to merge, or discard this patch.
engine/Default/message_send_processing.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -1,10 +1,10 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-$message = htmlentities(trim($_REQUEST['message']),ENT_COMPAT,'utf-8');
3
+$message = htmlentities(trim($_REQUEST['message']), ENT_COMPAT, 'utf-8');
4 4
 
5
-if($_REQUEST['action'] == 'Preview message') {
5
+if ($_REQUEST['action'] == 'Preview message') {
6 6
 	$container = create_container('skeleton.php');
7
-	if(isset($var['alliance_id']))
7
+	if (isset($var['alliance_id']))
8 8
 		$container['body'] = 'alliance_broadcast.php';
9 9
 	else
10 10
 		$container['body'] = 'message_send.php';
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 if (empty($message))
18 18
 	create_error('You have to enter a message to send!');
19 19
 
20
-if(isset($var['alliance_id'])) {
20
+if (isset($var['alliance_id'])) {
21 21
 	$db->query('SELECT account_id FROM player
22 22
 				WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . '
23 23
 				AND alliance_id = ' . $var['alliance_id'] . '
@@ -34,8 +34,8 @@  discard block
 block discarded – undo
34 34
 	$player->sendGlobalMessage($message);
35 35
 }
36 36
 
37
-$container=create_container('skeleton.php');
38
-if(isset($var['alliance_id'])) {
37
+$container = create_container('skeleton.php');
38
+if (isset($var['alliance_id'])) {
39 39
 	$container['body'] = 'alliance_roster.php';
40 40
 	transfer('alliance_id');
41 41
 } else {
Please login to merge, or discard this patch.
engine/Default/planet_ownership_processing.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 // get a planet from the sector where the player is in
5 5
 $planet = $player->getSectorPlanet();
6 6
 $action = $_REQUEST['action'];
7
-$password = isset($_REQUEST['password'])?$_REQUEST['password']:'';
7
+$password = isset($_REQUEST['password']) ? $_REQUEST['password'] : '';
8 8
 
9 9
 if ($action == 'Take Ownership') {
10 10
 	if ($planet->hasOwner() && $planet->getPassword() != $password)
@@ -26,14 +26,14 @@  discard block
 block discarded – undo
26 26
 	// rename planet
27 27
 	$planet->setName($name);
28 28
 	$planet->update();
29
-	$account->log(LOG_TYPE_PLANETS, 'Player renames planet to '.$name.'.', $player->getSectorID());
29
+	$account->log(LOG_TYPE_PLANETS, 'Player renames planet to ' . $name . '.', $player->getSectorID());
30 30
 
31 31
 }
32 32
 else if ($action == 'Set Password') {
33 33
 	// set password
34 34
 	$planet->setPassword($password);
35 35
 	$planet->update();
36
-	$account->log(LOG_TYPE_PLANETS, 'Player sets planet password to '.$password, $player->getSectorID());
36
+	$account->log(LOG_TYPE_PLANETS, 'Player sets planet password to ' . $password, $player->getSectorID());
37 37
 }
38 38
 
39 39
 forward(create_container('skeleton.php', 'planet_ownership.php'));
Please login to merge, or discard this patch.
engine/Default/trader_examine.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,14 +1,14 @@
 block discarded – undo
1 1
 <?php
2 2
 
3 3
 // Get the player we're attacking
4
-$targetPlayer = SmrPlayer::getPlayer($var['target'],$player->getGameID());
4
+$targetPlayer = SmrPlayer::getPlayer($var['target'], $player->getGameID());
5 5
 
6
-if($targetPlayer->isDead()) {
7
-	$container=create_container('skeleton.php', 'current_sector.php');
6
+if ($targetPlayer->isDead()) {
7
+	$container = create_container('skeleton.php', 'current_sector.php');
8 8
 	$container['msg'] = '<span class="red bold">ERROR:</span> Target already dead.';
9 9
 	forward($container);
10 10
 }
11 11
 
12 12
 
13
-$template->assign('PageTopic','Examine Ship');
14
-$template->assign('TargetPlayer',$targetPlayer);
13
+$template->assign('PageTopic', 'Examine Ship');
14
+$template->assign('TargetPlayer', $targetPlayer);
Please login to merge, or discard this patch.
engine/Default/trader_attack_processing.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -1,32 +1,32 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if($player->hasNewbieTurns())
3
+if ($player->hasNewbieTurns())
4 4
 	create_error('You are under newbie protection.');
5
-if($player->hasFederalProtection())
5
+if ($player->hasFederalProtection())
6 6
 	create_error('You are under federal protection.');
7
-if($player->isLandedOnPlanet())
7
+if ($player->isLandedOnPlanet())
8 8
 	create_error('You cannot attack whilst on a planet!');
9
-if($player->getTurns() < 3)
9
+if ($player->getTurns() < 3)
10 10
 	create_error('You have insufficient turns to perform that action.');
11
-if(!$player->canFight())
11
+if (!$player->canFight())
12 12
 	create_error('You are not allowed to fight!');
13 13
 
14
-$targetPlayer = SmrPlayer::getPlayer($var['target'],$player->getGameID());
14
+$targetPlayer = SmrPlayer::getPlayer($var['target'], $player->getGameID());
15 15
 
16
-	if($player->traderNAPAlliance($targetPlayer))
16
+	if ($player->traderNAPAlliance($targetPlayer))
17 17
 		create_error('Your alliance does not allow you to attack this trader.');
18
-	else if($targetPlayer->isDead())
18
+	else if ($targetPlayer->isDead())
19 19
 		create_error('Target is already dead.');
20
-	else if($targetPlayer->getSectorID() != $player->getSectorID())
20
+	else if ($targetPlayer->getSectorID() != $player->getSectorID())
21 21
 		create_error('Target is no longer in this sector.');
22
-	else if($targetPlayer->hasNewbieTurns())
22
+	else if ($targetPlayer->hasNewbieTurns())
23 23
 		create_error('Target is under newbie protection.');
24
-	else if($targetPlayer->isLandedOnPlanet())
24
+	else if ($targetPlayer->isLandedOnPlanet())
25 25
 		create_error('Target is protected by planetary shields.');
26
-	else if($targetPlayer->hasFederalProtection())
26
+	else if ($targetPlayer->hasFederalProtection())
27 27
 		create_error('Target is under federal protection.');
28 28
 
29
-$fightingPlayers = $sector->getFightingTraders($player,$targetPlayer);
29
+$fightingPlayers = $sector->getFightingTraders($player, $targetPlayer);
30 30
 
31 31
 //decloak all fighters
32 32
 foreach ($fightingPlayers as $teamPlayers) {
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 
42 42
 function teamAttack(&$results, $fightingPlayers, $attack, $defend) {
43 43
 	foreach ($fightingPlayers[$attack] as $accountID => $teamPlayer) {
44
-		$playerResults =& $teamPlayer->shootPlayers($fightingPlayers[$defend]);
45
-		$results[$attack]['Traders'][$teamPlayer->getAccountID()] =& $playerResults;
44
+		$playerResults = & $teamPlayer->shootPlayers($fightingPlayers[$defend]);
45
+		$results[$attack]['Traders'][$teamPlayer->getAccountID()] = & $playerResults;
46 46
 		$results[$attack]['TotalDamage'] += $playerResults['TotalDamage'];
47 47
 
48 48
 		// Award assists (if there are multiple attackers)
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 
68 68
 $ship->removeUnderAttack(); //Don't show attacker the under attack message.
69 69
 
70
-$account->log(LOG_TYPE_TRADER_COMBAT, 'Player attacks player, their team does ' . $results['Attackers']['TotalDamage'].' and the other team does '.$results['Defenders']['TotalDamage'], $sector->getSectorID());
70
+$account->log(LOG_TYPE_TRADER_COMBAT, 'Player attacks player, their team does ' . $results['Attackers']['TotalDamage'] . ' and the other team does ' . $results['Defenders']['TotalDamage'], $sector->getSectorID());
71 71
 
72 72
 $serializedResults = serialize($results);
73 73
 $db->query('INSERT INTO combat_logs VALUES(\'\',' . $db->escapeNumber($player->getGameID()) . ',\'PLAYER\',' . $db->escapeNumber($sector->getSectorID()) . ',' . $db->escapeNumber(TIME) . ',' . $db->escapeNumber($player->getAccountID()) . ',' . $db->escapeNumber($player->getAllianceID()) . ',' . $db->escapeNumber($var['target']) . ',' . $db->escapeNumber($targetPlayer->getAllianceID()) . ',' . $db->escapeBinary(gzcompress($serializedResults)) . ')');
@@ -76,13 +76,13 @@  discard block
 block discarded – undo
76 76
 $container = create_container('skeleton.php', 'trader_attack.php');
77 77
 
78 78
 // If their target is dead there is no continue attack button
79
-if(!$targetPlayer->isDead())
79
+if (!$targetPlayer->isDead())
80 80
 	$container['target'] = $var['target'];
81 81
 else
82 82
 	$container['target'] = 0;
83 83
 
84 84
 // If they died on the shot they get to see the results
85
-if($player->isDead()) {
85
+if ($player->isDead()) {
86 86
 	$container['override_death'] = TRUE;
87 87
 	$container['target'] = 0;
88 88
 }
Please login to merge, or discard this patch.
engine/Default/course_plot.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -1,28 +1,28 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-$template->assign('PageTopic','Plot A Course');
3
+$template->assign('PageTopic', 'Plot A Course');
4 4
 
5 5
 Menu::navigation($template, $player);
6 6
 
7
-$container=array();
7
+$container = array();
8 8
 $container['url'] = 'course_plot_processing.php';
9 9
 $container['body'] = '';
10 10
 
11
-$template->assign('PlotCourseFormLink',SmrSession::getNewHREF($container));
11
+$template->assign('PlotCourseFormLink', SmrSession::getNewHREF($container));
12 12
 $container['url'] = 'course_plot_nearest_processing.php';
13
-$template->assign('PlotNearestFormLink',SmrSession::getNewHREF($container));
13
+$template->assign('PlotNearestFormLink', SmrSession::getNewHREF($container));
14 14
 
15 15
 if ($ship->hasJump()) {
16
-	$container=create_container('sector_jump_processing.php','');
16
+	$container = create_container('sector_jump_processing.php', '');
17 17
 	$container['target_page'] = 'current_sector.php';
18
-	$template->assign('JumpDriveFormLink',SmrSession::getNewHREF($container));
18
+	$template->assign('JumpDriveFormLink', SmrSession::getNewHREF($container));
19 19
 }
20
-if(isset($_REQUEST['xtype']))
21
-	SmrSession::updateVar('XType',$_REQUEST['xtype']);
22
-else if(!isset($var['XType']))
23
-	SmrSession::updateVar('XType','Technology');
24
-$template->assign('XType',$var['XType']);
25
-$template->assign('AllXTypes',array('Technology','Ships','Weapons','Locations','Sell Goods','Buy Goods','Galaxies'));
20
+if (isset($_REQUEST['xtype']))
21
+	SmrSession::updateVar('XType', $_REQUEST['xtype']);
22
+else if (!isset($var['XType']))
23
+	SmrSession::updateVar('XType', 'Technology');
24
+$template->assign('XType', $var['XType']);
25
+$template->assign('AllXTypes', array('Technology', 'Ships', 'Weapons', 'Locations', 'Sell Goods', 'Buy Goods', 'Galaxies'));
26 26
 
27 27
 
28 28
 // get saved destinations
Please login to merge, or discard this patch.