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

Passed
Push — master ( a942b9...9fc47f )
by Dan
06:36 queued 01:08
created
engine/Default/planet_attack_processing.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -58,8 +58,8 @@  discard block
 block discarded – undo
58 58
 
59 59
 $totalShieldDamage = 0;
60 60
 foreach ($attackers as $attacker) {
61
-	$playerResults =& $attacker->shootPlanet($planet, false);
62
-	$results['Attackers']['Traders'][$attacker->getAccountID()] =& $playerResults;
61
+	$playerResults = & $attacker->shootPlanet($planet, false);
62
+	$results['Attackers']['Traders'][$attacker->getAccountID()] = & $playerResults;
63 63
 	$results['Attackers']['TotalDamage'] += $playerResults['TotalDamage'];
64 64
 	foreach ($playerResults['Weapons'] as $weapon) {
65 65
 		if (isset($weapon['ActualDamage'])) { // Only set if the weapon hits
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 $downgradeDamage = $results['Attackers']['TotalDamage'] - $totalShieldDamage;
73 73
 $results['Attackers']['Downgrades'] = $planet->checkForDowngrade($downgradeDamage);
74 74
 
75
-$results['Planet'] =& $planet->shootPlayers($attackers);
75
+$results['Planet'] = & $planet->shootPlayers($attackers);
76 76
 
77 77
 $account->log(LOG_TYPE_PLANET_BUSTING, 'Player attacks planet, the planet does ' . $results['Planet']['TotalDamage'] . ', their team does ' . $results['Attackers']['TotalDamage'] . ' and downgrades: ' . var_export($results['Attackers']['Downgrades'], true), $planet->getSectorID());
78 78
 
Please login to merge, or discard this patch.
engine/Default/port_attack_processing.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -53,8 +53,8 @@  discard block
 block discarded – undo
53 53
 
54 54
 $totalShieldDamage = 0;
55 55
 foreach ($attackers as $attacker) {
56
-	$playerResults =& $attacker->shootPort($port);
57
-	$results['Attackers']['Traders'][$attacker->getAccountID()] =& $playerResults;
56
+	$playerResults = & $attacker->shootPort($port);
57
+	$results['Attackers']['Traders'][$attacker->getAccountID()] = & $playerResults;
58 58
 	$results['Attackers']['TotalDamage'] += $playerResults['TotalDamage'];
59 59
 	foreach ($playerResults['Weapons'] as $weapon) {
60 60
 		if (isset($weapon['ActualDamage'])) { // Only set if the weapon hits
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 $downgradeDamage = $results['Attackers']['TotalDamage'] - $totalShieldDamage;
68 68
 $results['Attackers']['Downgrades'] = $port->checkForDowngrade($downgradeDamage);
69 69
 
70
-$results['Port'] =& $port->shootPlayers($attackers);
70
+$results['Port'] = & $port->shootPlayers($attackers);
71 71
 
72 72
 $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());
73 73
 
Please login to merge, or discard this patch.
admin/Default/1.6/game_create_processing.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
 	create_error('Join Date is not valid!');
20 20
 }
21 21
 $start = empty(Request::get('game_start')) ? $join :
22
-         DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
22
+		 DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
23 23
 if ($start === false) {
24 24
 	create_error('Start Date is not valid!');
25 25
 }
Please login to merge, or discard this patch.
Spacing   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,14 +1,14 @@  discard block
 block discarded – undo
1 1
 <?php declare(strict_types=1);
2 2
 
3 3
 //first create the game
4
-$db->query('SELECT game_id FROM game WHERE game_name='.$db->escapeString(Request::get('game_name')).' LIMIT 1');
4
+$db->query('SELECT game_id FROM game WHERE game_name=' . $db->escapeString(Request::get('game_name')) . ' LIMIT 1');
5 5
 if ($db->nextRecord()) {
6 6
 	create_error('That game name is already taken.');
7 7
 }
8 8
 
9 9
 $db->query('SELECT game_id FROM game ORDER BY game_id DESC LIMIT 1');
10 10
 if ($db->nextRecord()) {
11
-	$newID = $db->getInt('game_id')+1;
11
+	$newID = $db->getInt('game_id') + 1;
12 12
 } else {
13 13
 	$newID = 1;
14 14
 }
@@ -18,8 +18,7 @@  discard block
 block discarded – undo
18 18
 if ($join === false) {
19 19
 	create_error('Join Date is not valid!');
20 20
 }
21
-$start = empty(Request::get('game_start')) ? $join :
22
-         DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
21
+$start = empty(Request::get('game_start')) ? $join : DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
23 22
 if ($start === false) {
24 23
 	create_error('Start Date is not valid!');
25 24
 }
Please login to merge, or discard this patch.
admin/Default/manage_post_editors.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
 $db->query('SELECT game_id, game_name FROM game WHERE join_time < ' . $db->escapeNumber(SmrSession::getTime()) . ' AND end_time > ' . $db->escapeNumber(SmrSession::getTime()) . ' ORDER BY start_time DESC');
11 11
 while ($db->nextRecord()) {
12 12
 	$activeGames[] = array('game_name' => $db->getField('game_name'),
13
-	                       'game_id' => $db->getInt('game_id'));
13
+						   'game_id' => $db->getInt('game_id'));
14 14
 }
15 15
 $template->assign('ActiveGames', $activeGames);
16 16
 
Please login to merge, or discard this patch.
admin/Default/manage_draft_leaders.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
 $db->query('SELECT game_id, game_name FROM game WHERE game_type=' . $db->escapeNumber(SmrGame::GAME_TYPE_DRAFT) . ' AND join_time < ' . $db->escapeNumber(SmrSession::getTime()) . ' AND end_time > ' . $db->escapeNumber(SmrSession::getTime()) . ' ORDER BY start_time DESC');
11 11
 while ($db->nextRecord()) {
12 12
 	$activeGames[] = array('game_name' => $db->getField('game_name'),
13
-	                       'game_id' => $db->getInt('game_id'));
13
+						   'game_id' => $db->getInt('game_id'));
14 14
 }
15 15
 $template->assign('ActiveGames', $activeGames);
16 16
 
Please login to merge, or discard this patch.
tools/npc/chess.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -10,22 +10,22 @@  discard block
 block discarded – undo
10 10
 
11 11
 	debug('Script started');
12 12
 	define('SCRIPT_ID', $db->getInsertID());
13
-	$db->query('UPDATE npc_logs SET script_id='.SCRIPT_ID.' WHERE log_id='.SCRIPT_ID);
13
+	$db->query('UPDATE npc_logs SET script_id=' . SCRIPT_ID . ' WHERE log_id=' . SCRIPT_ID);
14 14
 
15 15
 	define('NPC_SCRIPT', true);
16 16
 
17 17
 	$descriptorSpec = array(
18
-		0 => array("pipe", "r"),  // stdin is a pipe that the child will read from
18
+		0 => array("pipe", "r"), // stdin is a pipe that the child will read from
19 19
 		1 => array("pipe", "w")  // stdout is a pipe that the child will write to
20 20
 	);
21 21
 	$engine = proc_open(UCI_CHESS_ENGINE, $descriptorSpec, $pipes);
22
-	$toEngine =& $pipes[0];
23
-	$fromEngine =& $pipes[1];
22
+	$toEngine = & $pipes[0];
23
+	$fromEngine = & $pipes[1];
24 24
 
25 25
 	function readFromEngine($block = true) {
26 26
 		global $fromEngine;
27 27
 		stream_set_blocking($fromEngine, $block == true ? 1 : 0);
28
-		while(($s = fgets($fromEngine)) !== false) {
28
+		while (($s = fgets($fromEngine)) !== false) {
29 29
 			debug('<-- ' . trim($s));
30 30
 			stream_set_blocking($fromEngine, 0);
31 31
 		}
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 		global $toEngine;
35 35
 		debug('--> ' . $s);
36 36
 		fputs($toEngine, $s . EOL);
37
-		if($read === true) {
37
+		if ($read === true) {
38 38
 			readFromEngine($block);
39 39
 		}
40 40
 	}
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	writeToEngine('isready');
46 46
 	writeToEngine('ucinewgame', false);
47 47
 
48
-	while(true) {
48
+	while (true) {
49 49
 		// The next "page request" must occur at an updated time.
50 50
 		SmrSession::updateTime();
51 51
 
@@ -54,9 +54,9 @@  discard block
 block discarded – undo
54 54
 			writeToEngine('position fen ' . $chessGame->getFENString(), false);
55 55
 			writeToEngine('go ' . ($chessGame->getCurrentTurnColour() == ChessGame::PLAYER_WHITE ? 'w' : 'b') . 'time ' . UCI_TIME_PER_MOVE_MS, true, false);
56 56
 			stream_set_blocking($fromEngine, 1);
57
-			while(stripos($move = trim(fgets($fromEngine)), 'bestmove') !== 0) {
57
+			while (stripos($move = trim(fgets($fromEngine)), 'bestmove') !== 0) {
58 58
 				debug('<-- ' . $move);
59
-				if(stripos($move, 'Seg') === 0) {
59
+				if (stripos($move, 'Seg') === 0) {
60 60
 					// Segfault
61 61
 					debug('UCI engine segfaulted?');
62 62
 					exit;
@@ -76,12 +76,12 @@  discard block
 block discarded – undo
76 76
 	fclose($toEngine);
77 77
 	fclose($fromEngine);
78 78
 	proc_close($engine);
79
-} catch(Throwable $e) {
79
+} catch (Throwable $e) {
80 80
 	logException($e);
81 81
 	exit;
82 82
 }
83 83
 function debug($message, $debugObject = null) {
84
-	global $account,$var,$db;
85
-	echo date('Y-m-d H:i:s - ').$message.($debugObject!==null?EOL.var_export($debugObject,true):'').EOL;
86
-	$db->query('INSERT INTO npc_logs (script_id, npc_id, time, message, debug_info, var) VALUES ('.(defined('SCRIPT_ID')?SCRIPT_ID:0).', '.(is_object($account)?$account->getAccountID():0).',NOW(),'.$db->escapeString($message).','.$db->escapeString(var_export($debugObject,true)).','.$db->escapeString(var_export($var,true)).')');
84
+	global $account, $var, $db;
85
+	echo date('Y-m-d H:i:s - ') . $message . ($debugObject !== null ?EOL.var_export($debugObject, true) : '') . EOL;
86
+	$db->query('INSERT INTO npc_logs (script_id, npc_id, time, message, debug_info, var) VALUES (' . (defined('SCRIPT_ID') ?SCRIPT_ID:0) . ', ' . (is_object($account) ? $account->getAccountID() : 0) . ',NOW(),' . $db->escapeString($message) . ',' . $db->escapeString(var_export($debugObject, true)) . ',' . $db->escapeString(var_export($var, true)) . ')');
87 87
 }
Please login to merge, or discard this patch.
lib/Smr/Container/DiContainer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,10 +36,10 @@
 block discarded – undo
36 36
 			 * The factories themselves are able to use dependency injection as well, so we can provide the MySqlProperties
37 37
 			 * typehint to make sure the container constructs an instance and provides it to the factory.
38 38
 			 */
39
-			mysqli::class => function (MySqlProperties $mysqlProperties): mysqli {
39
+			mysqli::class => function(MySqlProperties $mysqlProperties): mysqli {
40 40
 				return MySqlDatabase::mysqliFactory($mysqlProperties);
41 41
 			},
42
-			Dotenv::class => function (): Dotenv {
42
+			Dotenv::class => function(): Dotenv {
43 43
 				return Dotenv::createArrayBacked(ROOT);
44 44
 			},
45 45
 			// Explicitly name all classes that are autowired, so we can take advantage of
Please login to merge, or discard this patch.
lib/Default/MySqlDatabase.class.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
 	}
39 39
 
40 40
 	public static function mysqliFactory(MySqlProperties $mysqlProperties): mysqli {
41
-		if (!mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT)) {
41
+		if (!mysqli_report(MYSQLI_REPORT_ERROR|MYSQLI_REPORT_STRICT)) {
42 42
 			throw new RuntimeException('Failed to enable mysqli error reporting');
43 43
 		}
44 44
 		$mysql = new mysqli(
Please login to merge, or discard this patch.
lib/Default/AbstractSmrAccount.class.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -361,26 +361,26 @@
 block discarded – undo
361 361
 	}
362 362
 
363 363
 	public function isNPC() : bool {
364
-		if(!isset($this->npc)) {
365
-			$this->db->query('SELECT login FROM npc_logins WHERE login = '.$this->db->escapeString($this->getLogin()).' LIMIT 1;');
364
+		if (!isset($this->npc)) {
365
+			$this->db->query('SELECT login FROM npc_logins WHERE login = ' . $this->db->escapeString($this->getLogin()) . ' LIMIT 1;');
366 366
 			$this->npc = $this->db->nextRecord();
367 367
 		}
368 368
 		return $this->npc;
369 369
 	}
370 370
 
371 371
 	protected function getHOFData() : void {
372
-		if(!isset($this->HOF)) {
372
+		if (!isset($this->HOF)) {
373 373
 			//Get Player HOF
374 374
 			$this->db->query('SELECT type,sum(amount) as amount FROM player_hof WHERE ' . $this->SQL . ' AND game_id IN (SELECT game_id FROM game WHERE ignore_stats = \'FALSE\') GROUP BY type');
375 375
 			$this->HOF = array();
376
-			while($this->db->nextRecord()) {
377
-				$hof =& $this->HOF;
378
-				$typeList = explode(':',$this->db->getField('type'));
379
-				foreach($typeList as $type) {
380
-					if(!isset($hof[$type])) {
376
+			while ($this->db->nextRecord()) {
377
+				$hof = & $this->HOF;
378
+				$typeList = explode(':', $this->db->getField('type'));
379
+				foreach ($typeList as $type) {
380
+					if (!isset($hof[$type])) {
381 381
 						$hof[$type] = array();
382 382
 					}
383
-					$hof =& $hof[$type];
383
+					$hof = & $hof[$type];
384 384
 				}
385 385
 				$hof = $this->db->getFloat('amount');
386 386
 			}
Please login to merge, or discard this patch.