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

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);