smrealms / smr

src/pages/Player/MessageView.php

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 					AND receiver_delete = ' . $db->escapeBoolean(false) . '
 					ORDER BY send_time DESC');
 	foreach ($dbResult->records() as $dbRecord) {
-		$groupBox =& $messageBox['GroupedMessages'][$dbRecord->getInt('sender_id')];
+		$groupBox = & $messageBox['GroupedMessages'][$dbRecord->getInt('sender_id')];
 		// Limit the number of messages in each group
 		if (!isset($groupBox['Messages']) || count($groupBox['Messages']) < MESSAGE_SCOUT_GROUP_LIMIT) {
 			$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());

src/pages/Player/ChatSharing.php

Indentation +4 added lines, -4 removed lines

@@ -38,8 +38,8 @@ 
 			$shareFrom[$fromAccountId] = [
 				'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(),
 				'Player Name' => $otherPlayer == null ?
-				                 '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() :
-				                 $otherPlayer->getDisplayName(),
+								 '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() :
+								 $otherPlayer->getDisplayName(),
 				'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>',
 				'Game ID' => $gameId,
 			];
@@ -59,8 +59,8 @@ 
 			$shareTo[$toAccountId] = [
 				'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(),
 				'Player Name' => $otherPlayer == null ?
-				                 '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() :
-				                 $otherPlayer->getDisplayName(),
+								 '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() :
+								 $otherPlayer->getDisplayName(),
 				'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>',
 				'Game ID' => $gameId,
 			];

Spacing +2 added lines, -4 removed lines

@@ -38,8 +38,7 @@ 
 			$shareFrom[$fromAccountId] = [
 				'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(),
 				'Player Name' => $otherPlayer == null ?
-				                 '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() :
-				                 $otherPlayer->getDisplayName(),
+				                 '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(),
 				'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>',
 				'Game ID' => $gameId,
 			];
@@ -59,8 +58,7 @@ 
 			$shareTo[$toAccountId] = [
 				'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(),
 				'Player Name' => $otherPlayer == null ?
-				                 '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() :
-				                 $otherPlayer->getDisplayName(),
+				                 '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(),
 				'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>',
 				'Game ID' => $gameId,
 			];

src/pages/Player/Rankings/AllianceVsAlliance.php

Indentation +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 			foreach ($alliance_vs_ids as $id) {
 				$row_alliance = SmrAlliance::getAlliance($id, $player->getGameID());
 				$showRed = (!$curr_alliance->isNone() && $curr_alliance->hasDisbanded()) ||
-				           (!$row_alliance->isNone() && $row_alliance->hasDisbanded());
+						   (!$row_alliance->isNone() && $row_alliance->hasDisbanded());
 				$showBold = $curr_id == $player->getAllianceID() || $id == $player->getAllianceID();
 				$style = '';
 				if ($curr_id == $id && !$row_alliance->isNone()) {

src/pages/Player/ShopGoods.php

Indentation +3 added lines, -3 removed lines

@@ -65,9 +65,9 @@
 					//get base for ports that dont happen to trade that good
 					$GOODS = Globals::getGoods();
 					$fine = $totalFine = $port->getLevel() *
-					    (($ship->getCargo(GOODS_SLAVES) * $GOODS[GOODS_SLAVES]['BasePrice']) +
-					     ($ship->getCargo(GOODS_WEAPONS) * $GOODS[GOODS_WEAPONS]['BasePrice']) +
-					     ($ship->getCargo(GOODS_NARCOTICS) * $GOODS[GOODS_NARCOTICS]['BasePrice']));
+						(($ship->getCargo(GOODS_SLAVES) * $GOODS[GOODS_SLAVES]['BasePrice']) +
+						 ($ship->getCargo(GOODS_WEAPONS) * $GOODS[GOODS_WEAPONS]['BasePrice']) +
+						 ($ship->getCargo(GOODS_NARCOTICS) * $GOODS[GOODS_NARCOTICS]['BasePrice']));
 					$player->increaseHOF($ship->getCargo(GOODS_SLAVES) + $ship->getCargo(GOODS_WEAPONS) + $ship->getCargo(GOODS_NARCOTICS), ['Trade', 'Search', 'Caught', 'Goods Confiscated'], HOF_PUBLIC);
 					$player->increaseHOF($totalFine, ['Trade', 'Search', 'Caught', 'Amount Fined'], HOF_PUBLIC);
 					$template->assign('TotalFine', $totalFine);

src/pages/Player/AttackPort.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 	 * @param array<mixed> $results
 	 */
 	public function __construct(
-		private readonly ?array $results = null,
+		private readonly ? array $results = null,
 		bool $playerDied = false
 	) {
 		// If the player died, make sure they see combat results

src/pages/Account/NewsReadAdvanced.php

Spacing +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
 		private readonly int $gameID,
 		private readonly ?string $submit = null,
 		private readonly ?string $label = null,
-		private readonly ?array $accountIDs = null,
-		private readonly ?array $allianceIDs = null,
+		private readonly ? array $accountIDs = null,
+		private readonly ? array $allianceIDs = null,
 	) {}
 
 	public function build(SmrAccount $account, Template $template): void {

src/lib/Default/AbstractMenu.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 		$menuItems[] = ['Link' => Globals::getPlanetListHREF($alliance_id), 'Text' => 'Defense'];
 		$menuItems[] = ['Link' => Globals::getPlanetListFinancialHREF($alliance_id), 'Text' => 'Financial'];
 		// make the selected index bold
-		$boldItem =& $menuItems[$selected_index]['Text'];
+		$boldItem = & $menuItems[$selected_index]['Text'];
 		$boldItem = '<span class="bold">' . $boldItem . '</span>';
 
 		$template = Smr\Template::getInstance();

src/pages/Admin/DatabaseCleanup.php

Spacing +2 added lines, -2 removed lines

@@ -17,9 +17,9 @@
 	 */
 	public function __construct(
 		private readonly ?string $action = null,
-		private readonly ?array $results = null,
+		private readonly ? array $results = null,
 		private readonly ?int $diffBytes = null,
-		private readonly ?array $endedGames = null
+		private readonly ? array $endedGames = null
 	) {}
 
 	public function build(SmrAccount $account, Template $template): void {