We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
| @@ -52,7 +52,7 @@ | ||
| 52 | 52 |  	foreach ($alliance_vs_ids as $id) { | 
| 53 | 53 | $row_alliance = SmrAlliance::getAlliance($id, $player->getGameID()); | 
| 54 | 54 | $showRed = (!$curr_alliance->isNone() && $curr_alliance->hasDisbanded()) || | 
| 55 | - (!$row_alliance->isNone() && $row_alliance->hasDisbanded()); | |
| 55 | + (!$row_alliance->isNone() && $row_alliance->hasDisbanded()); | |
| 56 | 56 | $showBold = $curr_id == $player->getAllianceID() || $id == $player->getAllianceID(); | 
| 57 | 57 | $style = ''; | 
| 58 | 58 |  		if ($curr_id == $id && !$row_alliance->isNone()) { | 
| @@ -337,8 +337,8 @@ | ||
| 337 | 337 |  					foreach (Globals::getAvailableTemplates() as $Template) { | 
| 338 | 338 |  						foreach (Globals::getAvailableColourSchemes($Template) as $ColourScheme) { | 
| 339 | 339 | $selected = ($ThisAccount->getTemplate() == $Template && | 
| 340 | - $ThisAccount->getColourScheme() == $ColourScheme && | |
| 341 | - $ThisAccount->isDefaultCSSEnabled()) ? 'selected' : ''; | |
| 340 | + $ThisAccount->getColourScheme() == $ColourScheme && | |
| 341 | + $ThisAccount->isDefaultCSSEnabled()) ? 'selected' : ''; | |
| 342 | 342 | $name = $Template . ' - ' . $ColourScheme; | 
| 343 | 343 | ?><option value="<?php echo $name; ?>" <?php echo $selected; ?>><?php echo $name; ?></option><?php | 
| 344 | 344 | } | 
| @@ -69,7 +69,7 @@ | ||
| 69 | 69 |  	if (!empty(BUG_REPORT_TO_ADDRESSES)) { | 
| 70 | 70 | $mail = setupMailer(); | 
| 71 | 71 |  		$mail->Subject = (defined('PAGE_PREFIX') ? PAGE_PREFIX : '??? ') . | 
| 72 | - 'Automatic Bug Report'; | |
| 72 | + 'Automatic Bug Report'; | |
| 73 | 73 |  		$mail->setFrom('[email protected]'); | 
| 74 | 74 | $mail->Body = $message; | 
| 75 | 75 |  		foreach (BUG_REPORT_TO_ADDRESSES as $toAddress) { | 
| @@ -121,7 +121,7 @@ | ||
| 121 | 121 | * Can be used to convert any type of notice into an exception. | 
| 122 | 122 | */ | 
| 123 | 123 |  function exception_error_handler(int $errno, string $errstr, string $errfile, int $errline): bool { | 
| 124 | -	if (!(error_reporting() & $errno)) { | |
| 124 | +	if (!(error_reporting()&$errno)) { | |
| 125 | 125 | return false; // error is suppressed | 
| 126 | 126 | } | 
| 127 | 127 | throw new ErrorException($errstr, $errno, E_ERROR, $errfile, $errline); | 
| @@ -6,7 +6,7 @@ | ||
| 6 | 6 |  } else { | 
| 7 | 7 | // Killed by port, planet, forces | 
| 8 | 8 | echo 'The <span class="creds"> ' . number_format($KillResults['LostCredits']) | 
| 9 | - . '</span> credits that were onboard ' . $TargetPlayer->getDisplayName() | |
| 10 | - . "'s ship are lost in the wreckage.<br />"; | |
| 9 | + . '</span> credits that were onboard ' . $TargetPlayer->getDisplayName() | |
| 10 | + . "'s ship are lost in the wreckage.<br />"; | |
| 11 | 11 | } | 
| 12 | 12 | ?> | 
| @@ -102,7 +102,7 @@ | ||
| 102 | 102 |  						if (!$UniGen) { | 
| 103 | 103 | $CanScanSector = ($ThisShip->hasScanner() && $isLinkedSector) || $isCurrentSector; | 
| 104 | 104 | $ShowFriendlyForces = isset($HideAlliedForces) && $HideAlliedForces ? | 
| 105 | - $Sector->hasPlayerForces($MapPlayer) : $Sector->hasFriendlyForces($MapPlayer); | |
| 105 | + $Sector->hasPlayerForces($MapPlayer) : $Sector->hasFriendlyForces($MapPlayer); | |
| 106 | 106 |  							if (($CanScanSector && ($Sector->hasForces() || $Sector->hasPlayers())) || $ShowFriendlyForces || $Sector->hasFriendlyTraders($MapPlayer)) { ?> | 
| 107 | 107 | <div class="lmtf"><?php | 
| 108 | 108 |  									if ($CanScanSector && $Sector->hasEnemyTraders($MapPlayer)) { | 
| @@ -128,7 +128,7 @@ | ||
| 128 | 128 | AND receiver_delete = ' . $db->escapeBoolean(false) . ' | 
| 129 | 129 | ORDER BY send_time DESC'); | 
| 130 | 130 |  	foreach ($dbResult->records() as $dbRecord) { | 
| 131 | -		$groupBox =& $messageBox['GroupedMessages'][$dbRecord->getInt('sender_id')]; | |
| 131 | +		$groupBox = & $messageBox['GroupedMessages'][$dbRecord->getInt('sender_id')]; | |
| 132 | 132 | // Limit the number of messages in each group | 
| 133 | 133 |  		if (!isset($groupBox['Messages']) || count($groupBox['Messages']) < MESSAGE_SCOUT_GROUP_LIMIT) { | 
| 134 | 134 |  			$groupBox['Messages'][] = displayMessage($dbRecord->getInt('message_id'), $dbRecord->getInt('account_id'), $dbRecord->getInt('sender_id'), $player->getGameID(), $dbRecord->getString('message_text'), $dbRecord->getInt('send_time'), $dbRecord->getBoolean('msg_read'), MSG_SCOUT, $player->getAccount()); | 
| @@ -14,8 +14,8 @@ | ||
| 14 | 14 | 1 => ['pipe', 'w'], // stdout is a pipe that the child will write to | 
| 15 | 15 | ]; | 
| 16 | 16 | $engine = proc_open(UCI_CHESS_ENGINE, $descriptorSpec, $pipes); | 
| 17 | - $toEngine =& $pipes[0]; | |
| 18 | - $fromEngine =& $pipes[1]; | |
| 17 | + $toEngine = & $pipes[0]; | |
| 18 | + $fromEngine = & $pipes[1]; | |
| 19 | 19 | |
| 20 | 20 |  	function readFromEngine(bool $block = true): void { | 
| 21 | 21 | global $fromEngine; | 
| @@ -53,9 +53,9 @@ | ||
| 53 | 53 | //get base for ports that dont happen to trade that good | 
| 54 | 54 | $GOODS = Globals::getGoods(); | 
| 55 | 55 | $fine = $totalFine = $port->getLevel() * | 
| 56 | - (($ship->getCargo(GOODS_SLAVES) * $GOODS[GOODS_SLAVES]['BasePrice']) + | |
| 57 | - ($ship->getCargo(GOODS_WEAPONS) * $GOODS[GOODS_WEAPONS]['BasePrice']) + | |
| 58 | - ($ship->getCargo(GOODS_NARCOTICS) * $GOODS[GOODS_NARCOTICS]['BasePrice'])); | |
| 56 | + (($ship->getCargo(GOODS_SLAVES) * $GOODS[GOODS_SLAVES]['BasePrice']) + | |
| 57 | + ($ship->getCargo(GOODS_WEAPONS) * $GOODS[GOODS_WEAPONS]['BasePrice']) + | |
| 58 | + ($ship->getCargo(GOODS_NARCOTICS) * $GOODS[GOODS_NARCOTICS]['BasePrice'])); | |
| 59 | 59 | $player->increaseHOF($ship->getCargo(GOODS_SLAVES) + $ship->getCargo(GOODS_WEAPONS) + $ship->getCargo(GOODS_NARCOTICS), ['Trade', 'Search', 'Caught', 'Goods Confiscated'], HOF_PUBLIC); | 
| 60 | 60 | $player->increaseHOF($totalFine, ['Trade', 'Search', 'Caught', 'Amount Fined'], HOF_PUBLIC); | 
| 61 | 61 |  			$template->assign('TotalFine', $totalFine); | 
| @@ -26,8 +26,8 @@ discard block | ||
| 26 | 26 | $shareFrom[$fromAccountId] = [ | 
| 27 | 27 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), | 
| 28 | 28 | 'Player Name' => $otherPlayer == null ? | 
| 29 | - '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : | |
| 30 | - $otherPlayer->getDisplayName(), | |
| 29 | + '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : | |
| 30 | + $otherPlayer->getDisplayName(), | |
| 31 | 31 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', | 
| 32 | 32 | 'Game ID' => $gameId, | 
| 33 | 33 | ]; | 
| @@ -47,8 +47,8 @@ discard block | ||
| 47 | 47 | $shareTo[$toAccountId] = [ | 
| 48 | 48 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), | 
| 49 | 49 | 'Player Name' => $otherPlayer == null ? | 
| 50 | - '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : | |
| 51 | - $otherPlayer->getDisplayName(), | |
| 50 | + '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : | |
| 51 | + $otherPlayer->getDisplayName(), | |
| 52 | 52 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', | 
| 53 | 53 | 'Game ID' => $gameId, | 
| 54 | 54 | ]; | 
| @@ -26,8 +26,7 @@ discard block | ||
| 26 | 26 | $shareFrom[$fromAccountId] = [ | 
| 27 | 27 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), | 
| 28 | 28 | 'Player Name' => $otherPlayer == null ? | 
| 29 | - '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : | |
| 30 | - $otherPlayer->getDisplayName(), | |
| 29 | + '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(), | |
| 31 | 30 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', | 
| 32 | 31 | 'Game ID' => $gameId, | 
| 33 | 32 | ]; | 
| @@ -47,8 +46,7 @@ discard block | ||
| 47 | 46 | $shareTo[$toAccountId] = [ | 
| 48 | 47 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), | 
| 49 | 48 | 'Player Name' => $otherPlayer == null ? | 
| 50 | - '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : | |
| 51 | - $otherPlayer->getDisplayName(), | |
| 49 | + '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(), | |
| 52 | 50 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', | 
| 53 | 51 | 'Game ID' => $gameId, | 
| 54 | 52 | ]; |