diff --git a/src/main/java/forge/card/spellability/SpellAbility.java b/src/main/java/forge/card/spellability/SpellAbility.java index d5cea13787a..09db36ef58d 100644 --- a/src/main/java/forge/card/spellability/SpellAbility.java +++ b/src/main/java/forge/card/spellability/SpellAbility.java @@ -69,7 +69,7 @@ public abstract class SpellAbility { private Card originalHost = null; private List splicedCards = null; - private List targetList; +// private List targetList; // targetList doesn't appear to be used anymore private boolean spell; @@ -1288,49 +1288,49 @@ public abstract class SpellAbility { this.setStackDescription(desc); } - /** - *

- * Getter for the field targetList. - *

- * - * @return a {@link forge.CardList} object. - */ - public List getTargetList() { - return this.targetList; - } +// /** +// *

+// * Getter for the field targetList. +// *

+// * +// * @return a {@link forge.CardList} object. +// */ +// public List getTargetList() { +// return this.targetList; +// } - /** - *

- * Setter for the field targetList. - *

- * - * @param list - * a {@link forge.CardList} object. - */ - public void setTargetList(final List list) { - // The line below started to create a null error at - // forge.CardFactoryUtil.canBeTargetedBy(CardFactoryUtil.java:3329) - // after ForgeSVN r2699. I hope that commenting out the line below will - // not result in other bugs. :) - // targetPlayer = null;//reset setTargetPlayer() - - this.targetList = list; - final StringBuilder sb = new StringBuilder(); - sb.append(this.getSourceCard().getName()).append(" - targeting "); - for (int i = 0; i < this.targetList.size(); i++) { - - if (!this.targetList.get(i).isFaceDown()) { - sb.append(this.targetList.get(i)); - } else { - sb.append("Morph(").append(this.targetList.get(i).getUniqueNumber()).append(")"); - } - - if (i < (this.targetList.size() - 1)) { - sb.append(", "); - } - } - this.setStackDescription(sb.toString()); - } +// /** +// *

+// * Setter for the field targetList. +// *

+// * +// * @param list +// * a {@link forge.CardList} object. +// */ +// public void setTargetList(final List list) { +// // The line below started to create a null error at +// // forge.CardFactoryUtil.canBeTargetedBy(CardFactoryUtil.java:3329) +// // after ForgeSVN r2699. I hope that commenting out the line below will +// // not result in other bugs. :) +// // targetPlayer = null;//reset setTargetPlayer() +// +// this.targetList = list; +// final StringBuilder sb = new StringBuilder(); +// sb.append(this.getSourceCard().getName()).append(" - targeting "); +// for (int i = 0; i < this.targetList.size(); i++) { +// +// if (!this.targetList.get(i).isFaceDown()) { +// sb.append(this.targetList.get(i)); +// } else { +// sb.append("Morph(").append(this.targetList.get(i).getUniqueNumber()).append(")"); +// } +// +// if (i < (this.targetList.size() - 1)) { +// sb.append(", "); +// } +// } +// this.setStackDescription(sb.toString()); +// } /** *

diff --git a/src/main/java/forge/card/trigger/TriggerHandler.java b/src/main/java/forge/card/trigger/TriggerHandler.java index 9b8a98ca706..62472d1b94f 100644 --- a/src/main/java/forge/card/trigger/TriggerHandler.java +++ b/src/main/java/forge/card/trigger/TriggerHandler.java @@ -698,10 +698,10 @@ public class TriggerHandler { return sa[0].getTargetCard(); } - @Override - public List getTargetList() { - return sa[0].getTargetList(); - } +// @Override +// public List getTargetList() { +// return sa[0].getTargetList(); +// } @Override public Player getTargetPlayer() { @@ -889,10 +889,10 @@ public class TriggerHandler { sa[0].setTargetCard(card); } - @Override - public void setTargetList(final List list) { - sa[0].setTargetList(list); - } +// @Override +// public void setTargetList(final List list) { +// sa[0].setTargetList(list); +// } @Override public void setTargetPlayer(final Player p) { diff --git a/src/main/java/forge/card/trigger/TriggerSpellAbilityCast.java b/src/main/java/forge/card/trigger/TriggerSpellAbilityCast.java index b65054106df..54118aba725 100644 --- a/src/main/java/forge/card/trigger/TriggerSpellAbilityCast.java +++ b/src/main/java/forge/card/trigger/TriggerSpellAbilityCast.java @@ -100,25 +100,11 @@ public class TriggerSpellAbilityCast extends Trigger { final SpellAbility sa = si.getSpellAbility(); if (sa.getTarget() == null) { if (sa.getTargetCard() == null) { - if (sa.getTargetList() == null) { - if (sa.getTargetPlayer() == null) { - return false; - } else { - if (!matchesValid(sa.getTargetPlayer(), - this.getMapParams().get("TargetsValid").split(","), this.getHostCard())) { - return false; - } - } + if (sa.getTargetPlayer() == null) { + return false; } else { - boolean validTgtFound = false; - for (final Card tgt : sa.getTargetList()) { - if (matchesValid(tgt, this.getMapParams().get("TargetsValid").split(","), - this.getHostCard())) { - validTgtFound = true; - break; - } - } - if (!validTgtFound) { + if (!matchesValid(sa.getTargetPlayer(), + this.getMapParams().get("TargetsValid").split(","), this.getHostCard())) { return false; } } diff --git a/src/main/java/forge/game/zone/MagicStack.java b/src/main/java/forge/game/zone/MagicStack.java index 528267f2e80..d23e33899bb 100644 --- a/src/main/java/forge/game/zone/MagicStack.java +++ b/src/main/java/forge/game/zone/MagicStack.java @@ -766,12 +766,6 @@ public class MagicStack extends MyObservable { runParams.put("Target", sp.getTargetCard()); game.getTriggerHandler().runTrigger(TriggerType.BecomesTarget, runParams); - } else if ((sp.getTargetList() != null) && (sp.getTargetList().size() > 0)) { - for (final Card ctgt : sp.getTargetList()) { - runParams.put("Target", ctgt); - - game.getTriggerHandler().runTrigger(TriggerType.BecomesTarget, runParams); - } } else if (sp.getTargetPlayer() != null) { runParams.put("Target", sp.getTargetPlayer());