We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -66,68 +66,68 @@ |
||
| 66 | 66 | abstract public function &getModifiedForceDamageAgainstPlayer(SmrForce $forces, AbstractSmrPlayer $targetPlayer); |
| 67 | 67 | |
| 68 | 68 | protected function &doPlayerDamageToForce(array &$return, AbstractSmrPlayer $weaponPlayer, SmrForce $forces) { |
| 69 | - $return['WeaponDamage'] =& $this->getModifiedDamageAgainstForces($weaponPlayer,$forces); |
|
| 70 | - $return['ActualDamage'] =& $forces->doWeaponDamage($return['WeaponDamage']); |
|
| 69 | + $return['WeaponDamage'] = & $this->getModifiedDamageAgainstForces($weaponPlayer, $forces); |
|
| 70 | + $return['ActualDamage'] = & $forces->doWeaponDamage($return['WeaponDamage']); |
|
| 71 | 71 | if ($return['ActualDamage']['KillingShot']) { |
| 72 | - $return['KillResults'] =& $forces->killForcesByPlayer($weaponPlayer); |
|
| 72 | + $return['KillResults'] = & $forces->killForcesByPlayer($weaponPlayer); |
|
| 73 | 73 | } |
| 74 | 74 | return $return; |
| 75 | 75 | } |
| 76 | 76 | |
| 77 | 77 | protected function &doPlayerDamageToPlayer(array &$return, AbstractSmrPlayer $weaponPlayer, AbstractSmrPlayer $targetPlayer) { |
| 78 | - $return['WeaponDamage'] =& $this->getModifiedDamageAgainstPlayer($weaponPlayer,$targetPlayer); |
|
| 79 | - $return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 78 | + $return['WeaponDamage'] = & $this->getModifiedDamageAgainstPlayer($weaponPlayer, $targetPlayer); |
|
| 79 | + $return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 80 | 80 | |
| 81 | 81 | if ($return['ActualDamage']['KillingShot']) { |
| 82 | - $return['KillResults'] =& $targetPlayer->killPlayerByPlayer($weaponPlayer); |
|
| 82 | + $return['KillResults'] = & $targetPlayer->killPlayerByPlayer($weaponPlayer); |
|
| 83 | 83 | } |
| 84 | 84 | return $return; |
| 85 | 85 | } |
| 86 | 86 | |
| 87 | 87 | protected function &doPlayerDamageToPort(array &$return, AbstractSmrPlayer $weaponPlayer, SmrPort $port) { |
| 88 | - $return['WeaponDamage'] =& $this->getModifiedDamageAgainstPort($weaponPlayer,$port); |
|
| 89 | - $return['ActualDamage'] =& $port->doWeaponDamage($return['WeaponDamage']); |
|
| 88 | + $return['WeaponDamage'] = & $this->getModifiedDamageAgainstPort($weaponPlayer, $port); |
|
| 89 | + $return['ActualDamage'] = & $port->doWeaponDamage($return['WeaponDamage']); |
|
| 90 | 90 | if ($return['ActualDamage']['KillingShot']) { |
| 91 | - $return['KillResults'] =& $port->killPortByPlayer($weaponPlayer); |
|
| 91 | + $return['KillResults'] = & $port->killPortByPlayer($weaponPlayer); |
|
| 92 | 92 | } |
| 93 | 93 | return $return; |
| 94 | 94 | } |
| 95 | 95 | |
| 96 | 96 | protected function &doPlayerDamageToPlanet(array &$return, AbstractSmrPlayer $weaponPlayer, SmrPlanet $planet, $delayed) { |
| 97 | - $return['WeaponDamage'] =& $this->getModifiedDamageAgainstPlanet($weaponPlayer,$planet); |
|
| 98 | - $return['ActualDamage'] =& $planet->doWeaponDamage($return['WeaponDamage'],$delayed); |
|
| 97 | + $return['WeaponDamage'] = & $this->getModifiedDamageAgainstPlanet($weaponPlayer, $planet); |
|
| 98 | + $return['ActualDamage'] = & $planet->doWeaponDamage($return['WeaponDamage'], $delayed); |
|
| 99 | 99 | if ($return['ActualDamage']['KillingShot']) { |
| 100 | - $return['KillResults'] =& $planet->killPlanetByPlayer($weaponPlayer); |
|
| 100 | + $return['KillResults'] = & $planet->killPlanetByPlayer($weaponPlayer); |
|
| 101 | 101 | } |
| 102 | 102 | return $return; |
| 103 | 103 | } |
| 104 | 104 | |
| 105 | 105 | protected function &doPortDamageToPlayer(array &$return, SmrPort $port, AbstractSmrPlayer $targetPlayer) { |
| 106 | - $return['WeaponDamage'] =& $this->getModifiedPortDamageAgainstPlayer($port,$targetPlayer); |
|
| 107 | - $return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 106 | + $return['WeaponDamage'] = & $this->getModifiedPortDamageAgainstPlayer($port, $targetPlayer); |
|
| 107 | + $return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 108 | 108 | |
| 109 | 109 | if ($return['ActualDamage']['KillingShot']) { |
| 110 | - $return['KillResults'] =& $targetPlayer->killPlayerByPort($port); |
|
| 110 | + $return['KillResults'] = & $targetPlayer->killPlayerByPort($port); |
|
| 111 | 111 | } |
| 112 | 112 | return $return; |
| 113 | 113 | } |
| 114 | 114 | |
| 115 | 115 | protected function &doPlanetDamageToPlayer(array &$return, SmrPlanet $planet, AbstractSmrPlayer $targetPlayer) { |
| 116 | - $return['WeaponDamage'] =& $this->getModifiedPlanetDamageAgainstPlayer($planet,$targetPlayer); |
|
| 117 | - $return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 116 | + $return['WeaponDamage'] = & $this->getModifiedPlanetDamageAgainstPlayer($planet, $targetPlayer); |
|
| 117 | + $return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 118 | 118 | |
| 119 | 119 | if ($return['ActualDamage']['KillingShot']) { |
| 120 | - $return['KillResults'] =& $targetPlayer->killPlayerByPlanet($planet); |
|
| 120 | + $return['KillResults'] = & $targetPlayer->killPlayerByPlanet($planet); |
|
| 121 | 121 | } |
| 122 | 122 | return $return; |
| 123 | 123 | } |
| 124 | 124 | |
| 125 | 125 | protected function &doForceDamageToPlayer(array &$return, SmrForce $forces, AbstractSmrPlayer $targetPlayer) { |
| 126 | - $return['WeaponDamage'] =& $this->getModifiedForceDamageAgainstPlayer($forces,$targetPlayer); |
|
| 127 | - $return['ActualDamage'] =& $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 126 | + $return['WeaponDamage'] = & $this->getModifiedForceDamageAgainstPlayer($forces, $targetPlayer); |
|
| 127 | + $return['ActualDamage'] = & $targetPlayer->getShip()->doWeaponDamage($return['WeaponDamage']); |
|
| 128 | 128 | |
| 129 | 129 | if ($return['ActualDamage']['KillingShot']) { |
| 130 | - $return['KillResults'] =& $targetPlayer->killPlayerByForces($forces); |
|
| 130 | + $return['KillResults'] = & $targetPlayer->killPlayerByForces($forces); |
|
| 131 | 131 | } |
| 132 | 132 | return $return; |
| 133 | 133 | } |
@@ -93,7 +93,7 @@ discard block |
||
| 93 | 93 | $return = array('MaxDamage' => 0, 'Shield' => 0, 'Armour' => 0, 'Rollover' => $this->isDamageRollover()); |
| 94 | 94 | return $return; |
| 95 | 95 | } |
| 96 | - $damage =& $this->getModifiedDamage(); |
|
| 96 | + $damage = & $this->getModifiedDamage(); |
|
| 97 | 97 | if ($targetPlayer->getShip()->isFederal()) { // do less damage to fed ships |
| 98 | 98 | $damage['MaxDamage'] = IRound($damage['MaxDamage'] * self::FED_SHIP_DAMAGE_MODIFIER); |
| 99 | 99 | $damage['Shield'] = IRound($damage['Shield'] * self::FED_SHIP_DAMAGE_MODIFIER); |
@@ -130,12 +130,12 @@ discard block |
||
| 130 | 130 | } |
| 131 | 131 | |
| 132 | 132 | protected function &doForceDamageToPlayer(array &$return, SmrForce $forces, AbstractSmrPlayer $targetPlayer, $minesAreAttacker = false) { |
| 133 | - $return['WeaponDamage'] =& $this->getModifiedForceDamageAgainstPlayer($forces, $targetPlayer, $minesAreAttacker); |
|
| 134 | - $return['ActualDamage'] =& $targetPlayer->getShip()->doMinesDamage($return['WeaponDamage']); |
|
| 133 | + $return['WeaponDamage'] = & $this->getModifiedForceDamageAgainstPlayer($forces, $targetPlayer, $minesAreAttacker); |
|
| 134 | + $return['ActualDamage'] = & $targetPlayer->getShip()->doMinesDamage($return['WeaponDamage']); |
|
| 135 | 135 | $return['ActualDamage']['Launched'] = ICeil($return['WeaponDamage']['Launched'] * $return['ActualDamage']['TotalDamage'] / $return['WeaponDamage']['MaxDamage']); |
| 136 | 136 | |
| 137 | 137 | if ($return['ActualDamage']['KillingShot']) { |
| 138 | - $return['KillResults'] =& $targetPlayer->killPlayerByForces($forces); |
|
| 138 | + $return['KillResults'] = & $targetPlayer->killPlayerByForces($forces); |
|
| 139 | 139 | } |
| 140 | 140 | return $return; |
| 141 | 141 | } |
@@ -98,7 +98,7 @@ |
||
| 98 | 98 | global $player, $db; |
| 99 | 99 | $offset = $minRank - 1; |
| 100 | 100 | $limit = $maxRank - $offset; |
| 101 | - $db->query('SELECT alliance_id, alliance_' . $stat . ' AS amount FROM alliance WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' ORDER BY amount DESC, alliance_name LIMIT ' . $offset. ', ' . $limit); |
|
| 101 | + $db->query('SELECT alliance_id, alliance_' . $stat . ' AS amount FROM alliance WHERE game_id = ' . $db->escapeNumber($player->getGameID()) . ' ORDER BY amount DESC, alliance_name LIMIT ' . $offset . ', ' . $limit); |
|
| 102 | 102 | return self::collectAllianceRankings($db, $player, $offset); |
| 103 | 103 | } |
| 104 | 104 | |
@@ -25,7 +25,7 @@ |
||
| 25 | 25 | $template->assign('OurRank', $ourRank); |
| 26 | 26 | } |
| 27 | 27 | |
| 28 | -$expRanks = function (int $minRank, int $maxRank) use ($player, $db) : array { |
|
| 28 | +$expRanks = function(int $minRank, int $maxRank) use ($player, $db) : array { |
|
| 29 | 29 | $offset = $minRank - 1; |
| 30 | 30 | $limit = $maxRank - $offset; |
| 31 | 31 | $db->query('SELECT alliance_id, COALESCE(SUM(experience), 0) amount |
@@ -28,7 +28,7 @@ |
||
| 28 | 28 | $template->assign('OurRank', $ourRank); |
| 29 | 29 | } |
| 30 | 30 | |
| 31 | -$profitRanks = function (int $minRank, int $maxRank) use ($player, $db, $profitTypeEscaped) : array { |
|
| 31 | +$profitRanks = function(int $minRank, int $maxRank) use ($player, $db, $profitTypeEscaped) : array { |
|
| 32 | 32 | $offset = $minRank - 1; |
| 33 | 33 | $limit = $maxRank - $offset; |
| 34 | 34 | $db->query('SELECT alliance_id, COALESCE(SUM(amount), 0) amount |
@@ -24,7 +24,7 @@ |
||
| 24 | 24 | |
| 25 | 25 | $totalPlayers = $player->getGame()->getTotalPlayers(); |
| 26 | 26 | |
| 27 | -$profitRanks = function (int $minRank, int $maxRank) use ($player, $db, $profitTypeEscaped) : array { |
|
| 27 | +$profitRanks = function(int $minRank, int $maxRank) use ($player, $db, $profitTypeEscaped) : array { |
|
| 28 | 28 | $offset = $minRank - 1; |
| 29 | 29 | $limit = $maxRank - $offset; |
| 30 | 30 | $db->query('SELECT p.*, COALESCE(ph.amount,0) amount FROM player p LEFT JOIN player_hof ph ON p.account_id = ph.account_id AND p.game_id = ph.game_id AND ph.type = ' . $profitTypeEscaped . ' WHERE p.game_id = ' . $db->escapeNumber($player->getGameID()) . ' ORDER BY amount DESC, player_name ASC LIMIT ' . $offset . ', ' . $limit); |