diff --git a/src/main/java/forge/card/ability/AbilityFactory.java b/src/main/java/forge/card/ability/AbilityFactory.java
index 9fb66b4c33e..0870a394652 100644
--- a/src/main/java/forge/card/ability/AbilityFactory.java
+++ b/src/main/java/forge/card/ability/AbilityFactory.java
@@ -29,6 +29,7 @@ import forge.card.spellability.SpellAbilityCondition;
import forge.card.spellability.SpellAbilityRestriction;
import forge.card.spellability.Target;
import forge.game.zone.ZoneType;
+import forge.util.FileSection;
/**
*
@@ -61,7 +62,7 @@ public final class AbilityFactory {
Map mapParams;
try {
- mapParams = AbilityUtils.getMapParams(abString);
+ mapParams = AbilityFactory.getMapParams(abString);
}
catch (RuntimeException ex) {
throw new RuntimeException(hostCard.getName() + ": " + ex.getMessage());
@@ -304,4 +305,8 @@ public final class AbilityFactory {
return null;
}
+ public static final Map getMapParams(final String abString) {
+ return FileSection.parseToMap(abString, "$", "|");
+ }
+
} // end class AbilityFactory
diff --git a/src/main/java/forge/card/ability/AbilityUtils.java b/src/main/java/forge/card/ability/AbilityUtils.java
index 8eb22821755..fc5f64da904 100644
--- a/src/main/java/forge/card/ability/AbilityUtils.java
+++ b/src/main/java/forge/card/ability/AbilityUtils.java
@@ -28,7 +28,6 @@ import forge.game.ai.ComputerUtilCost;
import forge.game.player.AIPlayer;
import forge.game.player.Player;
import forge.game.zone.ZoneType;
-import forge.util.FileSection;
/**
* TODO: Write javadoc for this type.
@@ -950,10 +949,6 @@ public class AbilityUtils {
return sas;
}
- public static final Map getMapParams(final String abString) {
- return FileSection.parseToMap(abString, "$", "|");
- }
-
public static ArrayList getProtectionList(final SpellAbility sa) {
final ArrayList gains = new ArrayList();
diff --git a/src/main/java/forge/card/ability/effects/ChooseGenericEffect.java b/src/main/java/forge/card/ability/effects/ChooseGenericEffect.java
index a537dc079b5..cb9caaaa632 100644
--- a/src/main/java/forge/card/ability/effects/ChooseGenericEffect.java
+++ b/src/main/java/forge/card/ability/effects/ChooseGenericEffect.java
@@ -36,7 +36,7 @@ public class ChooseGenericEffect extends SpellEffect {
final Card host = sa.getSourceCard();
final BiMap choices = HashBiMap.create();
for (String s : Arrays.asList(sa.getParam("Choices").split(","))) {
- final Map theseParams = AbilityUtils.getMapParams(host.getSVar(s));
+ final Map theseParams = AbilityFactory.getMapParams(host.getSVar(s));
choices.put(s, theseParams.get("ChoiceDescription"));
}
diff --git a/src/main/java/forge/game/ai/ComputerUtilCombat.java b/src/main/java/forge/game/ai/ComputerUtilCombat.java
index 98addfa1f25..b1a4d1e0b30 100644
--- a/src/main/java/forge/game/ai/ComputerUtilCombat.java
+++ b/src/main/java/forge/game/ai/ComputerUtilCombat.java
@@ -29,6 +29,7 @@ import forge.CardLists;
import forge.CounterType;
import forge.Singletons;
import forge.card.TriggerReplacementBase;
+import forge.card.ability.AbilityFactory;
import forge.card.ability.AbilityUtils;
import forge.card.ability.ApiType;
import forge.card.cardfactory.CardFactoryUtil;
@@ -741,7 +742,7 @@ public class ComputerUtilCombat {
continue;
}
final String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
if (abilityParams.containsKey("AB") && !abilityParams.get("AB").equals("Pump")) {
continue;
}
@@ -837,7 +838,7 @@ public class ComputerUtilCombat {
continue;
}
final String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
// DealDamage triggers
if ((abilityParams.containsKey("AB") && abilityParams.get("AB").equals("DealDamage"))
@@ -993,7 +994,7 @@ public class ComputerUtilCombat {
continue;
}
final String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
if (abilityParams.containsKey("ValidTgts") || abilityParams.containsKey("Tgt")) {
continue; // targeted pumping not supported
}
@@ -1141,7 +1142,7 @@ public class ComputerUtilCombat {
continue;
}
final String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
if (abilityParams.containsKey("ValidTgts") || abilityParams.containsKey("Tgt")) {
continue; // targeted pumping not supported
}
@@ -1266,7 +1267,7 @@ public class ComputerUtilCombat {
continue;
}
String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
// Destroy triggers
if ((abilityParams.containsKey("AB") && abilityParams.get("AB").equals("Destroy"))
|| (abilityParams.containsKey("DB") && abilityParams.get("DB").equals("Destroy"))) {
@@ -1321,7 +1322,7 @@ public class ComputerUtilCombat {
continue;
}
String ability = source.getSVar(trigParams.get("Execute"));
- final Map abilityParams = AbilityUtils.getMapParams(ability);
+ final Map abilityParams = AbilityFactory.getMapParams(ability);
// Destroy triggers
if ((abilityParams.containsKey("AB") && abilityParams.get("AB").equals("Destroy"))
|| (abilityParams.containsKey("DB") && abilityParams.get("DB").equals("Destroy"))) {