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 | } |