From 91a3d9a6a2ce01d5fc76c19b352d067c218ea799 Mon Sep 17 00:00:00 2001 From: Justin Herlehy Date: Wed, 21 Dec 2016 02:33:58 -0500 Subject: [PATCH 1/2] [AER] Heart of Kiran As the alternate crew cost payment ability is just a simple activated ability with a different wording, it is implemented here as an anonymous inner class to override the getRule() method rather than generate large amounts of boilerplate code to get the same effect with a full class. --- Mage.Sets/src/mage/cards/h/HeartOfKiran.java | 93 ++++++++++++++++++++ Mage.Sets/src/mage/sets/AetherRevolt.java | 1 + 2 files changed, 94 insertions(+) create mode 100644 Mage.Sets/src/mage/cards/h/HeartOfKiran.java diff --git a/Mage.Sets/src/mage/cards/h/HeartOfKiran.java b/Mage.Sets/src/mage/cards/h/HeartOfKiran.java new file mode 100644 index 0000000000..b652e73559 --- /dev/null +++ b/Mage.Sets/src/mage/cards/h/HeartOfKiran.java @@ -0,0 +1,93 @@ +/* + * Copyright 2010 BetaSteward_at_googlemail.com. All rights reserved. + * + * Redistribution and use in source and binary forms, with or without modification, are + * permitted provided that the following conditions are met: + * + * 1. Redistributions of source code must retain the above copyright notice, this list of + * conditions and the following disclaimer. + * + * 2. Redistributions in binary form must reproduce the above copyright notice, this list + * of conditions and the following disclaimer in the documentation and/or other materials + * provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY BetaSteward_at_googlemail.com ``AS IS'' AND ANY EXPRESS OR IMPLIED + * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND + * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL BetaSteward_at_googlemail.com OR + * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR + * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON + * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF + * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * The views and conclusions contained in the software and documentation are those of the + * authors and should not be interpreted as representing official policies, either expressed + * or implied, of BetaSteward_at_googlemail.com. + */ +package mage.cards.h; + +import mage.MageInt; +import mage.abilities.Ability; +import mage.abilities.common.SimpleActivatedAbility; +import mage.abilities.costs.Cost; +import mage.abilities.costs.common.RemoveCounterCost; +import mage.abilities.effects.common.continuous.AddCardTypeSourceEffect; +import mage.abilities.keyword.CrewAbility; +import mage.abilities.keyword.FlyingAbility; +import mage.abilities.keyword.VigilanceAbility; +import mage.cards.CardImpl; +import mage.cards.CardSetInfo; +import mage.constants.CardType; +import mage.constants.Duration; +import mage.constants.Zone; +import mage.counters.CounterType; +import mage.filter.common.FilterControlledPlaneswalkerPermanent; +import mage.target.common.TargetControlledPermanent; + +import java.util.UUID; + +/** + * @author JRHerlehy + */ +public class HeartOfKiran extends CardImpl { + + private static final FilterControlledPlaneswalkerPermanent filter = new FilterControlledPlaneswalkerPermanent("planeswalker you control"); + + public HeartOfKiran(UUID ownerId, CardSetInfo setInfo) { + super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{2}"); + + this.supertype.add("Legendary"); + this.subtype.add("Vehicle"); + this.power = new MageInt(4); + this.toughness = new MageInt(4); + + // Flying + this.addAbility(FlyingAbility.getInstance()); + + // Vigilance + this.addAbility(VigilanceAbility.getInstance()); + + // Crew 3 + this.addAbility(new CrewAbility(3)); + + // You may remove a loyalty counter from a planeswalker you control rather than pay Heart of Kiran's crew cost. + Cost cost = new RemoveCounterCost(new TargetControlledPermanent(filter), CounterType.LOYALTY, 1); + Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new AddCardTypeSourceEffect(CardType.CREATURE, Duration.EndOfTurn), cost) { + @Override + public String getRule() { + return "You may remove a loyalty counter from a planeswalker you control rather than pay {this}'s crew cost."; + } + }; + this.addAbility(ability); + } + + public HeartOfKiran(final HeartOfKiran card) { + super(card); + } + + @Override + public HeartOfKiran copy() { + return new HeartOfKiran(this); + } +} diff --git a/Mage.Sets/src/mage/sets/AetherRevolt.java b/Mage.Sets/src/mage/sets/AetherRevolt.java index 1a4864be2d..ce57ddc72b 100644 --- a/Mage.Sets/src/mage/sets/AetherRevolt.java +++ b/Mage.Sets/src/mage/sets/AetherRevolt.java @@ -69,6 +69,7 @@ public class AetherRevolt extends ExpansionSet { cards.add(new SetCardInfo("Consulate Crackdown", 11, Rarity.RARE, mage.cards.c.ConsulateCrackdown.class)); cards.add(new SetCardInfo("Dark Intimations", 128, Rarity.RARE, mage.cards.d.DarkIntimations.class)); cards.add(new SetCardInfo("Disallow", 31, Rarity.RARE, mage.cards.d.Disallow.class)); + cards.add(new SetCardInfo("Heart of Kiran", 153, Rarity.MYTHIC, mage.cards.h.HeartOfKiran.class)); cards.add(new SetCardInfo("Oath of Ajani", 131, Rarity.RARE, mage.cards.o.OathOfAjani.class)); cards.add(new SetCardInfo("Pia's Revolution", 91, Rarity.RARE, mage.cards.p.PiasRevolution.class)); cards.add(new SetCardInfo("Quicksmith Rebel", 93, Rarity.RARE, mage.cards.q.QuicksmithRebel.class)); From 4ef339d521e009d4960759c5ab6494963bfafb01 Mon Sep 17 00:00:00 2001 From: Justin Herlehy Date: Wed, 21 Dec 2016 23:07:55 -0500 Subject: [PATCH 2/2] [AER] Heart of Kiran Fixed the Alternate Crew Cost to push the correct game events for crewing a vehicle and the vehicle being crewed. --- Mage.Sets/src/mage/cards/h/HeartOfKiran.java | 65 ++++++++++++++++++-- 1 file changed, 61 insertions(+), 4 deletions(-) diff --git a/Mage.Sets/src/mage/cards/h/HeartOfKiran.java b/Mage.Sets/src/mage/cards/h/HeartOfKiran.java index b652e73559..dca01ade0a 100644 --- a/Mage.Sets/src/mage/cards/h/HeartOfKiran.java +++ b/Mage.Sets/src/mage/cards/h/HeartOfKiran.java @@ -31,7 +31,7 @@ import mage.MageInt; import mage.abilities.Ability; import mage.abilities.common.SimpleActivatedAbility; import mage.abilities.costs.Cost; -import mage.abilities.costs.common.RemoveCounterCost; +import mage.abilities.costs.CostImpl; import mage.abilities.effects.common.continuous.AddCardTypeSourceEffect; import mage.abilities.keyword.CrewAbility; import mage.abilities.keyword.FlyingAbility; @@ -40,9 +40,14 @@ import mage.cards.CardImpl; import mage.cards.CardSetInfo; import mage.constants.CardType; import mage.constants.Duration; +import mage.constants.Outcome; import mage.constants.Zone; import mage.counters.CounterType; import mage.filter.common.FilterControlledPlaneswalkerPermanent; +import mage.game.Game; +import mage.game.events.GameEvent; +import mage.game.permanent.Permanent; +import mage.target.Target; import mage.target.common.TargetControlledPermanent; import java.util.UUID; @@ -52,8 +57,6 @@ import java.util.UUID; */ public class HeartOfKiran extends CardImpl { - private static final FilterControlledPlaneswalkerPermanent filter = new FilterControlledPlaneswalkerPermanent("planeswalker you control"); - public HeartOfKiran(UUID ownerId, CardSetInfo setInfo) { super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{2}"); @@ -72,7 +75,7 @@ public class HeartOfKiran extends CardImpl { this.addAbility(new CrewAbility(3)); // You may remove a loyalty counter from a planeswalker you control rather than pay Heart of Kiran's crew cost. - Cost cost = new RemoveCounterCost(new TargetControlledPermanent(filter), CounterType.LOYALTY, 1); + Cost cost = new HeartOfKiranAlternateCrewCost(CounterType.LOYALTY, 1); Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new AddCardTypeSourceEffect(CardType.CREATURE, Duration.EndOfTurn), cost) { @Override public String getRule() { @@ -91,3 +94,57 @@ public class HeartOfKiran extends CardImpl { return new HeartOfKiran(this); } } + +class HeartOfKiranAlternateCrewCost extends CostImpl { + + private CounterType counterTypeToRemove; + private int countersToRemove; + + private static final FilterControlledPlaneswalkerPermanent filter = new FilterControlledPlaneswalkerPermanent("planeswalker you control"); + + public HeartOfKiranAlternateCrewCost(CounterType counterTypeToRemove, int countersToRemove) { + this.counterTypeToRemove = counterTypeToRemove; + this.countersToRemove = countersToRemove; + } + + public HeartOfKiranAlternateCrewCost(final HeartOfKiranAlternateCrewCost cost) { + super(cost); + this.counterTypeToRemove = cost.counterTypeToRemove; + this.countersToRemove = cost.countersToRemove; + } + + @Override + public boolean pay(Ability ability, Game game, UUID sourceId, UUID controllerId, boolean noMana, Cost costToPay) { + paid = false; + + Target target = new TargetControlledPermanent(1, 1, filter, true); + + if (target.choose(Outcome.Benefit, controllerId, sourceId, game)) { + Permanent permanent = game.getPermanent(target.getFirstTarget()); + int originalLoyalty = permanent.getCounters(game).getCount(counterTypeToRemove); + + GameEvent event = new GameEvent(GameEvent.EventType.CREW_VEHICLE, target.getFirstTarget(), sourceId, controllerId); + if (!game.replaceEvent(event)) { + permanent.removeCounters(counterTypeToRemove.createInstance(), game); + } + + paid = permanent.getCounters(game).getCount(counterTypeToRemove) < originalLoyalty; + + if (paid) { + game.fireEvent(GameEvent.getEvent(GameEvent.EventType.CREWED_VEHICLE, target.getFirstTarget(), sourceId, controllerId)); + } + } + + return paid; + } + + @Override + public boolean canPay(Ability ability, UUID sourceId, UUID controllerId, Game game) { + return game.getBattlefield().getAllActivePermanents(filter, game).size() > 0; + } + + @Override + public HeartOfKiranAlternateCrewCost copy() { + return new HeartOfKiranAlternateCrewCost(this); + } +}