We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
| @@ -2,21 +2,21 @@ discard block | ||
| 2 | 2 | |
| 3 | 3 | $template = Smr\Template::getInstance(); | 
| 4 | 4 | |
| 5 | -$template->assign('PageTopic','Combat Simulator'); | |
| 5 | +$template->assign('PageTopic', 'Combat Simulator'); | |
| 6 | 6 | |
| 7 | -$template->assign('EditDummysLink', Page::create('skeleton.php','edit_dummys.php')->href()); | |
| 7 | +$template->assign('EditDummysLink', Page::create('skeleton.php', 'edit_dummys.php')->href()); | |
| 8 | 8 |  $template->assign('DummyNames', DummyPlayer::getDummyPlayerNames()); | 
| 9 | 9 | |
| 10 | 10 | $duplicates = false; | 
| 11 | 11 | $usedNames = array(); | 
| 12 | 12 | $realAttackers = array(); | 
| 13 | 13 | $attackers = array(); | 
| 14 | -$i=1; | |
| 15 | -if(isset($_POST['attackers'])) | |
| 16 | -	foreach($_POST['attackers'] as $attackerName) { | |
| 17 | - if($attackerName=='none') | |
| 14 | +$i = 1; | |
| 15 | +if (isset($_POST['attackers'])) | |
| 16 | +	foreach ($_POST['attackers'] as $attackerName) { | |
| 17 | + if ($attackerName == 'none') | |
| 18 | 18 | continue; | 
| 19 | -		if(isset($usedNames[$attackerName])) { | |
| 19 | +		if (isset($usedNames[$attackerName])) { | |
| 20 | 20 | $duplicates = true; | 
| 21 | 21 | continue; | 
| 22 | 22 | } | 
| @@ -27,18 +27,18 @@ discard block | ||
| 27 | 27 | ++$i; | 
| 28 | 28 | } | 
| 29 | 29 | |
| 30 | -for(;$i<=10;++$i) | |
| 30 | +for (;$i <= 10; ++$i) | |
| 31 | 31 | $attackers[$i] = null; | 
| 32 | -$template->assign('Attackers',$attackers); | |
| 32 | +$template->assign('Attackers', $attackers); | |
| 33 | 33 | |
| 34 | -$i=1; | |
| 34 | +$i = 1; | |
| 35 | 35 | $realDefenders = array(); | 
| 36 | 36 | $defenders = array(); | 
| 37 | -if(isset($_POST['defenders'])) | |
| 38 | -	foreach($_POST['defenders'] as $defenderName) { | |
| 39 | - if($defenderName=='none') | |
| 37 | +if (isset($_POST['defenders'])) | |
| 38 | +	foreach ($_POST['defenders'] as $defenderName) { | |
| 39 | + if ($defenderName == 'none') | |
| 40 | 40 | continue; | 
| 41 | -		if(isset($usedNames[$defenderName])) { | |
| 41 | +		if (isset($usedNames[$defenderName])) { | |
| 42 | 42 | $duplicates = true; | 
| 43 | 43 | continue; | 
| 44 | 44 | } | 
| @@ -49,34 +49,34 @@ discard block | ||
| 49 | 49 | ++$i; | 
| 50 | 50 | } | 
| 51 | 51 | |
| 52 | -for(;$i<=10;++$i) | |
| 52 | +for (;$i <= 10; ++$i) | |
| 53 | 53 | $defenders[$i] = null; | 
| 54 | -$template->assign('Defenders',$defenders); | |
| 54 | +$template->assign('Defenders', $defenders); | |
| 55 | 55 | |
| 56 | -$template->assign('Duplicates',$duplicates); | |
| 56 | +$template->assign('Duplicates', $duplicates); | |
| 57 | 57 | |
| 58 | -$template->assign('CombatSimHREF', Page::create('skeleton.php','combat_simulator.php')->href()); | |
| 58 | +$template->assign('CombatSimHREF', Page::create('skeleton.php', 'combat_simulator.php')->href()); | |
| 59 | 59 | |
| 60 | 60 |  if (!empty($realAttackers) && !empty($realDefenders)) { | 
| 61 | -	if(isset($_REQUEST['run'])) { | |
| 62 | - runAnAttack($realAttackers,$realDefenders); | |
| 61 | +	if (isset($_REQUEST['run'])) { | |
| 62 | + runAnAttack($realAttackers, $realDefenders); | |
| 63 | 63 | } | 
| 64 | -	if(isset($_REQUEST['death_run'])) { | |
| 65 | -		while(count($realAttackers)>0 && count($realDefenders)>0) { | |
| 66 | - runAnAttack($realAttackers,$realDefenders); | |
| 67 | -			foreach($realAttackers as $key => &$teamPlayer) { | |
| 68 | - if($teamPlayer->isDead()) | |
| 64 | +	if (isset($_REQUEST['death_run'])) { | |
| 65 | +		while (count($realAttackers) > 0 && count($realDefenders) > 0) { | |
| 66 | + runAnAttack($realAttackers, $realDefenders); | |
| 67 | +			foreach ($realAttackers as $key => &$teamPlayer) { | |
| 68 | + if ($teamPlayer->isDead()) | |
| 69 | 69 | unset($realAttackers[$key]); | 
| 70 | 70 | } unset($teamPlayer); | 
| 71 | -			foreach($realDefenders as $key => &$teamPlayer) { | |
| 72 | - if($teamPlayer->isDead()) | |
| 71 | +			foreach ($realDefenders as $key => &$teamPlayer) { | |
| 72 | + if ($teamPlayer->isDead()) | |
| 73 | 73 | unset($realDefenders[$key]); | 
| 74 | 74 | } unset($teamPlayer); | 
| 75 | 75 | } | 
| 76 | 76 | } | 
| 77 | 77 | } | 
| 78 | 78 | |
| 79 | -function runAnAttack($realAttackers,$realDefenders) { | |
| 79 | +function runAnAttack($realAttackers, $realDefenders) { | |
| 80 | 80 |  	$results = array('Attackers' => array('Traders' => array(), 'TotalDamage' => 0), | 
| 81 | 81 |  					'Defenders' => array('Traders' => array(), 'TotalDamage' => 0)); | 
| 82 | 82 |  	foreach ($realAttackers as $accountID => $teamPlayer) { | 
| @@ -86,10 +86,10 @@ discard block | ||
| 86 | 86 | } | 
| 87 | 87 |  	foreach ($realDefenders as $accountID => $teamPlayer) { | 
| 88 | 88 | $playerResults = $teamPlayer->shootPlayers($realAttackers); | 
| 89 | - $results['Defenders']['Traders'][] = $playerResults; | |
| 89 | + $results['Defenders']['Traders'][] = $playerResults; | |
| 90 | 90 | $results['Defenders']['TotalDamage'] += $playerResults['TotalDamage']; | 
| 91 | 91 | } | 
| 92 | 92 | |
| 93 | 93 | $template = Smr\Template::getInstance(); | 
| 94 | -	$template->assign('TraderCombatResults',$results); | |
| 94 | +	$template->assign('TraderCombatResults', $results); | |
| 95 | 95 | } | 
| @@ -1,30 +1,30 @@ discard block | ||
| 1 | 1 | <?php declare(strict_types=1); | 
| 2 | 2 | |
| 3 | 3 |  class DummyPlayer extends AbstractSmrPlayer { | 
| 4 | -	public function __construct($gameID=0,$playerName='Dummy',$raceID=1,$experience=1000,$alignment=100,$allianceID=0,$shipTypeID=60) { | |
| 4 | +	public function __construct($gameID = 0, $playerName = 'Dummy', $raceID = 1, $experience = 1000, $alignment = 100, $allianceID = 0, $shipTypeID = 60) { | |
| 5 | 5 | $this->accountID = 0; | 
| 6 | - $this->gameID = (int) $gameID; | |
| 7 | - $this->playerName = (string) $playerName; | |
| 6 | + $this->gameID = (int)$gameID; | |
| 7 | + $this->playerName = (string)$playerName; | |
| 8 | 8 | $this->playerID = 0; | 
| 9 | 9 | $this->sectorID = 0; | 
| 10 | - $this->lastSectorID = 0; | |
| 10 | + $this->lastSectorID = 0; | |
| 11 | 11 | $this->turns = 1000; | 
| 12 | - $this->newbieTurns = 0; | |
| 12 | + $this->newbieTurns = 0; | |
| 13 | 13 | $this->lastNewsUpdate = 0; | 
| 14 | 14 | $this->dead = false; | 
| 15 | 15 | $this->landedOnPlanet = false; | 
| 16 | - $this->lastActive = 0; | |
| 17 | - $this->lastCPLAction = 0; | |
| 18 | - $this->raceID = (int) $raceID; | |
| 19 | - $this->credits = 0; | |
| 20 | - $this->experience = (int) $experience; | |
| 21 | - $this->alignment = (int) $alignment; | |
| 22 | - $this->militaryPayment = 0; | |
| 23 | - $this->allianceID = (int) $allianceID; | |
| 24 | - $this->shipID = (int) $shipTypeID; | |
| 16 | + $this->lastActive = 0; | |
| 17 | + $this->lastCPLAction = 0; | |
| 18 | + $this->raceID = (int)$raceID; | |
| 19 | + $this->credits = 0; | |
| 20 | + $this->experience = (int)$experience; | |
| 21 | + $this->alignment = (int)$alignment; | |
| 22 | + $this->militaryPayment = 0; | |
| 23 | + $this->allianceID = (int)$allianceID; | |
| 24 | + $this->shipID = (int)$shipTypeID; | |
| 25 | 25 | $this->kills = 0; | 
| 26 | 26 | $this->deaths = 0; | 
| 27 | - $this->lastPort = 0; | |
| 27 | + $this->lastPort = 0; | |
| 28 | 28 | $this->bank = 0; | 
| 29 | 29 | $this->zoom = 0; | 
| 30 | 30 | |
| @@ -48,9 +48,9 @@ discard block | ||
| 48 | 48 | } | 
| 49 | 49 | |
| 50 | 50 |  	public function setAllianceID($ID) { | 
| 51 | - if($this->allianceID == $ID) | |
| 51 | + if ($this->allianceID == $ID) | |
| 52 | 52 | return; | 
| 53 | - $this->allianceID=$ID; | |
| 53 | + $this->allianceID = $ID; | |
| 54 | 54 | } | 
| 55 | 55 | |
| 56 | 56 |  	public function killPlayerByPlayer(AbstractSmrPlayer $killer) { | 
| @@ -75,7 +75,7 @@ discard block | ||
| 75 | 75 | $db = Smr\Database::getInstance(); | 
| 76 | 76 |  		$db->query('REPLACE INTO cached_dummys ' . | 
| 77 | 77 | '(type, id, info) ' . | 
| 78 | - 'VALUES (\'DummyPlayer\', '.$db->escapeString($this->getPlayerName()).', '.$db->escapeObject($this).')'); | |
| 78 | + 'VALUES (\'DummyPlayer\', ' . $db->escapeString($this->getPlayerName()) . ', ' . $db->escapeObject($this) . ')'); | |
| 79 | 79 | } | 
| 80 | 80 | |
| 81 | 81 |  	public static function getCachedDummyPlayer($name) { | 
| @@ -83,7 +83,7 @@ discard block | ||
| 83 | 83 |  		$db->query('SELECT info FROM cached_dummys | 
| 84 | 84 | WHERE type = \'DummyPlayer\' | 
| 85 | 85 | AND id = ' . $db->escapeString($name) . ' LIMIT 1'); | 
| 86 | -		if($db->nextRecord()) { | |
| 86 | +		if ($db->nextRecord()) { | |
| 87 | 87 |  			return $db->getObject('info'); | 
| 88 | 88 | } | 
| 89 | 89 |  		else { | 
| @@ -96,7 +96,7 @@ discard block | ||
| 96 | 96 |  		$db->query('SELECT id FROM cached_dummys | 
| 97 | 97 | WHERE type = \'DummyPlayer\''); | 
| 98 | 98 | $dummyNames = array(); | 
| 99 | -		while($db->nextRecord()) { | |
| 99 | +		while ($db->nextRecord()) { | |
| 100 | 100 |  			$dummyNames[] = $db->getField('id'); | 
| 101 | 101 | } | 
| 102 | 102 | return $dummyNames; | 
| @@ -48,8 +48,9 @@ discard block | ||
| 48 | 48 | } | 
| 49 | 49 | |
| 50 | 50 |  	public function setAllianceID($ID) { | 
| 51 | - if($this->allianceID == $ID) | |
| 52 | - return; | |
| 51 | +		if($this->allianceID == $ID) { | |
| 52 | + return; | |
| 53 | + } | |
| 53 | 54 | $this->allianceID=$ID; | 
| 54 | 55 | } | 
| 55 | 56 | |
| @@ -85,8 +86,7 @@ discard block | ||
| 85 | 86 | AND id = ' . $db->escapeString($name) . ' LIMIT 1'); | 
| 86 | 87 |  		if($db->nextRecord()) { | 
| 87 | 88 |  			return $db->getObject('info'); | 
| 88 | - } | |
| 89 | -		else { | |
| 89 | +		} else { | |
| 90 | 90 | return new DummyPlayer(); | 
| 91 | 91 | } | 
| 92 | 92 | } |