We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -2,8 +2,7 @@ |
||
2 | 2 | |
3 | 3 | // Get the dates ("|" sets hr/min/sec to 0) |
4 | 4 | $join = DateTime::createFromFormat('d/m/Y|', Request::get('game_join')); |
5 | -$start = empty(Request::get('game_start')) ? $join : |
|
6 | - DateTime::createFromFormat('d/m/Y|', Request::get('game_start')); |
|
5 | +$start = empty(Request::get('game_start')) ? $join : DateTime::createFromFormat('d/m/Y|', Request::get('game_start')); |
|
7 | 6 | $end = DateTime::createFromFormat('d/m/Y|', Request::get('game_end')); |
8 | 7 | |
9 | 8 | $game = SmrGame::getGame($var['game_id']); |
@@ -20,13 +20,13 @@ |
||
20 | 20 | const USER_SCORE_NAME = 'User Score'; |
21 | 21 | $hofTypes = array(DONATION_NAME=>true, USER_SCORE_NAME=>true); |
22 | 22 | while ($db->nextRecord()) { |
23 | - $hof =& $hofTypes; |
|
23 | + $hof = & $hofTypes; |
|
24 | 24 | $typeList = explode(':', $db->getField('type')); |
25 | 25 | foreach ($typeList as $type) { |
26 | 26 | if (!isset($hof[$type])) { |
27 | 27 | $hof[$type] = array(); |
28 | 28 | } |
29 | - $hof =& $hof[$type]; |
|
29 | + $hof = & $hof[$type]; |
|
30 | 30 | } |
31 | 31 | $hof = true; |
32 | 32 | } |
@@ -19,8 +19,7 @@ discard block |
||
19 | 19 | $shareFrom[$fromAccountId] = array( |
20 | 20 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), |
21 | 21 | 'Player Name' => $otherPlayer == null ? |
22 | - '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : |
|
23 | - $otherPlayer->getDisplayName(), |
|
22 | + '<b>Account</b>: ' . SmrAccount::getAccount($fromAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(), |
|
24 | 23 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', |
25 | 24 | 'Game ID' => $gameId, |
26 | 25 | ); |
@@ -40,8 +39,7 @@ discard block |
||
40 | 39 | $shareTo[$toAccountId] = array( |
41 | 40 | 'Player ID' => $otherPlayer == null ? '-' : $otherPlayer->getPlayerID(), |
42 | 41 | 'Player Name' => $otherPlayer == null ? |
43 | - '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : |
|
44 | - $otherPlayer->getDisplayName(), |
|
42 | + '<b>Account</b>: ' . SmrAccount::getAccount($toAccountId)->getHofDisplayName() : $otherPlayer->getDisplayName(), |
|
45 | 43 | 'All Games' => $gameId == 0 ? '<span class="green">YES</span>' : '<span class="red">NO</span>', |
46 | 44 | 'Game ID' => $gameId, |
47 | 45 | ); |
@@ -20,13 +20,13 @@ |
||
20 | 20 | const USER_SCORE_NAME = 'User Score'; |
21 | 21 | $hofTypes = array(DONATION_NAME=>true, USER_SCORE_NAME=>true); |
22 | 22 | while ($db->nextRecord()) { |
23 | - $hof =& $hofTypes; |
|
23 | + $hof = & $hofTypes; |
|
24 | 24 | $typeList = explode(':', $db->getField('type')); |
25 | 25 | foreach ($typeList as $type) { |
26 | 26 | if (!isset($hof[$type])) { |
27 | 27 | $hof[$type] = array(); |
28 | 28 | } |
29 | - $hof =& $hof[$type]; |
|
29 | + $hof = & $hof[$type]; |
|
30 | 30 | } |
31 | 31 | $hof = true; |
32 | 32 | } |
@@ -18,8 +18,7 @@ |
||
18 | 18 | if ($join === false) { |
19 | 19 | create_error('Join Date is not valid!'); |
20 | 20 | } |
21 | -$start = empty(Request::get('game_start')) ? $join : |
|
22 | - DateTime::createFromFormat('d/m/Y|', Request::get('game_start')); |
|
21 | +$start = empty(Request::get('game_start')) ? $join : DateTime::createFromFormat('d/m/Y|', Request::get('game_start')); |
|
23 | 22 | if ($start === false) { |
24 | 23 | create_error('Start Date is not valid!'); |
25 | 24 | } |
@@ -374,13 +374,13 @@ |
||
374 | 374 | $this->db->query('SELECT type,sum(amount) as amount FROM player_hof WHERE ' . $this->SQL . ' AND game_id IN (SELECT game_id FROM game WHERE ignore_stats = \'FALSE\') GROUP BY type'); |
375 | 375 | $this->HOF = array(); |
376 | 376 | while ($this->db->nextRecord()) { |
377 | - $hof =& $this->HOF; |
|
377 | + $hof = & $this->HOF; |
|
378 | 378 | $typeList = explode(':', $this->db->getField('type')); |
379 | 379 | foreach ($typeList as $type) { |
380 | 380 | if (!isset($hof[$type])) { |
381 | 381 | $hof[$type] = array(); |
382 | 382 | } |
383 | - $hof =& $hof[$type]; |
|
383 | + $hof = & $hof[$type]; |
|
384 | 384 | } |
385 | 385 | $hof = $this->db->getFloat('amount'); |
386 | 386 | } |
@@ -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() { |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | } |
117 | 117 | |
118 | 118 | try { |
119 | - $TRADE_ROUTE =& $GLOBALS['TRADE_ROUTE']; |
|
119 | + $TRADE_ROUTE = & $GLOBALS['TRADE_ROUTE']; |
|
120 | 120 | debug('Action #' . $actions); |
121 | 121 | |
122 | 122 | //We have to reload player on each loop |
@@ -140,14 +140,14 @@ discard block |
||
140 | 140 | } |
141 | 141 | |
142 | 142 | if (!isset($TRADE_ROUTE)) { //We only want to change trade route if there isn't already one set. |
143 | - $TRADE_ROUTES =& findRoutes($player); |
|
144 | - $TRADE_ROUTE =& changeRoute($TRADE_ROUTES); |
|
143 | + $TRADE_ROUTES = & findRoutes($player); |
|
144 | + $TRADE_ROUTE = & changeRoute($TRADE_ROUTES); |
|
145 | 145 | } |
146 | 146 | |
147 | 147 | if ($player->isDead()) { |
148 | 148 | debug('Some evil person killed us, let\'s move on now.'); |
149 | 149 | $previousContainer = null; //We died, we don't care what we were doing beforehand. |
150 | - $TRADE_ROUTE =& changeRoute($TRADE_ROUTES); //Change route |
|
150 | + $TRADE_ROUTE = & changeRoute($TRADE_ROUTES); //Change route |
|
151 | 151 | processContainer(Page::create('death_processing.php')); |
152 | 152 | } |
153 | 153 | if ($player->getNewbieTurns() <= NEWBIE_TURNS_WARNING_LIMIT && $player->getNewbieWarning()) { |
@@ -223,7 +223,7 @@ discard block |
||
223 | 223 | processContainer(tradeGoods($goodID, $player, $port)); |
224 | 224 | } else { |
225 | 225 | //Move to next route or fed. |
226 | - if (($TRADE_ROUTE =& changeRoute($TRADE_ROUTES)) === false) { |
|
226 | + if (($TRADE_ROUTE = & changeRoute($TRADE_ROUTES)) === false) { |
|
227 | 227 | debug('Changing Route Failed'); |
228 | 228 | processContainer(plotToFed($player)); |
229 | 229 | } else { |
@@ -250,7 +250,7 @@ discard block |
||
250 | 250 | processContainer(tradeGoods($goodID, $player, $port)); |
251 | 251 | } else { |
252 | 252 | //Move to next route or fed. |
253 | - if (($TRADE_ROUTE =& changeRoute($TRADE_ROUTES)) === false) { |
|
253 | + if (($TRADE_ROUTE = & changeRoute($TRADE_ROUTES)) === false) { |
|
254 | 254 | debug('Changing Route Failed'); |
255 | 255 | processContainer(plotToFed($player)); |
256 | 256 | } else { |
@@ -582,9 +582,9 @@ discard block |
||
582 | 582 | return $false; |
583 | 583 | } |
584 | 584 | $routeKey = array_rand($tradeRoutes); |
585 | - $tradeRoute =& $tradeRoutes[$routeKey]; |
|
585 | + $tradeRoute = & $tradeRoutes[$routeKey]; |
|
586 | 586 | unset($tradeRoutes[$routeKey]); |
587 | - $GLOBALS['TRADE_ROUTE'] =& $tradeRoute; |
|
587 | + $GLOBALS['TRADE_ROUTE'] = & $tradeRoute; |
|
588 | 588 | debug('Switched route', $tradeRoute); |
589 | 589 | return $tradeRoute; |
590 | 590 | } |