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
Push — master ( 6ccf62...b6decb )
by Dan
19s queued 16s
created
src/lib/Default/Routes/RouteGenerator.class.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 				foreach (\Globals::getGoods() as $goodId => $value) {
91 91
 					if ($goods[$goodId] === true) {
92 92
 						if ($sectors[$currentSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_SELLS &&
93
-						    $sectors[$targetSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_BUYS) {
93
+							$sectors[$targetSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_BUYS) {
94 94
 							$rl[] = new OneWayRoute($currentSectorId, $targetSectorId, $raceID, $sectors[$targetSectorId]->getPort()->getRaceID(), $sectors[$currentSectorId]->getPort()->getGoodDistance($goodId), $sectors[$targetSectorId]->getPort()->getGoodDistance($goodId), $distance, $goodId);
95 95
 						}
96 96
 					}
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 				foreach (\Globals::getGoods() as $goodId => $value) {
119 119
 					if ($goods[$goodId] === true) {
120 120
 						if ($sectors[$currentSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_SELLS &&
121
-						    $sectors[$targetSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_BUYS) {
121
+							$sectors[$targetSectorId]->getPort()->getGoodTransaction($goodId) === TRADER_BUYS) {
122 122
 							$owr = new OneWayRoute($currentSectorId, $targetSectorId, $sectors[$currentSectorId]->getPort()->getRaceID(), $sectors[$targetSectorId]->getPort()->getRaceID(), $sectors[$currentSectorId]->getPort()->getGoodDistance($goodId), $sectors[$targetSectorId]->getPort()->getGoodDistance($goodId), $distance, $goodId);
123 123
 							$fakeReturn = new OneWayRoute($targetSectorId, $currentSectorId, $sectors[$targetSectorId]->getPort()->getRaceID(), $sectors[$currentSectorId]->getPort()->getRaceID(), 0, 0, $distance, GOODS_NOTHING);
124 124
 							$mpr = new MultiplePortRoute($owr, $fakeReturn);
Please login to merge, or discard this patch.
src/admin/Default/notify_view.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
 	/**
22 22
 	 * @var $messagePlayer SmrPlayer | string
23 23
 	 */
24
-	$getName = function ($messagePlayer) use ($container, $account) : string {
24
+	$getName = function($messagePlayer) use ($container, $account) : string {
25 25
 		$name = $messagePlayer;
26 26
 		if ($messagePlayer instanceof SmrPlayer) {
27 27
 			$name = $messagePlayer->getAccount()->getLogin();
Please login to merge, or discard this patch.
src/lib/Default/AbstractSmrPort.class.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1219,7 +1219,7 @@  discard block
 block discarded – undo
1219 1219
 			do {
1220 1220
 				$targetPlayer = array_rand_value($targetPlayers);
1221 1221
 			} while ($results['TotalShotsPerTargetPlayer'][$targetPlayer->getAccountID()] > min($results['TotalShotsPerTargetPlayer']));
1222
-			$results['Weapons'][$orderID] =& $weapon->shootPlayerAsPort($this, $targetPlayer);
1222
+			$results['Weapons'][$orderID] = & $weapon->shootPlayerAsPort($this, $targetPlayer);
1223 1223
 			$results['TotalShotsPerTargetPlayer'][$targetPlayer->getAccountID()]++;
1224 1224
 			if ($results['Weapons'][$orderID]['Hit']) {
1225 1225
 				$results['TotalDamage'] += $results['Weapons'][$orderID]['ActualDamage']['TotalDamage'];
@@ -1228,7 +1228,7 @@  discard block
 block discarded – undo
1228 1228
 		}
1229 1229
 		if ($this->hasCDs()) {
1230 1230
 			$thisCDs = new SmrCombatDrones($this->getGameID(), $this->getCDs(), true);
1231
-			$results['Drones'] =& $thisCDs->shootPlayerAsPort($this, array_rand_value($targetPlayers));
1231
+			$results['Drones'] = & $thisCDs->shootPlayerAsPort($this, array_rand_value($targetPlayers));
1232 1232
 			$results['TotalDamage'] += $results['Drones']['ActualDamage']['TotalDamage'];
1233 1233
 			$results['TotalDamagePerTargetPlayer'][$results['Drones']['TargetPlayer']->getAccountID()] += $results['Drones']['ActualDamage']['TotalDamage'];
1234 1234
 		}
Please login to merge, or discard this patch.
src/lib/Default/SmrForce.class.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -460,20 +460,20 @@
 block discarded – undo
460 460
 
461 461
 		if ($this->hasMines()) {
462 462
 			$thisMines = new SmrMines($this->getGameID(), $this->getMines());
463
-			$results['Results']['Mines'] =& $thisMines->shootPlayerAsForce($this, array_rand_value($targetPlayers), $minesAreAttacker);
463
+			$results['Results']['Mines'] = & $thisMines->shootPlayerAsForce($this, array_rand_value($targetPlayers), $minesAreAttacker);
464 464
 			$results['TotalDamage'] += $results['Results']['Mines']['ActualDamage']['TotalDamage'];
465 465
 		}
466 466
 
467 467
 		if ($this->hasCDs()) {
468 468
 			$thisCDs = new SmrCombatDrones($this->getGameID(), $this->getCDs());
469
-			$results['Results']['Drones'] =& $thisCDs->shootPlayerAsForce($this, array_rand_value($targetPlayers));
469
+			$results['Results']['Drones'] = & $thisCDs->shootPlayerAsForce($this, array_rand_value($targetPlayers));
470 470
 			$results['TotalDamage'] += $results['Results']['Drones']['ActualDamage']['TotalDamage'];
471 471
 		}
472 472
 
473 473
 		if (!$minesAreAttacker) {
474 474
 			if ($this->hasSDs()) {
475 475
 				$thisSDs = new SmrScoutDrones($this->getGameID(), $this->getSDs());
476
-				$results['Results']['Scouts'] =& $thisSDs->shootPlayerAsForce($this, array_rand_value($targetPlayers));
476
+				$results['Results']['Scouts'] = & $thisSDs->shootPlayerAsForce($this, array_rand_value($targetPlayers));
477 477
 				$results['TotalDamage'] += $results['Results']['Scouts']['ActualDamage']['TotalDamage'];
478 478
 			}
479 479
 		}
Please login to merge, or discard this patch.
src/lib/Smr/SocialLogin/Twitter.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
 		}
36 36
 		$helper = self::getTwitterObj($_SESSION['TwitterToken']);
37 37
 		$accessToken = $helper->oauth('oauth/access_token',
38
-		                              ['oauth_verifier' => \Request::get('oauth_verifier')]);
38
+									  ['oauth_verifier' => \Request::get('oauth_verifier')]);
39 39
 		$auth = self::getTwitterObj($accessToken);
40 40
 		$userInfo = $auth->get('account/verify_credentials', ['include_email' => 'true']);
41 41
 		if ($auth->getLastHttpCode() == 200) {
Please login to merge, or discard this patch.
src/lib/Default/SmrGalaxy.class.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -340,7 +340,7 @@
 block discarded – undo
340 340
 	/**
341 341
 	 * Check if the galaxy contains a specific sector.
342 342
 	 */
343
-	public function contains(int | SmrSector $sectorID) : bool {
343
+	public function contains(int|SmrSector $sectorID) : bool {
344 344
 		if ($sectorID instanceof SmrSector) {
345 345
 			return $sectorID->getGalaxyID() == $this->getGalaxyID();
346 346
 		}
Please login to merge, or discard this patch.
src/admin/Default/ip_view_results.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@
 block discarded – undo
96 96
 	}
97 97
 	$accountID = (int)$variable;
98 98
 	$template->assign('BanAccountID', $accountID);
99
-	$summary = 'Account ' . $accountID. ' has had the following IPs at the following times.';
99
+	$summary = 'Account ' . $accountID . ' has had the following IPs at the following times.';
100 100
 	$template->assign('Summary', $summary);
101 101
 	$db2->query('SELECT * FROM account_exceptions WHERE account_id = ' . $db->escapeNumber($variable));
102 102
 	if ($db2->nextRecord()) {
Please login to merge, or discard this patch.
src/lib/Default/AbstractSmrCombatWeapon.class.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -67,68 +67,68 @@
 block discarded – undo
67 67
 	abstract public function &getModifiedPlanetDamageAgainstPlayer(SmrPlanet $planet, AbstractSmrPlayer $targetPlayer);
68 68
 
69 69
 	protected function &doPlayerDamageToForce(array &$return, AbstractSmrPlayer $weaponPlayer, SmrForce $forces) {
70
-		$return['WeaponDamage'] =& $this->getModifiedDamageAgainstForces($weaponPlayer,$forces);
71
-		$return['ActualDamage'] =& $forces->doWeaponDamage($return['WeaponDamage']);
70
+		$return['WeaponDamage'] = & $this->getModifiedDamageAgainstForces($weaponPlayer, $forces);
71
+		$return['ActualDamage'] = & $forces->doWeaponDamage($return['WeaponDamage']);
72 72
 		if ($return['ActualDamage']['KillingShot']) {
73
-			$return['KillResults'] =& $forces->killForcesByPlayer($weaponPlayer);
73
+			$return['KillResults'] = & $forces->killForcesByPlayer($weaponPlayer);
74 74
 		}
75 75
 		return $return;
76 76
 	}
77 77
 
78 78
 	protected function &doPlayerDamageToPlayer(array &$return, AbstractSmrPlayer $weaponPlayer, AbstractSmrPlayer $targetPlayer) {
79
-		$return['WeaponDamage'] =& $this->getModifiedDamageAgainstPlayer($weaponPlayer,$targetPlayer);
80
-		$return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
79
+		$return['WeaponDamage'] = & $this->getModifiedDamageAgainstPlayer($weaponPlayer, $targetPlayer);
80
+		$return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
81 81
 
82 82
 		if ($return['ActualDamage']['KillingShot']) {
83
-			$return['KillResults'] =& $targetPlayer->killPlayerByPlayer($weaponPlayer);
83
+			$return['KillResults'] = & $targetPlayer->killPlayerByPlayer($weaponPlayer);
84 84
 		}
85 85
 		return $return;
86 86
 	}
87 87
 
88 88
 	protected function &doPlayerDamageToPort(array &$return, AbstractSmrPlayer $weaponPlayer, SmrPort $port) {
89
-		$return['WeaponDamage'] =& $this->getModifiedDamageAgainstPort($weaponPlayer,$port);
90
-		$return['ActualDamage'] =& $port->doWeaponDamage($return['WeaponDamage']);
89
+		$return['WeaponDamage'] = & $this->getModifiedDamageAgainstPort($weaponPlayer, $port);
90
+		$return['ActualDamage'] = & $port->doWeaponDamage($return['WeaponDamage']);
91 91
 		if ($return['ActualDamage']['KillingShot']) {
92
-			$return['KillResults'] =& $port->killPortByPlayer($weaponPlayer);
92
+			$return['KillResults'] = & $port->killPortByPlayer($weaponPlayer);
93 93
 		}
94 94
 		return $return;
95 95
 	}
96 96
 
97 97
 	protected function &doPlayerDamageToPlanet(array &$return, AbstractSmrPlayer $weaponPlayer, SmrPlanet $planet, $delayed) {
98
-		$return['WeaponDamage'] =& $this->getModifiedDamageAgainstPlanet($weaponPlayer,$planet);
99
-		$return['ActualDamage'] =& $planet->doWeaponDamage($return['WeaponDamage'],$delayed);
98
+		$return['WeaponDamage'] = & $this->getModifiedDamageAgainstPlanet($weaponPlayer, $planet);
99
+		$return['ActualDamage'] = & $planet->doWeaponDamage($return['WeaponDamage'], $delayed);
100 100
 		if ($return['ActualDamage']['KillingShot']) {
101
-			$return['KillResults'] =& $planet->killPlanetByPlayer($weaponPlayer);
101
+			$return['KillResults'] = & $planet->killPlanetByPlayer($weaponPlayer);
102 102
 		}
103 103
 		return $return;
104 104
 	}
105 105
 
106 106
 	protected function &doPortDamageToPlayer(array &$return, SmrPort $port, AbstractSmrPlayer $targetPlayer) {
107
-		$return['WeaponDamage'] =& $this->getModifiedPortDamageAgainstPlayer($port,$targetPlayer);
108
-		$return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
107
+		$return['WeaponDamage'] = & $this->getModifiedPortDamageAgainstPlayer($port, $targetPlayer);
108
+		$return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
109 109
 
110 110
 		if ($return['ActualDamage']['KillingShot']) {
111
-			$return['KillResults'] =& $targetPlayer->killPlayerByPort($port);
111
+			$return['KillResults'] = & $targetPlayer->killPlayerByPort($port);
112 112
 		}
113 113
 		return $return;
114 114
 	}
115 115
 
116 116
 	protected function &doPlanetDamageToPlayer(array &$return, SmrPlanet $planet, AbstractSmrPlayer $targetPlayer) {
117
-		$return['WeaponDamage'] =& $this->getModifiedPlanetDamageAgainstPlayer($planet,$targetPlayer);
118
-		$return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
117
+		$return['WeaponDamage'] = & $this->getModifiedPlanetDamageAgainstPlayer($planet, $targetPlayer);
118
+		$return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
119 119
 
120 120
 		if ($return['ActualDamage']['KillingShot']) {
121
-			$return['KillResults'] =& $targetPlayer->killPlayerByPlanet($planet);
121
+			$return['KillResults'] = & $targetPlayer->killPlayerByPlanet($planet);
122 122
 		}
123 123
 		return $return;
124 124
 	}
125 125
 
126 126
 	protected function &doForceDamageToPlayer(array &$return, SmrForce $forces, AbstractSmrPlayer $targetPlayer) {
127
-		$return['WeaponDamage'] =& $this->getModifiedForceDamageAgainstPlayer($forces,$targetPlayer);
128
-		$return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
127
+		$return['WeaponDamage'] = & $this->getModifiedForceDamageAgainstPlayer($forces, $targetPlayer);
128
+		$return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']);
129 129
 
130 130
 		if ($return['ActualDamage']['KillingShot']) {
131
-			$return['KillResults'] =& $targetPlayer->killPlayerByForces($forces);
131
+			$return['KillResults'] = & $targetPlayer->killPlayerByForces($forces);
132 132
 		}
133 133
 		return $return;
134 134
 	}
Please login to merge, or discard this patch.
src/lib/Default/ChessPiece.class.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
 	}
235 235
 
236 236
 	public static function getLetterForPiece(int $pieceID, string $colour) : string {
237
-		$letter = match ($pieceID) {
237
+		$letter = match($pieceID) {
238 238
 			self::KING => 'k',
239 239
 			self::QUEEN => 'q',
240 240
 			self::ROOK => 'r',
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
 	}
250 250
 
251 251
 	public static function getPieceForLetter(string $letter) : int {
252
-		return match (strtolower($letter)) {
252
+		return match(strtolower($letter)) {
253 253
 			'k' => self::KING,
254 254
 			'q' => self::QUEEN,
255 255
 			'r' => self::ROOK,
Please login to merge, or discard this patch.