From 3bebdd65bc2ae3e6a64afe8ca53223d198fb1eb2 Mon Sep 17 00:00:00 2001 From: Nathaniel Brandes Date: Wed, 8 Mar 2017 01:01:10 -0800 Subject: [PATCH] Fixed crewing staying if the object changed zones. --- .../continuous/AddCardTypeSourceEffect.java | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeSourceEffect.java b/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeSourceEffect.java index 35ee7f03fde..83ac0c1e94c 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeSourceEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeSourceEffect.java @@ -31,12 +31,7 @@ import mage.MageObjectReference; import mage.abilities.Ability; import mage.abilities.Mode; import mage.abilities.effects.ContinuousEffectImpl; -import mage.constants.CardType; -import mage.constants.DependencyType; -import mage.constants.Duration; -import mage.constants.Layer; -import mage.constants.Outcome; -import mage.constants.SubLayer; +import mage.constants.*; import mage.game.Game; import mage.game.permanent.Permanent; @@ -63,19 +58,13 @@ public class AddCardTypeSourceEffect extends ContinuousEffectImpl { @Override public void init(Ability source, Game game) { super.init(source, game); - if (this.duration == Duration.Custom || this.duration.toString().startsWith("End")) { - affectedObjectList.add(new MageObjectReference(source.getSourceId(), game.getState().getZoneChangeCounter(source.getSourceId()), game)); - if (affectedObjectList.isEmpty()) { - this.discard(); - } - } + affectedObjectList.add(new MageObjectReference(source.getSourceId(), game.getState().getZoneChangeCounter(source.getSourceId()), game)); } @Override public boolean apply(Game game, Ability source) { Permanent permanent = game.getPermanent(source.getSourceId()); - if (permanent != null - && (affectedObjectList.isEmpty() || affectedObjectList.contains(new MageObjectReference(permanent, game)))) { + if (permanent != null && affectedObjectList.contains(new MageObjectReference(permanent, game))) { if (!permanent.getCardType().contains(addedCardType)) { permanent.getCardType().add(addedCardType); } @@ -97,7 +86,7 @@ public class AddCardTypeSourceEffect extends ContinuousEffectImpl { return staticText; } StringBuilder sb = new StringBuilder(); - sb.append("{this} becomes ").append(addedCardType.toString()).append(" in addition to its other types until end of turn"); + sb.append("{this} becomes ").append(addedCardType.toString()).append(" in addition to its other types " + this.getDuration().toString()); return sb.toString(); } }