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

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/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 {