heroesofabenez / combat

src/TextCombatLogRender.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
    */
   public function setTemplate(string $template): void {
     if(!is_file($template)) {
-      throw new \RuntimeException("File $template does not exist.");
+      throw new \RuntimeException("file $template does not exist.");
     }
     $this->template = $template;
   }

src/BaseSkill.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
   protected $levels;
   
   protected function configureOptions(OptionsResolver $resolver): void {
-    $resolver->setRequired(["id", "name", "target", "levels",]);
+    $resolver->setRequired(["id", "name", "target", "levels", ]);
     $resolver->setAllowedTypes("id", "int");
     $resolver->setAllowedTypes("name", "string");
     $resolver->setAllowedTypes("target", "string");

src/CombatActions/SkillSpecial.php

Braces +3 added lines, -1 removed lines

@@ -52,9 +52,11 @@
   /**
    * @throws NotImplementedException
    */
-  public function do(CombatBase $combat, Character $character): void {
+  public function do {
+    (CombatBase $combat, Character $character): void {
     /** @var CharacterSpecialSkill $skill */
     $skill = $character->usableSkills[0];
+  }
     $targets = [];
     switch($skill->skill->target) {
       case Skill::TARGET_ENEMY:

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
       "character1" => $character1, "character2" => $target,
     ];
     $effect = new CharacterEffect([
-      "id" => "skill{$skill->skill->id}Effect",
+      "id" => "skill{$skill->skill->id}effect",
       "type" => $skill->skill->type,
       "stat" => ((in_array($skill->skill->type, Skill::NO_STAT_TYPES, true)) ? null : $skill->skill->stat),
       "value" => $skill->value,
@@ -70,7 +70,7 @@ 
         $targets = $combat->getEnemyTeam($character)->toArray();
         break;
       default:
-        throw new NotImplementedException("Target {$skill->skill->target} for special skills is not implemented.");
+        throw new NotImplementedException("target {$skill->skill->target} for special skills is not implemented.");
     }
     foreach($targets as $target) {
       $this->doSingleTarget($character, $target, $skill, $combat);

src/CombatActions/Heal.php

Braces +3 added lines, -1 removed lines

@@ -21,8 +21,10 @@
     return (in_array($character, $combat->findHealers()->toArray(), true) AND !is_null($combat->selectHealingTarget($character)));
   }
 
-  public function do(CombatBase $combat, Character $character): void {
+  public function do {
+    (CombatBase $combat, Character $character): void {
     $result = [];
+  }
     /** @var Character $patient */
     $patient = $combat->selectHealingTarget($character);
     $result["result"] = $combat->successCalculator->hasHealed($character);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
   }
 
   public function shouldUse(CombatBase $combat, Character $character): bool {
-    return (in_array($character, $combat->findHealers()->toArray(), true) AND !is_null($combat->selectHealingTarget($character)));
+    return (in_array($character, $combat->findHealers()->toArray(), true) and !is_null($combat->selectHealingTarget($character)));
   }
 
   public function do(CombatBase $combat, Character $character): void {

src/CombatActions/SkillAttack.php

Braces +3 added lines, -1 removed lines

@@ -54,9 +54,11 @@
   /**
    * @throws NotImplementedException
    */
-  public function do(CombatBase $combat, Character $character): void {
+  public function do {
+    (CombatBase $combat, Character $character): void {
     /** @var CharacterAttackSkill $skill */
     $skill = $character->usableSkills[0];
+  }
     $targets = [];
     /** @var Character $primaryTarget */
     $primaryTarget = $combat->selectAttackTarget($character);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         $targets = $combat->getTeam($primaryTarget)->getItems(["positionColumn" => $primaryTarget->positionColumn]);
         break;
       default:
-        throw new NotImplementedException("Target {$skill->skill->target} for attack skills is not implemented.");
+        throw new NotImplementedException("target {$skill->skill->target} for attack skills is not implemented.");
     }
     foreach($targets as $target) {
       for($i = 1; $i <= $skill->skill->strikes; $i++) {

src/CombatActions/ICombatAction.php

Braces +4 added lines, -2 removed lines

@@ -10,6 +10,8 @@
   public function getName(): string;
   public function getPriority(): int;
   public function shouldUse(CombatBase $combat, Character $character): bool;
-  public function do(CombatBase $combat, Character $character): void;
-}
+  public function do {
+    (CombatBase $combat, Character $character): void;
+  }
+  }
 ?>
\ No newline at end of file

src/CombatActions/Attack.php

Braces +3 added lines, -1 removed lines

@@ -26,8 +26,10 @@
    * Hit chance = Attacker's hit - Defender's dodge, but at least 15%
    * Damage = Attacker's damage - defender's defense
    */
-  public function do(CombatBase $combat, Character $character): void {
+  public function do {
+    (CombatBase $combat, Character $character): void {
     $result = [];
+  }
     /** @var Character $defender */
     $defender = $combat->selectAttackTarget($character);
     $result["result"] = $combat->successCalculator->hasHit($character, $defender);

src/CombatLogEntry.php

Spacing +2 added lines, -2 removed lines

@@ -56,8 +56,8 @@
   }
 
   protected function configureOptions(OptionsResolver $resolver): void {
-    $requiredStats = ["action", "result", "character1", "character2",];
-    $resolver->setDefined(["amount", "name",]);
+    $requiredStats = ["action", "result", "character1", "character2", ];
+    $resolver->setDefined(["amount", "name", ]);
     $resolver->setRequired($requiredStats);
     $resolver->setAllowedTypes("action", "string");
     $resolver->setAllowedValues("action", function(string $value) {