diff --git a/src/main/java/forge/Card.java b/src/main/java/forge/Card.java index 0556d794afe..134c200a890 100644 --- a/src/main/java/forge/Card.java +++ b/src/main/java/forge/Card.java @@ -4862,29 +4862,7 @@ public class Card extends GameEntity implements Comparable { || ((isCreature() || isTribal()) && CardUtil.isACreatureType(cardType) && typeContains("AllCreatureTypes"))) return true; return false; - } - - - /** - *

isValidCard.

- * Takes an array of arguments like Permanent.Blue+withFlying, only one of them has to be true - * - * @param Restrictions an array of {@link java.lang.String} objects. - * @param sourceController a {@link forge.Player} object. - * @param source a {@link forge.Card} object. - * @return a boolean. - */ - public boolean isValidCard(final String Restrictions[], final Player sourceController, final Card source) { - - if (getName().equals("Mana Pool") || isImmutable()) return false; - - for (int i = 0; i < Restrictions.length; i++) { - if (isValid(Restrictions[i], sourceController, source)) return true; - } - return false; - - }//isValidCard - + }//isType // Takes one argument like Permanent.Blue+withFlying /** @@ -4920,7 +4898,7 @@ public class Card extends GameEntity implements Comparable { if (hasProperty(exR[j], sourceController, source) == false) return false; } return true; - }//isValidCard(String Restriction) + }//isValid(String Restriction) // Takes arguments like Blue or withFlying /** diff --git a/src/main/java/forge/CardList.java b/src/main/java/forge/CardList.java index 5e29e59445e..b9d6e04bf18 100644 --- a/src/main/java/forge/CardList.java +++ b/src/main/java/forge/CardList.java @@ -675,7 +675,7 @@ public class CardList implements Iterable { public final CardList getValidCards(final String[] restrictions, final Player sourceController, final Card source) { return this.filter(new CardListFilter() { public boolean addCard(final Card c) { - return c != null && c.isValidCard(restrictions, sourceController, source); + return c != null && c.isValid(restrictions, sourceController, source); } }); } diff --git a/src/main/java/forge/CardUtil.java b/src/main/java/forge/CardUtil.java index 39b5ac43feb..dfef5d310b1 100644 --- a/src/main/java/forge/CardUtil.java +++ b/src/main/java/forge/CardUtil.java @@ -707,7 +707,7 @@ public final class CardUtil { } for(Card c : AllZoneUtil.getColorInPlay(strCol)) { - if(!res.contains(c) && c.isValidCard(valid, source.getController(), source) && !c.equals(origin)) { + if(!res.contains(c) && c.isValid(valid, source.getController(), source) && !c.equals(origin)) { res.add(c); } } diff --git a/src/main/java/forge/CombatUtil.java b/src/main/java/forge/CombatUtil.java index 5209de09954..649aefc669b 100644 --- a/src/main/java/forge/CombatUtil.java +++ b/src/main/java/forge/CombatUtil.java @@ -372,7 +372,7 @@ public class CombatUtil { String parse = attacker.getKeyword().get(KeywordPosition).toString(); String k[] = parse.split(" ", 2); final String restrictions[] = k[1].split(","); - if (blocker.isValidCard(restrictions, attacker.getController(), attacker)) + if (blocker.isValid(restrictions, attacker.getController(), attacker)) return false; } @@ -381,7 +381,7 @@ public class CombatUtil { String parse = blocker.getKeyword().get(KeywordPosition).toString(); String k[] = parse.split(" ", 2); final String restrictions[] = k[1].split(","); - if (attacker.isValidCard(restrictions, blocker.getController(), blocker)) + if (attacker.isValid(restrictions, blocker.getController(), blocker)) return false; } @@ -1187,8 +1187,8 @@ public class CombatUtil { if (abilityParams.containsKey("Defined") && abilityParams.get("Defined").equals("TriggeredAttacker")) list.add(attacker); if (abilityParams.containsKey("ValidCards")) - if (attacker.isValidCard(abilityParams.get("ValidCards").split(","), source.getController(), source) - || attacker.isValidCard(abilityParams.get("ValidCards").replace("attacking+", "").split(",") + if (attacker.isValid(abilityParams.get("ValidCards").split(","), source.getController(), source) + || attacker.isValid(abilityParams.get("ValidCards").replace("attacking+", "").split(",") , source.getController(), source)) list.add(attacker); if (list.isEmpty()) continue; @@ -1265,8 +1265,8 @@ public class CombatUtil { if (abilityParams.containsKey("Defined") && abilityParams.get("Defined").equals("TriggeredAttacker")) list.add(attacker); if (abilityParams.containsKey("ValidCards")) - if (attacker.isValidCard(abilityParams.get("ValidCards").split(","), source.getController(), source) - || attacker.isValidCard(abilityParams.get("ValidCards").replace("attacking+", "").split(",") + if (attacker.isValid(abilityParams.get("ValidCards").split(","), source.getController(), source) + || attacker.isValid(abilityParams.get("ValidCards").replace("attacking+", "").split(",") , source.getController(), source)) list.add(attacker); if (list.isEmpty()) continue; diff --git a/src/main/java/forge/ComputerAI_General.java b/src/main/java/forge/ComputerAI_General.java index 13d1fdb16f4..a8cbd63318e 100644 --- a/src/main/java/forge/ComputerAI_General.java +++ b/src/main/java/forge/ComputerAI_General.java @@ -101,7 +101,7 @@ public class ComputerAI_General implements Computer { if (buffedcard.getSVar("BuffedBy").length() > 0) { String buffedby = buffedcard.getSVar("BuffedBy"); String[] bffdby = buffedby.split(","); - if (c.isValidCard(bffdby, c.getController(), c)) { + if (c.isValid(bffdby, c.getController(), c)) { return true; } } @@ -114,7 +114,7 @@ public class ComputerAI_General implements Computer { if (buffedcard.getSVar("AntiBuffedBy").length() > 0) { String buffedby = buffedcard.getSVar("AntiBuffedBy"); String[] bffdby = buffedby.split(","); - if (c.isValidCard(bffdby, c.getController(), c)) { + if (c.isValid(bffdby, c.getController(), c)) { return true; } } diff --git a/src/main/java/forge/GameAction.java b/src/main/java/forge/GameAction.java index 48879a22a82..38d75831fe3 100644 --- a/src/main/java/forge/GameAction.java +++ b/src/main/java/forge/GameAction.java @@ -745,13 +745,13 @@ public class GameAction { if (entity instanceof Card){ Card perm = (Card)entity; - // I think the Keyword checks might be superfluous with the isValidCard check + // I think the Keyword checks might be superfluous with the isValid check if (!AllZoneUtil.isCardInPlay(perm) || CardFactoryUtil.hasProtectionFrom(c, perm) || ((c.hasKeyword("Enchant creature") || c.hasKeyword("Enchant tapped creature")) && !perm.isCreature()) || (c.hasKeyword("Enchant tapped creature") && perm.isUntapped()) - || (tgt != null && !perm.isValidCard(tgt.getValidTgts(), c.getController(), c))){ + || (tgt != null && !perm.isValid(tgt.getValidTgts(), c.getController(), c))){ c.unEnchantEntity(perm); moveToGraveyard(c); checkAgain = true; diff --git a/src/main/java/forge/GameEntity.java b/src/main/java/forge/GameEntity.java index a0a5918afb3..dc58a9d7ba6 100644 --- a/src/main/java/forge/GameEntity.java +++ b/src/main/java/forge/GameEntity.java @@ -222,7 +222,7 @@ public abstract class GameEntity extends MyObservable { } return false; - }//isValidCard + }//isValid public boolean isValid(final String Restriction, final Player sourceController, final Card source) { return false; diff --git a/src/main/java/forge/PhaseUtil.java b/src/main/java/forge/PhaseUtil.java index 9935c0b01d1..fec4ba6efd9 100644 --- a/src/main/java/forge/PhaseUtil.java +++ b/src/main/java/forge/PhaseUtil.java @@ -281,7 +281,7 @@ public class PhaseUtil { String k[] = parse.split(":"); final String restrictions[] = k[1].split(","); final Card card = ca; - if (c.isValidCard(restrictions, card.getController(), card)) return false; + if (c.isValid(restrictions, card.getController(), card)) return false; } } // end of Permanents don't untap during their controllers' untap steps diff --git a/src/main/java/forge/Upkeep.java b/src/main/java/forge/Upkeep.java index 2ac427baeba..3eab8d642a1 100644 --- a/src/main/java/forge/Upkeep.java +++ b/src/main/java/forge/Upkeep.java @@ -900,7 +900,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -980,7 +980,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1075,7 +1075,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1169,7 +1169,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1260,7 +1260,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1347,7 +1347,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1433,7 +1433,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1518,7 +1518,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1597,7 +1597,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1675,7 +1675,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1773,7 +1773,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); @@ -1851,7 +1851,7 @@ public class Upkeep implements java.io.Serializable { // We do not want to slow down the pace of the game by asking too many questions. // Dialogs outside of the Ability appear at the previous end of turn phase !!! - if (peek[0].isValidCard(shareTypes, k.getController(), k)) { + if (peek[0].isValid(shareTypes, k.getController(), k)) { if (player.isHuman()) { StringBuilder question = new StringBuilder(); question.append("Your top card is ").append(peek[0].getName()); diff --git a/src/main/java/forge/card/cardFactory/CardFactoryUtil.java b/src/main/java/forge/card/cardFactory/CardFactoryUtil.java index 80a27bfddc6..f8c4cfce3be 100644 --- a/src/main/java/forge/card/cardFactory/CardFactoryUtil.java +++ b/src/main/java/forge/card/cardFactory/CardFactoryUtil.java @@ -2326,7 +2326,7 @@ public class CardFactoryUtil { // Reconfirm the Validity of a TgtValid, or if the Creature is still // a Creature if (tgt.doesTarget() - && !target.isValidCard(tgt.getValidTgts(), ability.getActivatingPlayer(), ability.getSourceCard())) + && !target.isValid(tgt.getValidTgts(), ability.getActivatingPlayer(), ability.getSourceCard())) { return false; } @@ -2551,10 +2551,10 @@ public class CardFactoryUtil { return true; } - if (kw.startsWith("Protection:")) { // uses isValidCard + if (kw.startsWith("Protection:")) { // uses isValid String characteristic = kw.split(":")[1]; String[] characteristics = characteristic.split(","); - if (card.isValidCard(characteristics, card.getController(), card)) { + if (card.isValid(characteristics, card.getController(), card)) { return true; } } @@ -3102,7 +3102,7 @@ public class CardFactoryUtil { cl = cl.filter(new CardListFilter() { public boolean addCard(final Card cdev) { - return cdev.isValidCard(validDevoured.split(","), csource.getController(), csource); + return cdev.isValid(validDevoured.split(","), csource.getController(), csource); } }); @@ -3750,7 +3750,7 @@ public class CardFactoryUtil { String[] k = parse.split(":"); String[] restrictions = k[1].split(","); - if (!c.isValidCard(restrictions, card.getController(), card)) { + if (!c.isValid(restrictions, card.getController(), card)) { continue; } diff --git a/src/main/java/forge/card/spellability/Target.java b/src/main/java/forge/card/spellability/Target.java index 78bf8e49b0b..f778d7c2023 100644 --- a/src/main/java/forge/card/spellability/Target.java +++ b/src/main/java/forge/card/spellability/Target.java @@ -485,7 +485,7 @@ public class Target { } for (Card c : AllZoneUtil.getCardsIn(tgtZone)){ - if (c.isValidCard(ValidTgts, srcCard.getController(), srcCard) && (!isTargeted || CardFactoryUtil.canTarget(srcCard, c))) { + if (c.isValid(ValidTgts, srcCard.getController(), srcCard) && (!isTargeted || CardFactoryUtil.canTarget(srcCard, c))) { return true; } } diff --git a/src/main/java/forge/card/spellability/Target_Selection.java b/src/main/java/forge/card/spellability/Target_Selection.java index 832575050a3..f705a6ec696 100644 --- a/src/main/java/forge/card/spellability/Target_Selection.java +++ b/src/main/java/forge/card/spellability/Target_Selection.java @@ -465,7 +465,7 @@ public class Target_Selection { Player activatingPlayer = sa.getActivatingPlayer(); if (o instanceof Card) { Card c = (Card) o; - return c.isValidCard(valids, activatingPlayer, srcCard); + return c.isValid(valids, activatingPlayer, srcCard); } if (o instanceof Player) { @@ -485,7 +485,7 @@ public class Target_Selection { if (o instanceof SpellAbility) { Card c = ((SpellAbility) o).getSourceCard(); - return c.isValidCard(valids, activatingPlayer, srcCard); + return c.isValid(valids, activatingPlayer, srcCard); } return false; diff --git a/src/main/java/forge/card/staticAbility/StaticAbility.java b/src/main/java/forge/card/staticAbility/StaticAbility.java index 458478c85f9..9bea1b35f6f 100644 --- a/src/main/java/forge/card/staticAbility/StaticAbility.java +++ b/src/main/java/forge/card/staticAbility/StaticAbility.java @@ -196,7 +196,7 @@ public class StaticAbility { if (mapParams.containsKey("TopCardOfLibraryIs")) { Card topCard = controller.getCardsIn(Zone.Library).get(0); - if (!topCard.isValidCard(mapParams.get("TopCardOfLibraryIs").split(","), controller, hostCard)) + if (!topCard.isValid(mapParams.get("TopCardOfLibraryIs").split(","), controller, hostCard)) return false; } diff --git a/src/main/java/forge/card/trigger/Trigger.java b/src/main/java/forge/card/trigger/Trigger.java index ce2670e9c99..d16adeaef53 100644 --- a/src/main/java/forge/card/trigger/Trigger.java +++ b/src/main/java/forge/card/trigger/Trigger.java @@ -458,7 +458,7 @@ public abstract class Trigger { public boolean matchesValid(Object o, String[] valids, Card srcCard) { if (o instanceof Card) { Card c = (Card) o; - return c.isValidCard(valids, srcCard.getController(), srcCard); + return c.isValid(valids, srcCard.getController(), srcCard); } if (o instanceof Player) { diff --git a/src/main/java/forge/card/trigger/Trigger_Championed.java b/src/main/java/forge/card/trigger/Trigger_Championed.java index 221de890db5..006ead47226 100644 --- a/src/main/java/forge/card/trigger/Trigger_Championed.java +++ b/src/main/java/forge/card/trigger/Trigger_Championed.java @@ -30,7 +30,7 @@ public class Trigger_Championed extends Trigger { Card championed = (Card) runParams2.get("Championed"); if (mapParams.containsKey("ValidCard")) { - if (!championed.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!championed.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_ChangesZone.java b/src/main/java/forge/card/trigger/Trigger_ChangesZone.java index 4772dd354d6..acb0be40869 100644 --- a/src/main/java/forge/card/trigger/Trigger_ChangesZone.java +++ b/src/main/java/forge/card/trigger/Trigger_ChangesZone.java @@ -47,7 +47,7 @@ public class Trigger_ChangesZone extends Trigger { if (mapParams.containsKey("ValidCard")) { Card moved = (Card) runParams2.get("Card"); - if (!moved.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!moved.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_CounterAdded.java b/src/main/java/forge/card/trigger/Trigger_CounterAdded.java index 84becf10ba1..8dc42c12974 100644 --- a/src/main/java/forge/card/trigger/Trigger_CounterAdded.java +++ b/src/main/java/forge/card/trigger/Trigger_CounterAdded.java @@ -31,7 +31,7 @@ public class Trigger_CounterAdded extends Trigger { Counters addedType = (Counters) runParams2.get("CounterType"); if (mapParams.containsKey("ValidCard")) { - if (!addedTo.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!addedTo.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_DamageDone.java b/src/main/java/forge/card/trigger/Trigger_DamageDone.java index 75e2a67a82b..545961ef0fd 100644 --- a/src/main/java/forge/card/trigger/Trigger_DamageDone.java +++ b/src/main/java/forge/card/trigger/Trigger_DamageDone.java @@ -31,7 +31,7 @@ public class Trigger_DamageDone extends Trigger { Object tgt = runParams2.get("DamageTarget"); if (mapParams.containsKey("ValidSource")) { - if (!src.isValidCard(mapParams.get("ValidSource").split(","), hostCard.getController(), hostCard)) { + if (!src.isValid(mapParams.get("ValidSource").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_Drawn.java b/src/main/java/forge/card/trigger/Trigger_Drawn.java index 4c06c8910b3..aafeaa0d46b 100644 --- a/src/main/java/forge/card/trigger/Trigger_Drawn.java +++ b/src/main/java/forge/card/trigger/Trigger_Drawn.java @@ -29,7 +29,7 @@ public class Trigger_Drawn extends Trigger { Card draw = ((Card) runParams2.get("Card")); if (mapParams.containsKey("ValidCard")) { - if (!draw.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!draw.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_Sacrificed.java b/src/main/java/forge/card/trigger/Trigger_Sacrificed.java index c715ae89adb..dee31dde1de 100644 --- a/src/main/java/forge/card/trigger/Trigger_Sacrificed.java +++ b/src/main/java/forge/card/trigger/Trigger_Sacrificed.java @@ -33,7 +33,7 @@ public class Trigger_Sacrificed extends Trigger { } } if (mapParams.containsKey("ValidCard")) { - if (!sac.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!sac.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_SpellAbilityCast.java b/src/main/java/forge/card/trigger/Trigger_SpellAbilityCast.java index bc1b6efcef0..ba1918cc965 100644 --- a/src/main/java/forge/card/trigger/Trigger_SpellAbilityCast.java +++ b/src/main/java/forge/card/trigger/Trigger_SpellAbilityCast.java @@ -103,7 +103,7 @@ public class Trigger_SpellAbilityCast extends Trigger { if (sa.getTarget().doesTarget()) { boolean validTgtFound = false; for (Card tgt : sa.getTarget().getTargetCards()) { - if (tgt.isValidCard(mapParams.get("TargetsValid").split(","), hostCard.getController(), hostCard)) { + if (tgt.isValid(mapParams.get("TargetsValid").split(","), hostCard.getController(), hostCard)) { validTgtFound = true; break; } diff --git a/src/main/java/forge/card/trigger/Trigger_Taps.java b/src/main/java/forge/card/trigger/Trigger_Taps.java index 493158c2709..880b2ecb767 100644 --- a/src/main/java/forge/card/trigger/Trigger_Taps.java +++ b/src/main/java/forge/card/trigger/Trigger_Taps.java @@ -30,7 +30,7 @@ public class Trigger_Taps extends Trigger { Card tapper = (Card) runParams2.get("Card"); if (mapParams.containsKey("ValidCard")) { - if (!tapper.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!tapper.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_TapsForMana.java b/src/main/java/forge/card/trigger/Trigger_TapsForMana.java index b50abd62fbf..8c832546948 100644 --- a/src/main/java/forge/card/trigger/Trigger_TapsForMana.java +++ b/src/main/java/forge/card/trigger/Trigger_TapsForMana.java @@ -29,7 +29,7 @@ public class Trigger_TapsForMana extends Trigger { Card tapper = (Card) runParams2.get("Card"); if (mapParams.containsKey("ValidCard")) { - if (!tapper.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!tapper.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_Unequip.java b/src/main/java/forge/card/trigger/Trigger_Unequip.java index 5f246336bde..6072aefe6fe 100644 --- a/src/main/java/forge/card/trigger/Trigger_Unequip.java +++ b/src/main/java/forge/card/trigger/Trigger_Unequip.java @@ -30,13 +30,13 @@ public class Trigger_Unequip extends Trigger { Card equipment = (Card) runParams2.get("Equipment"); if (mapParams.containsKey("ValidCard")) { - if (!equipped.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!equipped.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } } if (mapParams.containsKey("ValidEquipment")) { - if (!equipment.isValidCard(mapParams.get("ValidEquipment").split(","), hostCard.getController(), hostCard)) { + if (!equipment.isValid(mapParams.get("ValidEquipment").split(","), hostCard.getController(), hostCard)) { return false; } } diff --git a/src/main/java/forge/card/trigger/Trigger_Untaps.java b/src/main/java/forge/card/trigger/Trigger_Untaps.java index 975b8e86e38..313ad0adcf6 100644 --- a/src/main/java/forge/card/trigger/Trigger_Untaps.java +++ b/src/main/java/forge/card/trigger/Trigger_Untaps.java @@ -29,7 +29,7 @@ public class Trigger_Untaps extends Trigger { Card untapper = (Card) runParams2.get("Card"); if (mapParams.containsKey("ValidCard")) { - if (!untapper.isValidCard(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { + if (!untapper.isValid(mapParams.get("ValidCard").split(","), hostCard.getController(), hostCard)) { return false; } }