diff --git a/Mage.Sets/src/mage/sets/betrayersofkamigawa/WalkerOfSecretWays.java b/Mage.Sets/src/mage/sets/betrayersofkamigawa/WalkerOfSecretWays.java index c79a12b621d..1c236beb7dd 100644 --- a/Mage.Sets/src/mage/sets/betrayersofkamigawa/WalkerOfSecretWays.java +++ b/Mage.Sets/src/mage/sets/betrayersofkamigawa/WalkerOfSecretWays.java @@ -33,9 +33,9 @@ import mage.constants.CardType; import mage.constants.Rarity; import mage.MageInt; import mage.abilities.Ability; +import mage.abilities.common.ActivateIfConditionActivatedAbility; import mage.abilities.common.DealsCombatDamageToAPlayerTriggeredAbility; -import mage.abilities.common.SimpleActivatedAbility; -import mage.abilities.costs.common.OnlyDuringYourTurnCost; +import mage.abilities.condition.common.MyTurnCondition; import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.effects.OneShotEffect; import mage.abilities.effects.common.ReturnToHandTargetEffect; @@ -76,9 +76,8 @@ public class WalkerOfSecretWays extends CardImpl { this.addAbility(new DealsCombatDamageToAPlayerTriggeredAbility(new WalkerOfSecretWaysEffect(), true, true)); // {1}{U}: Return target Ninja you control to its owner's hand. Activate this ability only during your turn. - Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new ReturnToHandTargetEffect(), new ManaCostsImpl("{1}{U}")); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new ReturnToHandTargetEffect(), new ManaCostsImpl("{1}{U}"), MyTurnCondition.getInstance()); ability.addTarget(new TargetControlledCreaturePermanent(1,1, filterCreature, false)); - ability.addCost(new OnlyDuringYourTurnCost()); this.addAbility(ability); diff --git a/Mage.Sets/src/mage/sets/conflux/Fleshformer.java b/Mage.Sets/src/mage/sets/conflux/Fleshformer.java index 6f19f0f3771..b4f7bcfa0b0 100644 --- a/Mage.Sets/src/mage/sets/conflux/Fleshformer.java +++ b/Mage.Sets/src/mage/sets/conflux/Fleshformer.java @@ -30,8 +30,8 @@ package mage.sets.conflux; import java.util.UUID; import mage.MageInt; import mage.abilities.Ability; -import mage.abilities.common.SimpleActivatedAbility; -import mage.abilities.costs.common.OnlyDuringYourTurnCost; +import mage.abilities.common.ActivateIfConditionActivatedAbility; +import mage.abilities.condition.common.MyTurnCondition; import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.effects.common.continious.BoostSourceEffect; import mage.abilities.effects.common.continious.BoostTargetEffect; @@ -59,8 +59,7 @@ public class Fleshformer extends CardImpl { this.toughness = new MageInt(2); // {W}{U}{B}{R}{G}: Fleshformer gets +2/+2 and gains fear until end of turn. Target creature gets -2/-2 until end of turn. Activate this ability only during your turn. - Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new BoostSourceEffect(2, 2, Duration.EndOfTurn), new ManaCostsImpl("{W}{U}{B}{R}{G}")); - ability.addCost(new OnlyDuringYourTurnCost()); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new BoostSourceEffect(2, 2, Duration.EndOfTurn), new ManaCostsImpl("{W}{U}{B}{R}{G}"), MyTurnCondition.getInstance()); ability.addEffect(new BoostTargetEffect(-2, -2, Duration.EndOfTurn)); ability.addTarget(new TargetCreaturePermanent(true)); this.addAbility(ability); diff --git a/Mage.Sets/src/mage/sets/conflux/ScepterOfFugue.java b/Mage.Sets/src/mage/sets/conflux/ScepterOfFugue.java index 72998bb7ab6..4287871f1a8 100644 --- a/Mage.Sets/src/mage/sets/conflux/ScepterOfFugue.java +++ b/Mage.Sets/src/mage/sets/conflux/ScepterOfFugue.java @@ -32,8 +32,8 @@ import java.util.UUID; import mage.constants.CardType; import mage.constants.Rarity; import mage.abilities.Ability; -import mage.abilities.common.SimpleActivatedAbility; -import mage.abilities.costs.common.OnlyDuringYourTurnCost; +import mage.abilities.common.ActivateIfConditionActivatedAbility; +import mage.abilities.condition.common.MyTurnCondition; import mage.abilities.costs.common.TapSourceCost; import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.effects.common.DiscardTargetEffect; @@ -54,10 +54,9 @@ public class ScepterOfFugue extends CardImpl { this.color.setBlack(true); // {1}{B}, {tap}: Target player discards a card. Activate this ability only during your turn. - Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new DiscardTargetEffect(1), new ManaCostsImpl("{1}{B}")); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new DiscardTargetEffect(1), new ManaCostsImpl("{1}{B}"), MyTurnCondition.getInstance()); ability.addCost(new TapSourceCost()); - ability.addCost(new OnlyDuringYourTurnCost()); - ability.addTarget(new TargetPlayer()); + ability.addTarget(new TargetPlayer(true)); this.addAbility(ability); } diff --git a/Mage.Sets/src/mage/sets/lorwyn/HoofprintsOfTheStag.java b/Mage.Sets/src/mage/sets/lorwyn/HoofprintsOfTheStag.java index f06991d3ece..6ef2ef97082 100644 --- a/Mage.Sets/src/mage/sets/lorwyn/HoofprintsOfTheStag.java +++ b/Mage.Sets/src/mage/sets/lorwyn/HoofprintsOfTheStag.java @@ -32,8 +32,6 @@ import mage.constants.Rarity; import mage.MageInt; import mage.abilities.Ability; import mage.abilities.common.DrawCardTriggeredAbility; -import mage.abilities.common.SimpleActivatedAbility; -import mage.abilities.costs.common.OnlyDuringYourTurnCost; import mage.abilities.costs.common.RemoveCountersSourceCost; import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.effects.common.CreateTokenEffect; @@ -45,6 +43,8 @@ import mage.counters.CounterType; import mage.game.permanent.token.Token; import java.util.UUID; +import mage.abilities.common.ActivateIfConditionActivatedAbility; +import mage.abilities.condition.common.MyTurnCondition; /** * @@ -59,9 +59,8 @@ public class HoofprintsOfTheStag extends CardImpl { this.subtype.add("Elemental"); this.color.setWhite(true); this.addAbility(new DrawCardTriggeredAbility(new AddCountersSourceEffect(CounterType.HOOFPRINT.createInstance(1)), true)); - Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new CreateTokenEffect(new WhiteElementalToken(), 1), new ManaCostsImpl("{2}{W}")); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new CreateTokenEffect(new WhiteElementalToken(), 1), new ManaCostsImpl("{2}{W}"), MyTurnCondition.getInstance()); ability.addCost(new RemoveCountersSourceCost(CounterType.HOOFPRINT.createInstance(4))); - ability.addCost(new OnlyDuringYourTurnCost()); this.addAbility(ability); } diff --git a/Mage.Sets/src/mage/sets/magic2012/SundialOfTheInfinite.java b/Mage.Sets/src/mage/sets/magic2012/SundialOfTheInfinite.java index d3ac1d9242c..30982fcaf8d 100644 --- a/Mage.Sets/src/mage/sets/magic2012/SundialOfTheInfinite.java +++ b/Mage.Sets/src/mage/sets/magic2012/SundialOfTheInfinite.java @@ -34,8 +34,8 @@ import mage.constants.Zone; import mage.constants.CardType; import mage.constants.Rarity; import mage.abilities.Ability; -import mage.abilities.common.SimpleActivatedAbility; -import mage.abilities.costs.common.OnlyDuringYourTurnCost; +import mage.abilities.common.ActivateIfConditionActivatedAbility; +import mage.abilities.condition.common.MyTurnCondition; import mage.abilities.costs.common.TapSourceCost; import mage.abilities.costs.mana.GenericManaCost; import mage.cards.CardImpl; @@ -52,9 +52,8 @@ public class SundialOfTheInfinite extends CardImpl { this.expansionSetCode = "M12"; // {1}, {tap}: End the turn. Activate this ability only during your turn. - Ability ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new EndTurnEffect(), new GenericManaCost(1)); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new EndTurnEffect(), new GenericManaCost(1), MyTurnCondition.getInstance()); ability.addCost(new TapSourceCost()); - ability.addCost(new OnlyDuringYourTurnCost()); this.addAbility(ability); } diff --git a/Mage.Sets/src/mage/sets/seventhedition/DisruptingScepter.java b/Mage.Sets/src/mage/sets/seventhedition/DisruptingScepter.java index 0e93cf3d919..707c054de60 100644 --- a/Mage.Sets/src/mage/sets/seventhedition/DisruptingScepter.java +++ b/Mage.Sets/src/mage/sets/seventhedition/DisruptingScepter.java @@ -32,7 +32,8 @@ import java.util.UUID; import mage.constants.CardType; import mage.constants.Rarity; import mage.abilities.Ability; -import mage.abilities.common.ActivateAsSorceryActivatedAbility; +import mage.abilities.common.ActivateIfConditionActivatedAbility; +import mage.abilities.condition.common.MyTurnCondition; import mage.abilities.costs.common.TapSourceCost; import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.effects.common.DiscardTargetEffect; @@ -51,8 +52,8 @@ public class DisruptingScepter extends CardImpl { this.expansionSetCode = "7ED"; // {3}, {tap}: Target player discards a card. Activate this ability only during your turn. - Ability ability = new ActivateAsSorceryActivatedAbility(Zone.BATTLEFIELD, new DiscardTargetEffect(1), new ManaCostsImpl("{3}")); - ability.addTarget(new TargetPlayer()); + Ability ability = new ActivateIfConditionActivatedAbility(Zone.BATTLEFIELD, new DiscardTargetEffect(1), new ManaCostsImpl("{3}"), new MyTurnCondition()); + ability.addTarget(new TargetPlayer(true)); ability.addCost(new TapSourceCost()); this.addAbility(ability); } diff --git a/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java b/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java index eb375eb8087..dd365ad1fde 100644 --- a/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java +++ b/Mage/src/mage/abilities/common/ActivateIfConditionActivatedAbility.java @@ -29,11 +29,11 @@ package mage.abilities.common; import java.util.UUID; -import mage.constants.Zone; import mage.abilities.ActivatedAbilityImpl; import mage.abilities.condition.Condition; import mage.abilities.costs.Cost; import mage.abilities.effects.Effect; +import mage.constants.Zone; import mage.game.Game; /** @@ -80,7 +80,16 @@ public class ActivateIfConditionActivatedAbility extends ActivatedAbilityImpl { - - public OnlyDuringYourTurnCost() { - text = "Activate this ability only during your turn"; - } - - public OnlyDuringYourTurnCost(final OnlyDuringYourTurnCost cost) { - super(cost); - } - - @Override - public OnlyDuringYourTurnCost copy() { - return new OnlyDuringYourTurnCost(this); - } - - @Override - public boolean canPay(UUID sourceId, UUID controllerId, Game game) { - return game.getActivePlayerId().equals(controllerId); - } - - @Override - public boolean pay(Ability ability, Game game, UUID sourceId, UUID controllerId, boolean noMana) { - this.paid = true; - return paid; - } - -}