diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/CertainDeath.java b/Mage.Sets/src/mage/sets/eldritchmoon/CertainDeath.java index 0cc829f4362..639640636f0 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/CertainDeath.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/CertainDeath.java @@ -68,7 +68,7 @@ class CertainDeathEffect extends OneShotEffect { public CertainDeathEffect() { super(Outcome.DestroyPermanent); - this.staticText = "Destroy target creature. Its controler loses 2 life and you gain 2 life"; + this.staticText = "Destroy target creature. Its controller loses 2 life and you gain 2 life"; } public CertainDeathEffect(final CertainDeathEffect effect) { diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/Cryptbreaker.java b/Mage.Sets/src/mage/sets/eldritchmoon/Cryptbreaker.java index cfb2586c7b2..df5de13794d 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/Cryptbreaker.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/Cryptbreaker.java @@ -77,8 +77,10 @@ public class Cryptbreaker extends CardImpl { this.addAbility(ability); // Tap three untapped Zombies you control: You draw a card and you lose 1 life. - ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new DrawCardSourceControllerEffect(1), new TapTargetCost(new TargetControlledCreaturePermanent(3, 3, filter, true))); - Effect effect = new LoseLifeSourceControllerEffect(1); + Effect effect = new DrawCardSourceControllerEffect(1); + effect.setText("You draw a card"); + ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, effect, new TapTargetCost(new TargetControlledCreaturePermanent(3, 3, filter, true))); + effect = new LoseLifeSourceControllerEffect(1); effect.setText("and you lose 1 life"); ability.addEffect(effect); this.addAbility(ability); diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/DarkSalvation.java b/Mage.Sets/src/mage/sets/eldritchmoon/DarkSalvation.java index 0889c217b8b..26bed3fcc75 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/DarkSalvation.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/DarkSalvation.java @@ -41,8 +41,8 @@ import mage.constants.Rarity; import mage.filter.common.FilterCreaturePermanent; import mage.filter.predicate.mageobject.SubtypePredicate; import mage.game.Game; -import mage.game.permanent.Permanent; import mage.game.permanent.token.ZombieToken; +import mage.players.Player; import mage.target.TargetPlayer; import mage.target.common.TargetCreaturePermanent; import mage.target.targetpointer.SecondTargetPointer; @@ -62,9 +62,9 @@ public class DarkSalvation extends CardImpl { Effect effect = new CreateTokenTargetEffect(new ZombieToken(), new ManacostVariableValue()); effect.setText("Target player puts X 2/2 black Zombie creature tokens onto the battlefield"); this.getSpellAbility().addEffect(effect); - this.getSpellAbility().addTarget(new TargetCreaturePermanent(0, 1, new FilterCreaturePermanent(), false)); + DynamicValue value = new ZombiesControlledByTargetPlayerCount(); - DynamicValue value = new ZombiesControlledByTargetCreaturesControllerCount(); + this.getSpellAbility().addTarget(new TargetCreaturePermanent(0, 1, new FilterCreaturePermanent(), false)); effect = new BoostTargetEffect(value, value, Duration.EndOfTurn, true); effect.setTargetPointer(new SecondTargetPointer()); effect.setText(", then up to one target creature gets -1/-1 until end of turn for each Zombie that player controls"); @@ -81,7 +81,7 @@ public class DarkSalvation extends CardImpl { } } -class ZombiesControlledByTargetCreaturesControllerCount implements DynamicValue { +class ZombiesControlledByTargetPlayerCount implements DynamicValue { private final static FilterCreaturePermanent filter = new FilterCreaturePermanent("Zombies"); @@ -90,15 +90,15 @@ class ZombiesControlledByTargetCreaturesControllerCount implements DynamicValue } @Override - public ZombiesControlledByTargetCreaturesControllerCount copy() { - return new ZombiesControlledByTargetCreaturesControllerCount(); + public ZombiesControlledByTargetPlayerCount copy() { + return new ZombiesControlledByTargetPlayerCount(); } @Override public int calculate(Game game, Ability sourceAbility, Effect effect) { - Permanent targetCreature = game.getPermanent(effect.getTargetPointer().getFirst(game, sourceAbility)); - if (targetCreature != null) { - int value = game.getBattlefield().countAll(filter, targetCreature.getControllerId(), game); + Player player = game.getPlayer(sourceAbility.getTargets().get(0).getFirstTarget()); + if (player != null) { + int value = game.getBattlefield().countAll(filter, player.getId(), game); return -1 * value; } else { return 0; diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/DuskFeaster.java b/Mage.Sets/src/mage/sets/eldritchmoon/DuskFeaster.java index c8fc97b681d..6e4109aebe4 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/DuskFeaster.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/DuskFeaster.java @@ -62,6 +62,7 @@ public class DuskFeaster extends CardImpl { // Delirium — Dusk Feaster costs {2} less to cast if there are four or more card types among cards in your graveyard. this.addAbility(new SimpleStaticAbility(Zone.STACK, new DuskFeasterCostReductionEffect())); + // Flying this.addAbility(FlyingAbility.getInstance()); } @@ -80,7 +81,7 @@ class DuskFeasterCostReductionEffect extends CostModificationEffectImpl { DuskFeasterCostReductionEffect() { super(Duration.Custom, Outcome.Benefit, CostModificationType.REDUCE_COST); - staticText = "{this} costs {2} less to cast if there are four or more card types among cards in your graveyard"; + staticText = "Delirium — {this} costs {2} less to cast if there are four or more card types among cards in your graveyard"; } DuskFeasterCostReductionEffect(final DuskFeasterCostReductionEffect effect) { @@ -105,7 +106,7 @@ class DuskFeasterCostReductionEffect extends CostModificationEffectImpl { @Override public boolean applies(Ability abilityToModify, Ability source, Game game) { - + boolean hasDelirium = false; Player controller = game.getPlayer(source.getControllerId()); if (controller != null) { @@ -116,9 +117,9 @@ class DuskFeasterCostReductionEffect extends CostModificationEffectImpl { int number = foundCardTypes.size(); hasDelirium = number > 3; } - - return abilityToModify.getSourceId().equals(source.getSourceId()) - && (abilityToModify instanceof SpellAbility) + + return abilityToModify.getSourceId().equals(source.getSourceId()) + && (abilityToModify instanceof SpellAbility) && hasDelirium; } diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/GrafHarvest.java b/Mage.Sets/src/mage/sets/eldritchmoon/GrafHarvest.java index 7848dfeac15..ed5db502a70 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/GrafHarvest.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/GrafHarvest.java @@ -33,6 +33,7 @@ import mage.abilities.common.SimpleActivatedAbility; import mage.abilities.common.SimpleStaticAbility; import mage.abilities.costs.common.ExileFromGraveCost; import mage.abilities.costs.mana.ManaCostsImpl; +import mage.abilities.effects.Effect; import mage.abilities.effects.common.CreateTokenEffect; import mage.abilities.effects.common.continuous.GainAbilityAllEffect; import mage.abilities.keyword.MenaceAbility; @@ -52,7 +53,7 @@ import mage.target.common.TargetCardInYourGraveyard; * @author escplan9 (Derek Monturo - dmontur1 at gmail dot com) */ public class GrafHarvest extends CardImpl { - + private static final FilterControlledCreaturePermanent filter = new FilterControlledCreaturePermanent("Zombies you control"); static { @@ -64,8 +65,10 @@ public class GrafHarvest extends CardImpl { this.expansionSetCode = "EMN"; // Zombies you control have menace. - this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new GainAbilityAllEffect(new MenaceAbility(), Duration.WhileOnBattlefield, filter))); - + Effect effect = new GainAbilityAllEffect(new MenaceAbility(), Duration.WhileOnBattlefield, filter); + effect.setText("Zombies you control have menace. (They can't be blocked except by two or more creatures.)"); + this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, effect)); + // {3}{B}, Exile a creature card from your graveyard: Put a 2/2 black Zombie creature token onto the battlefield. Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new CreateTokenEffect(new ZombieToken()), new ManaCostsImpl("{3}{B}")); ability.addCost(new ExileFromGraveCost(new TargetCardInYourGraveyard(new FilterCreatureCard("a creature card from your graveyard")))); diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/NoosegrafMob.java b/Mage.Sets/src/mage/sets/eldritchmoon/NoosegrafMob.java index d6ef211a933..6d1c24d9066 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/NoosegrafMob.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/NoosegrafMob.java @@ -60,8 +60,7 @@ public class NoosegrafMob extends CardImpl { this.toughness = new MageInt(0); // Noosegraf Mob enters the battlefield with five +1/+1 counters on it. - this.addAbility(new EntersBattlefieldAbility(new AddCountersSourceEffect(CounterType.P1P1.createInstance(5)), - "{this} enters the battlefield with five +1/+1 counters on it")); + this.addAbility(new EntersBattlefieldAbility(new AddCountersSourceEffect(CounterType.P1P1.createInstance(5)), "with five +1/+1 counters on it")); // Whenever a player casts a spell, remove a +1/+1 counter from Noosegraf Mob. If you do, put a 2/2 black Zombie creature token onto the battlefield. this.addAbility(new SpellCastAllTriggeredAbility(new NoosegrafMobEffect(), false)); diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/RuthlessDisposal.java b/Mage.Sets/src/mage/sets/eldritchmoon/RuthlessDisposal.java index ecfa343cfc9..7f5e923f04f 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/RuthlessDisposal.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/RuthlessDisposal.java @@ -28,8 +28,10 @@ package mage.sets.eldritchmoon; import java.util.UUID; +import mage.abilities.costs.Cost; import mage.abilities.costs.common.DiscardTargetCost; import mage.abilities.costs.common.SacrificeTargetCost; +import mage.abilities.effects.Effect; import mage.abilities.effects.common.continuous.BoostTargetEffect; import mage.cards.CardImpl; import mage.constants.CardType; @@ -51,11 +53,16 @@ public class RuthlessDisposal extends CardImpl { this.expansionSetCode = "EMN"; // As an additional cost to cast Ruthless Disposal, discard a card and sacrifice a creature. - this.getSpellAbility().addCost(new DiscardTargetCost(new TargetCardInHand(new FilterCard("card to discard")))); - this.getSpellAbility().addCost(new SacrificeTargetCost(new TargetControlledCreaturePermanent())); + this.getSpellAbility().addCost(new DiscardTargetCost(new TargetCardInHand(new FilterCard("a card")))); + Cost cost = new SacrificeTargetCost(new TargetControlledCreaturePermanent()); + cost.setText("As an additional cost to cast {this}, sacrifice a creature"); + this.getSpellAbility().addCost(cost); + // Two target creatures each get -13/-13 until end of turn. + Effect effect = new BoostTargetEffect(-13, -13, Duration.EndOfTurn); + effect.setText("Two target creatures each get -13/-13 until end of turn"); this.getSpellAbility().addTarget(new TargetCreaturePermanent(2)); - this.getSpellAbility().addEffect(new BoostTargetEffect(-13, -13, Duration.EndOfTurn)); + this.getSpellAbility().addEffect(effect); } diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/StrangeAugmentation.java b/Mage.Sets/src/mage/sets/eldritchmoon/StrangeAugmentation.java index 910c0c60163..72e4ae52b0b 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/StrangeAugmentation.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/StrangeAugmentation.java @@ -66,7 +66,7 @@ public class StrangeAugmentation extends CardImpl { // Delirium &mdash Enchanted creature gets an additional +2/+2 as long as there are four or more card types among cards in your graveyard. this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new ConditionalContinuousEffect(new BoostEnchantedEffect(2, 2), DeliriumCondition.getInstance(), - "Delirium — Enchanted creature gets an additional +2/+2 as long as you have no cards in hand"))); + "Delirium — Enchanted creature gets an additional +2/+2 as long as there are four or more card types among cards in your graveyard."))); } public StrangeAugmentation(final StrangeAugmentation card) { diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/StromkirkCondemned.java b/Mage.Sets/src/mage/sets/eldritchmoon/StromkirkCondemned.java index 3faaa6213b3..93488a5eb24 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/StromkirkCondemned.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/StromkirkCondemned.java @@ -53,7 +53,7 @@ public class StromkirkCondemned extends CardImpl { } public StromkirkCondemned(UUID ownerId) { - super(ownerId, 105, "Stromkirk Condemned", Rarity.RARE, new CardType[]{CardType.CREATURE}, "{B}{B}"); + super(ownerId, 106, "Stromkirk Condemned", Rarity.RARE, new CardType[]{CardType.CREATURE}, "{B}{B}"); this.expansionSetCode = "EMN"; this.subtype.add("Vampire"); this.subtype.add("Horror"); diff --git a/Mage.Sets/src/mage/sets/eldritchmoon/WhispersOfEmrakul.java b/Mage.Sets/src/mage/sets/eldritchmoon/WhispersOfEmrakul.java index 0b6900ab251..bce763e8014 100644 --- a/Mage.Sets/src/mage/sets/eldritchmoon/WhispersOfEmrakul.java +++ b/Mage.Sets/src/mage/sets/eldritchmoon/WhispersOfEmrakul.java @@ -51,7 +51,7 @@ public class WhispersOfEmrakul extends CardImpl { this.getSpellAbility().addEffect(new ConditionalOneShotEffect( new DiscardTargetEffect(1, true), new InvertCondition(DeliriumCondition.getInstance()), - "Target player sacrifices a creature or planeswalker.")); + "Target opponent discards a card at random.")); // Delirium — If there are four or more card types among cards in your graveyard, that player discards two cards at random instead. this.getSpellAbility().addEffect(new ConditionalOneShotEffect(