From 09012aee8bb0616e590ee7d2d9c9a7348bd2bfba Mon Sep 17 00:00:00 2001 From: Sloth Date: Thu, 23 May 2013 16:34:11 +0000 Subject: [PATCH] - Fixed mandatory targets not being enforced for suspended spells. --- src/main/java/forge/card/ability/effects/PlayEffect.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/forge/card/ability/effects/PlayEffect.java b/src/main/java/forge/card/ability/effects/PlayEffect.java index 3209779b62c..7add7c04c86 100644 --- a/src/main/java/forge/card/ability/effects/PlayEffect.java +++ b/src/main/java/forge/card/ability/effects/PlayEffect.java @@ -228,15 +228,15 @@ public class PlayEffect extends SpellAbilityEffect { SpellAbility tgtSA = null; // only one mode can be used tgtSA = sa.getActivatingPlayer().getController().getAbilityToPlay(sas); + boolean noManaCost = sa.hasParam("WithoutManaCost"); + tgtSA = noManaCost ? tgtSA.copyWithNoManaCost() : tgtSA; if (tgtSA.getTarget() != null && !optional) { tgtSA.getTarget().setMandatory(true); } - boolean noManaCost = sa.hasParam("WithoutManaCost"); if (controller.isHuman()) { - SpellAbility newSA = noManaCost ? tgtSA.copyWithNoManaCost() : tgtSA; - HumanPlay.playSpellAbility(activator, newSA); + HumanPlay.playSpellAbility(activator, tgtSA); } else { if (tgtSA instanceof Spell) { // Isn't it ALWAYS a spell? Spell spell = (Spell) tgtSA;