smrealms / smr

src/pages/Admin/NewsletterSendProcessor.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
 		$mail->Subject = Request::get('subject');
 
-		$set_mail_body = function (PHPMailer $mail, ?string $newsletterHtml, ?string $newsletterText, ?string $salutation): void {
+		$set_mail_body = function(PHPMailer $mail, ?string $newsletterHtml, ?string $newsletterText, ?string $salutation): void {
 			// Prepend the salutation if one is given
 			if ($salutation) {
 				if (!empty($newsletterHtml)) {

src/pages/Admin/CombatSimulator.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	 * @param array<\AbstractSmrPlayer> $defenders
 	 */
 	public function __construct(
-		private readonly ?array $results = null,
+		private readonly ? array $results = null,
 		private readonly array $attackers = [],
 		private readonly array $defenders = []
 	) {}

src/pages/Admin/UniGen/CreateGameProcessor.php

Spacing +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
 		if ($join === false) {
 			create_error('Join Date is not valid!');
 		}
-		$start = empty(Request::get('game_start')) ? $join :
-			DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
+		$start = empty(Request::get('game_start')) ? $join : DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
 		if ($start === false) {
 			create_error('Start Date is not valid!');
 		}

src/pages/Admin/UniGen/EditGameProcessor.php

Spacing +1 added lines, -2 removed lines

@@ -18,8 +18,7 @@
 	public function build(SmrAccount $account): never {
 		// Get the dates ("|" sets hr/min/sec to 0)
 		$join = DateTime::createFromFormat('d/m/Y|', Request::get('game_join'));
-		$start = empty(Request::get('game_start')) ? $join :
-			DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
+		$start = empty(Request::get('game_start')) ? $join : DateTime::createFromFormat('d/m/Y|', Request::get('game_start'));
 		$end = DateTime::createFromFormat('d/m/Y|', Request::get('game_end'));
 
 		$game = SmrGame::getGame($this->gameID);

src/pages/Admin/DatabaseCleanup.php

Spacing +3 added lines, -3 removed lines

@@ -17,15 +17,15 @@
 	 */
 	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 {
 		$template->assign('PageTopic', 'Database Cleanup');
 
-		$bytesToMB = function (int $bytes): string {
+		$bytesToMB = function(int $bytes): string {
 			return round($bytes / (1024 * 1024), 1) . ' MB';
 		};
 

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 {