We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -230,13 +230,13 @@ |
||
| 230 | 230 | $planet->increaseBuilding(PLANET_HANGAR, 4); |
| 231 | 231 | $this->assertTrue($planet->hasBuilding(PLANET_HANGAR)); |
| 232 | 232 | $this->assertSame(4, $planet->getBuilding(PLANET_HANGAR)); |
| 233 | - $this->assertSame(4/3, $planet->getLevel()); |
|
| 233 | + $this->assertSame(4 / 3, $planet->getLevel()); |
|
| 234 | 234 | |
| 235 | 235 | // Destroy some hangars |
| 236 | 236 | $planet->destroyBuilding(PLANET_HANGAR, 2); |
| 237 | 237 | $this->assertTrue($planet->hasBuilding(PLANET_HANGAR)); |
| 238 | 238 | $this->assertSame(2, $planet->getBuilding(PLANET_HANGAR)); |
| 239 | - $this->assertSame(2/3, $planet->getLevel()); |
|
| 239 | + $this->assertSame(2 / 3, $planet->getLevel()); |
|
| 240 | 240 | } |
| 241 | 241 | |
| 242 | 242 | } |
@@ -100,7 +100,7 @@ |
||
| 100 | 100 | $ship = new AbstractSmrShip($this->player); |
| 101 | 101 | $this->expectException(\Exception::class); |
| 102 | 102 | $this->expectExceptionMessage('Ship does not have the supported hardware!'); |
| 103 | - $ship->setIllusion(SHIP_TYPE_THIEF, 12, 13);; |
|
| 103 | + $ship->setIllusion(SHIP_TYPE_THIEF, 12, 13); ; |
|
| 104 | 104 | } |
| 105 | 105 | |
| 106 | 106 | public function test_hardware() { |
@@ -375,8 +375,9 @@ |
||
| 375 | 375 | |
| 376 | 376 | if (!$player->isDead() && $player->getNewbieTurns() <= NEWBIE_TURNS_WARNING_LIMIT && |
| 377 | 377 | $player->getNewbieWarning() && |
| 378 | - $var['url'] != 'newbie_warning_processing.php') |
|
| 379 | - Page::create('newbie_warning_processing.php')->go(); |
|
| 378 | + $var['url'] != 'newbie_warning_processing.php') { |
|
| 379 | + Page::create('newbie_warning_processing.php')->go(); |
|
| 380 | + } |
|
| 380 | 381 | } |
| 381 | 382 | |
| 382 | 383 | // Initialize the template |
@@ -33,7 +33,7 @@ |
||
| 33 | 33 | |
| 34 | 34 | // Form to make planet changes |
| 35 | 35 | $container = Page::create('1.6/universe_create_save_processing.php', |
| 36 | - '1.6/universe_create_sectors.php', $var); |
|
| 36 | + '1.6/universe_create_sectors.php', $var); |
|
| 37 | 37 | $template->assign('CreatePlanetsFormHREF', $container->href()); |
| 38 | 38 | |
| 39 | 39 | // HREF to cancel and return to the previous page |
@@ -94,7 +94,7 @@ |
||
| 94 | 94 | |
| 95 | 95 | // Form to make location changes |
| 96 | 96 | $container = Page::create('1.6/universe_create_save_processing.php', |
| 97 | - '1.6/universe_create_sectors.php', $var); |
|
| 97 | + '1.6/universe_create_sectors.php', $var); |
|
| 98 | 98 | $template->assign('CreateLocationsFormHREF', $container->href()); |
| 99 | 99 | |
| 100 | 100 | // HREF to cancel and return to the previous page |
@@ -7,4 +7,4 @@ |
||
| 7 | 7 | $msg = '<span class="green">SUCCESS: </span>Enabled game ' . $game->getDisplayName(); |
| 8 | 8 | |
| 9 | 9 | Page::create('skeleton.php', 'enable_game.php', |
| 10 | - array('processing_msg' => $msg))->go(); |
|
| 10 | + array('processing_msg' => $msg))->go(); |
|
@@ -40,7 +40,7 @@ discard block |
||
| 40 | 40 | $menuItems[] = array('Link'=>Globals::getPlanetListHREF($alliance_id), 'Text'=>'Defense'); |
| 41 | 41 | $menuItems[] = array('Link'=>Globals::getPlanetListFinancialHREF($alliance_id), 'Text'=>'Financial'); |
| 42 | 42 | // make the selected index bold |
| 43 | - $boldItem =& $menuItems[$selected_index]['Text']; |
|
| 43 | + $boldItem = & $menuItems[$selected_index]['Text']; |
|
| 44 | 44 | $boldItem = '<span class="bold">' . $boldItem . '</span>'; |
| 45 | 45 | |
| 46 | 46 | $template = Smr\Template::getInstance(); |
@@ -146,7 +146,7 @@ discard block |
||
| 146 | 146 | 'Text' => 'Game News', |
| 147 | 147 | ]; |
| 148 | 148 | // make the selected index bold |
| 149 | - $boldItem =& $menuItems[$selected_index]['Text']; |
|
| 149 | + $boldItem = & $menuItems[$selected_index]['Text']; |
|
| 150 | 150 | $boldItem = '<b>' . $boldItem . '</b>'; |
| 151 | 151 | |
| 152 | 152 | $template = Smr\Template::getInstance(); |
@@ -163,7 +163,7 @@ |
||
| 163 | 163 | |
| 164 | 164 | // transfer this value to next container |
| 165 | 165 | if (!isset($var[$source])) { |
| 166 | - throw new Exception('Could not find "' . $source. '" in var!'); |
|
| 166 | + throw new Exception('Could not find "' . $source . '" in var!'); |
|
| 167 | 167 | } |
| 168 | 168 | if ($dest === null) { |
| 169 | 169 | $dest = $source; |
@@ -55,9 +55,9 @@ |
||
| 55 | 55 | $session = Smr\Session::getInstance(); |
| 56 | 56 | $account = $session->getAccount(); |
| 57 | 57 | if (($vis == HOF_PRIVATE && $account->getAccountID() != $accountID) || |
| 58 | - ($vis == HOF_ALLIANCE && isset($gameID) && |
|
| 59 | - !SmrGame::getGame($gameID)->hasEnded() && |
|
| 60 | - !SmrPlayer::getPlayer($accountID, $gameID)->sameAlliance($session->getPlayer()))) |
|
| 58 | + ($vis == HOF_ALLIANCE && isset($gameID) && |
|
| 59 | + !SmrGame::getGame($gameID)->hasEnded() && |
|
| 60 | + !SmrPlayer::getPlayer($accountID, $gameID)->sameAlliance($session->getPlayer()))) |
|
| 61 | 61 | { |
| 62 | 62 | return '-'; |
| 63 | 63 | } else { |