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/combat_log_list_processing.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
 
15 15
 // Do we need to save any logs (or delete any saved logs)?
16 16
 if ($submitAction == 'Save' || $submitAction == 'Delete') {
17
-	if($submitAction == 'Save') {
17
+	if ($submitAction == 'Save') {
18 18
 		//save the logs we checked
19 19
 		// Query means people can only save logs that they are allowd to view.
20 20
 		$db->query('INSERT IGNORE INTO player_saved_combat_logs (account_id, game_id, log_id)
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 						)
33 33
 					LIMIT ' . count($logIDs));
34 34
 	}
35
-	else if($submitAction == 'Delete') {
35
+	else if ($submitAction == 'Delete') {
36 36
 		$db->query('DELETE FROM player_saved_combat_logs
37 37
 					WHERE log_id IN (' . $db->escapeArray($logIDs) . ')
38 38
 						AND account_id = ' . $db->escapeNumber($player->getAccountID()) . '
Please login to merge, or discard this patch.
engine/Default/feature_request_processing.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
 if (empty($_REQUEST['feature'])) {
4 4
 	create_error('We need at least a feature desciption!');
5 5
 }
6
-if(strlen($_REQUEST['feature']) > 500) {
6
+if (strlen($_REQUEST['feature']) > 500) {
7 7
 	create_error('Feature request longer than 500 characters, please be more concise!');
8 8
 }
9 9
 
@@ -11,9 +11,9 @@  discard block
 block discarded – undo
11 11
 $db->query('INSERT INTO feature_request (feature_request_id) VALUES (NULL)');
12 12
 $featureRequestID = $db->getInsertID();
13 13
 $db->query('INSERT INTO feature_request_comments (feature_request_id, poster_id, posting_time, anonymous, text) ' .
14
-								'VALUES(' . $db->escapeNumber($featureRequestID) . ', ' . $db->escapeNumber($account->getAccountID()) . ',' . $db->escapeNumber(TIME) . ',' . $db->escapeBoolean(isset($_REQUEST['anon'])) . ',' . $db->escapeString(word_filter($_REQUEST['feature'])).')');
14
+								'VALUES(' . $db->escapeNumber($featureRequestID) . ', ' . $db->escapeNumber($account->getAccountID()) . ',' . $db->escapeNumber(TIME) . ',' . $db->escapeBoolean(isset($_REQUEST['anon'])) . ',' . $db->escapeString(word_filter($_REQUEST['feature'])) . ')');
15 15
 
16 16
 // vote for this feature
17
-$db->query('INSERT INTO account_votes_for_feature VALUES('.$db->escapeNumber($account->getAccountID()).', '.$db->escapeNumber($featureRequestID).',\'YES\')');
17
+$db->query('INSERT INTO account_votes_for_feature VALUES(' . $db->escapeNumber($account->getAccountID()) . ', ' . $db->escapeNumber($featureRequestID) . ',\'YES\')');
18 18
 
19 19
 forward(create_container('skeleton.php', 'feature_request.php'));
Please login to merge, or discard this patch.
engine/Default/bar_lotto_buy_processing.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -17,16 +17,16 @@
 block discarded – undo
17 17
 
18 18
 $db->query('INSERT INTO player_has_ticket (game_id, account_id, time) VALUES (' . $db->escapeNumber($player->getGameID()) . ', ' . $db->escapeNumber($player->getAccountID()) . ', ' . $db->escapeNumber($time) . ')');
19 19
 $player->decreaseCredits(1000000);
20
-$player->increaseHOF(1000000,array('Bar','Lotto', 'Money', 'Spent'), HOF_PUBLIC);
21
-$player->increaseHOF(1,array('Bar','Lotto', 'Tickets Bought'), HOF_PUBLIC);
20
+$player->increaseHOF(1000000, array('Bar', 'Lotto', 'Money', 'Spent'), HOF_PUBLIC);
21
+$player->increaseHOF(1, array('Bar', 'Lotto', 'Tickets Bought'), HOF_PUBLIC);
22 22
 $db->query('SELECT count(*) as num FROM player_has_ticket WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . '
23 23
 	AND account_id = ' . $db->escapeNumber($player->getAccountID()) . ' AND time > 0 GROUP BY account_id');
24 24
 $db->nextRecord();
25 25
 $num = $db->getInt('num');
26
-$message=('<div align=center>Thanks for your purchase and good luck!  You currently');
27
-$message.=(' own '.$num.' '.pluralise('ticket', $num).'!</div><br />');
26
+$message = ('<div align=center>Thanks for your purchase and good luck!  You currently');
27
+$message .= (' own ' . $num . ' ' . pluralise('ticket', $num) . '!</div><br />');
28 28
 
29
-$container=create_container('skeleton.php','bar_main.php');
29
+$container = create_container('skeleton.php', 'bar_main.php');
30 30
 transfer('LocationID');
31 31
 $container['message'] = $message;
32 32
 forward($container);
Please login to merge, or discard this patch.
engine/Default/bank_anon_detail_processing.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 	$player->update();
32 32
 
33 33
 	// log action
34
-	$account->log(LOG_TYPE_BANK, 'Deposits '.$amount.' credits in anonymous account #'.$account_num, $player->getSectorID());
34
+	$account->log(LOG_TYPE_BANK, 'Deposits ' . $amount . ' credits in anonymous account #' . $account_num, $player->getSectorID());
35 35
 }
36 36
 else {
37 37
 	$db->query('SELECT * FROM anon_bank WHERE anon_id = ' . $db->escapeNumber($account_num) . ' AND game_id = ' . $db->escapeNumber($player->getGameID()));
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	$player->update();
53 53
 
54 54
 	// log action
55
-	$account->log(LOG_TYPE_BANK, 'Takes '.$amount.' credits from anonymous account #'.$account_num, $player->getSectorID());
55
+	$account->log(LOG_TYPE_BANK, 'Takes ' . $amount . ' credits from anonymous account #' . $account_num, $player->getSectorID());
56 56
 }
57 57
 
58 58
 $container = create_container('skeleton.php', 'bank_anon_detail.php');
Please login to merge, or discard this patch.
engine/Default/death.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,2 +1,2 @@
 block discarded – undo
1 1
 <?php
2
-$template->assign('PageTopic','Death');
2
+$template->assign('PageTopic', 'Death');
Please login to merge, or discard this patch.
engine/Default/history_games_news.php 1 patch
Spacing   +2 added lines, -2 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', 'Game News : '.$var['game_name']);
3
+$template->assign('PageTopic', 'Game News : ' . $var['game_name']);
4 4
 Menu::history_games(3);
5 5
 
6 6
 if (isset($_REQUEST['min'])) $min = $_REQUEST['min'];
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 $template->assign('ShowHREF', SmrSession::getNewHREF($var));
14 14
 
15 15
 $db = new $var['HistoryDatabase']();
16
-$db->query('SELECT * FROM news WHERE game_id = '.$db->escapeNumber($var['view_game_id']).' AND news_id >= '.$db->escapeNumber($min).' AND news_id <= '.$db->escapeNumber($max));
16
+$db->query('SELECT * FROM news WHERE game_id = ' . $db->escapeNumber($var['view_game_id']) . ' AND news_id >= ' . $db->escapeNumber($min) . ' AND news_id <= ' . $db->escapeNumber($max));
17 17
 $rows = [];
18 18
 while ($db->nextRecord()) {
19 19
 	$rows[] = [
Please login to merge, or discard this patch.
engine/Default/history_games_detail.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,7 +1,7 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3 3
 $game_id = $var['view_game_id'];
4
-$template->assign('PageTopic', 'Extended Stats : '.$var['game_name']);
4
+$template->assign('PageTopic', 'Extended Stats : ' . $var['game_name']);
5 5
 Menu::history_games(1);
6 6
 
7 7
 $container = $var;
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	$db = new $var['HistoryDatabase']();
31 31
 	if ($from != 'alliance') {
32 32
 		$template->assign('Name', 'Sector ID');
33
-		$db->query('SELECT '.$sql.' as val, sector_id FROM '.$from.' WHERE game_id = '.$db->escapeNumber($game_id).' ORDER BY val DESC LIMIT 25');
33
+		$db->query('SELECT ' . $sql . ' as val, sector_id FROM ' . $from . ' WHERE game_id = ' . $db->escapeNumber($game_id) . ' ORDER BY val DESC LIMIT 25');
34 34
 		while ($db->nextRecord()) {
35 35
 			$rankings[] = [
36 36
 				'name' => $db->getInt('sector_id'),
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	}
41 41
 	else {
42 42
 		$template->assign('Name', 'Alliance');
43
-		$db->query('SELECT alliance_name, alliance_id, '.$sql.' as val FROM alliance WHERE game_id = '.$db->escapeNumber($game_id).' AND alliance_id > 0 GROUP BY alliance_id ORDER BY val DESC, alliance_id LIMIT 25');
43
+		$db->query('SELECT alliance_name, alliance_id, ' . $sql . ' as val FROM alliance WHERE game_id = ' . $db->escapeNumber($game_id) . ' AND alliance_id > 0 GROUP BY alliance_id ORDER BY val DESC, alliance_id LIMIT 25');
44 44
 		$container = $var;
45 45
 		$container['body'] = 'history_alliance_detail.php';
46 46
 		$container['selected_index'] = 1;
Please login to merge, or discard this patch.
engine/Default/album_edit.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-$template->assign('PageTopic','Edit Photo');
2
+$template->assign('PageTopic', 'Edit Photo');
3 3
 
4 4
 $db->query('SELECT * FROM album WHERE account_id = ' . $db->escapeNumber($account->getAccountID()));
5 5
 if ($db->nextRecord()) {
@@ -13,23 +13,23 @@  discard block
 block discarded – undo
13 13
 	$approved = $db->getField('approved');
14 14
 
15 15
 	if ($approved == 'TBC') {
16
-		$albumEntry['Status']=('<span style="color:orange;">Waiting approval</span>');
16
+		$albumEntry['Status'] = ('<span style="color:orange;">Waiting approval</span>');
17 17
 	}
18 18
 	elseif ($approved == 'NO') {
19
-		$albumEntry['Status']=('<span class="red">Approval denied</span>');
19
+		$albumEntry['Status'] = ('<span class="red">Approval denied</span>');
20 20
 	}
21 21
 	elseif ($db->getBoolean('disabled')) {
22
-		$albumEntry['Status']=('<span class="red">Disabled</span>');
22
+		$albumEntry['Status'] = ('<span class="red">Disabled</span>');
23 23
 	}
24 24
 	elseif ($approved == 'YES') {
25
-		$albumEntry['Status']=('<a href="album/?'.$account->getHofName().'" class="dgreen">Online</a>');
25
+		$albumEntry['Status'] = ('<a href="album/?' . $account->getHofName() . '" class="dgreen">Online</a>');
26 26
 	}
27 27
 		
28
-	if(is_readable(UPLOAD . $account->getAccountID())) {
29
-		$albumEntry['Image'] = '/upload/'.$account->getAccountID();
28
+	if (is_readable(UPLOAD . $account->getAccountID())) {
29
+		$albumEntry['Image'] = '/upload/' . $account->getAccountID();
30 30
 	}
31 31
 	
32
-	$template->assign('AlbumEntry',$albumEntry);
32
+	$template->assign('AlbumEntry', $albumEntry);
33 33
 }
34 34
 
35
-$template->assign('AlbumEditHref',SmrSession::getNewHREF(create_container('album_edit_processing.php', '')));
35
+$template->assign('AlbumEditHref', SmrSession::getNewHREF(create_container('album_edit_processing.php', '')));
Please login to merge, or discard this patch.
engine/Default/port_attack_warning.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,14 +1,14 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-if(!$sector->hasPort()) {
3
+if (!$sector->hasPort()) {
4 4
 	create_error('This sector does not have a port.');
5 5
 }
6 6
 
7
-if($sector->getPort()->isDestroyed()) {
7
+if ($sector->getPort()->isDestroyed()) {
8 8
 	forward(create_container('skeleton.php', 'port_attack.php'));
9 9
 }
10 10
 
11
-$template->assign('PageTopic','Port Raid');
11
+$template->assign('PageTopic', 'Port Raid');
12 12
 
13
-$template->assign('PortAttackHREF',SmrSession::getNewHREF(create_container('port_attack_processing.php')));
13
+$template->assign('PortAttackHREF', SmrSession::getNewHREF(create_container('port_attack_processing.php')));
14 14
 $template->assign('Port', $sector->getPort());
Please login to merge, or discard this patch.