diff --git a/Mage.Sets/src/mage/cards/a/AwakenedSkyclave.java b/Mage.Sets/src/mage/cards/a/AwakenedSkyclave.java index b104abbe8bda..48e9a24dee88 100644 --- a/Mage.Sets/src/mage/cards/a/AwakenedSkyclave.java +++ b/Mage.Sets/src/mage/cards/a/AwakenedSkyclave.java @@ -35,7 +35,7 @@ public AwakenedSkyclave(UUID ownerId, CardSetInfo setInfo) { this.addAbility(HasteAbility.getInstance()); // As long as Awakened Skyclave is on the battlefield, it's a land in addition to its other types. - this.addAbility(new SimpleStaticAbility(new AddCardTypeSourceEffect(Duration.WhileOnBattlefield) + this.addAbility(new SimpleStaticAbility(new AddCardTypeSourceEffect(Duration.WhileOnBattlefield, CardType.LAND) .setText("as long as {this} is on the battlefield, it's a land in addition to its other types"))); // {T}: Add one mana of any color. 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 cde16f3fa3c0..2bdd126d010f 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 @@ -21,12 +21,17 @@ public class AddCardTypeSourceEffect extends ContinuousEffectImpl { public AddCardTypeSourceEffect(Duration duration, CardType... addedCardType) { super(duration, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit); + if (addedCardType.length == 0) { + throw new IllegalArgumentException("AddCardTypeSourceEffect should be called with at least one card type."); + } for (CardType cardType : addedCardType) { this.addedCardTypes.add(cardType); if (cardType == CardType.ENCHANTMENT) { dependencyTypes.add(DependencyType.EnchantmentAddingRemoving); } else if (cardType == CardType.ARTIFACT) { dependencyTypes.add(DependencyType.ArtifactAddingRemoving); + } else if (cardType == CardType.LAND) { + dependencyTypes.add(DependencyType.BecomeNonbasicLand); } } } @@ -45,7 +50,8 @@ public void init(Ability source, Game game) { @Override public boolean apply(Game game, Ability source) { Permanent permanent = game.getPermanent(source.getSourceId()); - if (permanent != null && affectedObjectList.contains(new MageObjectReference(permanent, game))) { + if (permanent != null + && (affectedObjectList.contains(new MageObjectReference(permanent, game)) || !duration.isOnlyValidIfNoZoneChange())) { for (CardType cardType : addedCardTypes) { permanent.addCardType(game, cardType); } diff --git a/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeTargetEffect.java b/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeTargetEffect.java index 7cda13f3c0eb..188465afe27f 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeTargetEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/continuous/AddCardTypeTargetEffect.java @@ -21,12 +21,17 @@ public class AddCardTypeTargetEffect extends ContinuousEffectImpl { public AddCardTypeTargetEffect(Duration duration, CardType... addedCardType) { super(duration, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit); + if (addedCardType.length == 0) { + throw new IllegalArgumentException("AddCardTypeTargetEffect should be called with at least one card type."); + } for (CardType cardType : addedCardType) { this.addedCardTypes.add(cardType); if (cardType == CardType.ENCHANTMENT) { dependencyTypes.add(DependencyType.EnchantmentAddingRemoving); } else if (cardType == CardType.ARTIFACT) { dependencyTypes.add(DependencyType.ArtifactAddingRemoving); + } else if (cardType == CardType.LAND) { + dependencyTypes.add(DependencyType.BecomeNonbasicLand); } }