Completed
Push — master ( 5f69d8...11966f )
by Jakub
07:32
created
src/CombatActionSelector.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
   public function chooseAction(CombatBase $combat, Character $character): ?string {
24 24
     if($character->hitpoints < 1) {
25 25
       return null;
26
-    } elseif(in_array($character, $combat->findHealers()->toArray(), true) AND !is_null($combat->selectHealingTarget($character))) {
26
+    } elseif(in_array($character, $combat->findHealers()->toArray(), true) and !is_null($combat->selectHealingTarget($character))) {
27 27
       return CombatAction::ACTION_HEALING;
28 28
     }
29 29
     $attackTarget = $combat->selectAttackTarget($character);
Please login to merge, or discard this patch.
src/Team.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
    * @return Character[]
51 51
    */
52 52
   public function getAliveMembers(): array {
53
-    return $this->getItems(["hitpoints>" => 0,]);
53
+    return $this->getItems(["hitpoints>" => 0, ]);
54 54
   }
55 55
   
56 56
   /**
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
       return null;
129 129
     }
130 130
     for($i = 1; $i <= PHP_INT_MAX; $i++) {
131
-      if($this->hasItems(["positionRow" => $i, "hitpoints>" => 0,])) {
131
+      if($this->hasItems(["positionRow" => $i, "hitpoints>" => 0, ])) {
132 132
         return $i;
133 133
       }
134 134
     }
Please login to merge, or discard this patch.
Upper-Lower-Casing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -80,11 +80,11 @@  discard block
 block discarded – undo
80 80
    */
81 81
   public function setCharacterPosition($id, int $row, int $column): void {
82 82
     if(!$this->hasItems(["id" => $id])) {
83
-      throw new \OutOfBoundsException("Character $id is not in the team");
83
+      throw new \OutOfBoundsException("character $id is not in the team");
84 84
     } elseif(count($this->getItems(["positionRow" => $row])) >= $this->maxRowSize) {
85
-      throw new InvalidCharacterPositionException("Row $row is full.", InvalidCharacterPositionException::ROW_FULL);
85
+      throw new InvalidCharacterPositionException("row $row is full.", InvalidCharacterPositionException::ROW_FULL);
86 86
     } elseif($this->hasItems(["positionRow" => $row, "positionColumn" => $column])) {
87
-      throw new InvalidCharacterPositionException("Row $row column $column is occupied.", InvalidCharacterPositionException::POSITION_OCCUPIED);
87
+      throw new InvalidCharacterPositionException("row $row column $column is occupied.", InvalidCharacterPositionException::POSITION_OCCUPIED);
88 88
     }
89 89
     $character = $this->getItems(["id" => $id])[0];
90 90
     $character->positionRow = $row;
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
       return null;
113 113
     }
114 114
     foreach($this->aliveMembers as $index => $member) {
115
-      if($member->hitpoints <= $member->maxHitpoints * $threshold AND $member->hitpoints < $lowestHp) {
115
+      if($member->hitpoints <= $member->maxHitpoints * $threshold and $member->hitpoints < $lowestHp) {
116 116
         $lowestHp = $member->hitpoints;
117 117
         $lowestIndex = $index;
118 118
       }
Please login to merge, or discard this patch.
src/Equipment.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
   }
53 53
   
54 54
   protected function configureOptions(OptionsResolver $resolver): void {
55
-    $allStats = ["id", "name", "slot", "type", "strength", "worn",];
55
+    $allStats = ["id", "name", "slot", "type", "strength", "worn", ];
56 56
     $resolver->setRequired($allStats);
57 57
     $resolver->setAllowedTypes("id", "integer");
58 58
     $resolver->setAllowedTypes("name", "string");
Please login to merge, or discard this patch.
src/CombatBase.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     $this->onCombatEnd[] = [$this, "logCombatResult"];
83 83
     $this->onCombatEnd[] = [$this, "resetInitiative"];
84 84
     $this->onRoundStart[] = [$this, "decreaseEffectsDuration"];
85
-    $this->onRoundStart[] = [$this ,"recalculateStats"];
85
+    $this->onRoundStart[] = [$this, "recalculateStats"];
86 86
     $this->onRoundStart[] = [$this, "logRoundNumber"];
87 87
     $this->onRoundStart[] = [$this, "applyPoison"];
88 88
     $this->onRound[] = [$this, "mainStage"];
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
         return null;
364 364
       }
365 365
       /** @var Team $enemies */
366
-      $enemies = Team::fromArray($enemyTeam->getItems(["positionRow" => $rowToAttack, "hitpoints>" => 0,]), $enemyTeam->name);
366
+      $enemies = Team::fromArray($enemyTeam->getItems(["positionRow" => $rowToAttack, "hitpoints>" => 0, ]), $enemyTeam->name);
367 367
     } else {
368 368
       $enemies = $enemyTeam;
369 369
     }
Please login to merge, or discard this patch.
Upper-Lower-Casing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
       foreach($team as $character) {
238 238
         try {
239 239
           $column++;
240
-          if($character->positionRow > 0 AND $character->positionColumn > 0) {
240
+          if($character->positionRow > 0 and $character->positionColumn > 0) {
241 241
             continue;
242 242
           }
243 243
           setPosition:
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
     $characters = array_merge($combat->team1->toArray(), $combat->team2->toArray());
291 291
     foreach($characters as $character) {
292 292
       foreach($character->effects as $effect) {
293
-        if($effect->duration === CharacterEffect::DURATION_COMBAT OR is_int($effect->duration)) {
293
+        if($effect->duration === CharacterEffect::DURATION_COMBAT or is_int($effect->duration)) {
294 294
           $character->removeEffect($effect->id);
295 295
         }
296 296
       }
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
     $enemyTeam = $this->getEnemyTeam($attacker);
353 353
     $rangedWeapon = false;
354 354
     foreach($attacker->equipment as $equipment) {
355
-      if($equipment instanceof Weapon AND $equipment->isWorn() AND $equipment->ranged) {
355
+      if($equipment instanceof Weapon and $equipment->isWorn() and $equipment->ranged) {
356 356
         $rangedWeapon = true;
357 357
         break;
358 358
       }
@@ -412,7 +412,7 @@  discard block
 block discarded – undo
412 412
         $targets = $this->getTeam($primaryTarget)->getItems(["positionColumn" => $primaryTarget->positionColumn]);
413 413
         break;
414 414
       default:
415
-        throw new NotImplementedException("Target $skill->skill->target for attack skills is not implemented.");
415
+        throw new NotImplementedException("target $skill->skill->target for attack skills is not implemented.");
416 416
     }
417 417
     foreach($targets as $target) {
418 418
       for($i = 1; $i <= $skill->skill->strikes; $i++) {
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
         $targets = $this->getEnemyTeam($character)->toArray();
441 441
         break;
442 442
       default:
443
-        throw new NotImplementedException("Target $skill->skill->target for special skills is not implemented.");
443
+        throw new NotImplementedException("target $skill->skill->target for special skills is not implemented.");
444 444
     }
445 445
     foreach($targets as $target) {
446 446
       $this->onSkillSpecial($character, $target, $skill);
@@ -478,7 +478,7 @@  discard block
 block discarded – undo
478 478
           $combat->onHeal($character, $combat->selectHealingTarget($character));
479 479
           break;
480 480
         default:
481
-          throw new NotImplementedException("Action $action is not implemented.");
481
+          throw new NotImplementedException("action $action is not implemented.");
482 482
       }
483 483
     }
484 484
   }
@@ -589,7 +589,7 @@  discard block
 block discarded – undo
589 589
       "character1" => $character1, "character2" => $target,
590 590
     ];
591 591
     $effect = new CharacterEffect([
592
-      "id" => "skill{$skill->skill->id}Effect",
592
+      "id" => "skill{$skill->skill->id}effect",
593 593
       "type" => $skill->skill->type,
594 594
       "stat" => ((in_array($skill->skill->type, SkillSpecial::NO_STAT_TYPES, true)) ? null : $skill->skill->stat),
595 595
       "value" => $skill->value,
Please login to merge, or discard this patch.