External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-22 19:41:59 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixed Vizkopa Guildmage and Grisly Spectacle.

Browse source
This commit is contained in:
LevelX2 2013-02-01 08:37:37 +01:00
parent 271872cfe5
commit 0053bfda2e
2 changed files with 25 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Mage.Sets/src/mage/sets/gatecrash/GrislySpectacle.java
View file

@ -53,7 +53,7 @@ public class GrislySpectacle extends CardImpl<GrislySpectacle> {
private static final FilterCreaturePermanent filter = new FilterCreaturePermanent("nonartifact creature"); private static final FilterCreaturePermanent filter = new FilterCreaturePermanent("nonartifact creature");
static { static {
filter.add(Predicates.not(new CardTypePredicate(CardType.CREATURE))); filter.add(Predicates.not(new CardTypePredicate(CardType.ARTIFACT)));
} }
public GrislySpectacle (UUID ownerId) { public GrislySpectacle (UUID ownerId) {

40
Mage.Sets/src/mage/sets/gatecrash/VizkopaGuildmage.java
View file

@ -29,6 +29,7 @@
package mage.sets.gatecrash; package mage.sets.gatecrash;
import java.util.UUID; import java.util.UUID;
import mage.Constants;
import mage.Constants.CardType; import mage.Constants.CardType;
import mage.Constants.Duration; import mage.Constants.Duration;
import mage.Constants.Outcome; import mage.Constants.Outcome;
@ -36,12 +37,11 @@ import mage.Constants.Rarity;
import mage.Constants.Zone; import mage.Constants.Zone;
import mage.MageInt; import mage.MageInt;
import mage.abilities.Ability; import mage.abilities.Ability;
import mage.abilities.TriggeredAbilityImpl; import mage.abilities.DelayedTriggeredAbility;
import mage.abilities.common.SimpleActivatedAbility; import mage.abilities.common.SimpleActivatedAbility;
import mage.abilities.costs.mana.ManaCostsImpl; import mage.abilities.costs.mana.ManaCostsImpl;
import mage.abilities.effects.OneShotEffect; import mage.abilities.effects.OneShotEffect;
import mage.abilities.effects.common.DestroyTargetEffect; import mage.abilities.effects.common.CreateDelayedTriggeredAbilityEffect;
import mage.abilities.effects.common.continious.GainAbilitySourceEffect;
import mage.abilities.effects.common.continious.GainAbilityTargetEffect; import mage.abilities.effects.common.continious.GainAbilityTargetEffect;
import mage.abilities.keyword.LifelinkAbility; import mage.abilities.keyword.LifelinkAbility;
import mage.cards.CardImpl; import mage.cards.CardImpl;
@ -49,9 +49,17 @@ import mage.game.Game;
import mage.game.events.GameEvent; import mage.game.events.GameEvent;
import mage.players.Player; import mage.players.Player;
import mage.target.common.TargetCreaturePermanent; import mage.target.common.TargetCreaturePermanent;
import mage.target.common.TargetNonlandPermanent;
/** /**
* Gatecrash FAQ (01.2013)
* Multiple instances of lifelink are redundant. Giving the same creature lifelink
* more than once won't cause you to gain additional life.
*
* Each time the second ability resolves, a delayed triggered ability is created.
* Whenever you gain life that turn, each of those abilities will trigger. For
* example, if you activate the second ability twice (and let those abilities resolve)
* and then you gain 2 life, each opponent will lose a total of 4 life. Each instance
* will cause two abilities to trigger, each causing that player to lose 2 life.
* *
* @author LevelX2 * @author LevelX2
*/ */
@ -76,7 +84,8 @@ public class VizkopaGuildmage extends CardImpl<VizkopaGuildmage> {
this.addAbility(ability); this.addAbility(ability);
// 1{W}{B}: Whenever you gain life this turn, each opponent loses that much life. // 1{W}{B}: Whenever you gain life this turn, each opponent loses that much life.
this.addAbility(new SimpleActivatedAbility(Zone.BATTLEFIELD, new GainAbilitySourceEffect(new VizkopaGuildmageTriggeredAbility(), Duration.EndOfTurn), new ManaCostsImpl("{1}{W}{B}"))); this.addAbility(new SimpleActivatedAbility(Zone.BATTLEFIELD, new CreateDelayedTriggeredAbilityEffect(new VizkopaGuildmageDelayedTriggeredAbility()), new ManaCostsImpl("{1}{W}{B}")));
} }
public VizkopaGuildmage(final VizkopaGuildmage card) { public VizkopaGuildmage(final VizkopaGuildmage card) {
@ -89,22 +98,16 @@ public class VizkopaGuildmage extends CardImpl<VizkopaGuildmage> {
} }
} }
class VizkopaGuildmageTriggeredAbility extends TriggeredAbilityImpl<VizkopaGuildmageTriggeredAbility> { class VizkopaGuildmageDelayedTriggeredAbility extends DelayedTriggeredAbility<VizkopaGuildmageDelayedTriggeredAbility> {
public VizkopaGuildmageDelayedTriggeredAbility() {
public VizkopaGuildmageTriggeredAbility() { super(new OpponentsLoseLifeEffect(), Constants.Duration.EndOfTurn, false);
super(Zone.BATTLEFIELD, new OpponentsLoseLifeEffect());
} }
public VizkopaGuildmageTriggeredAbility(final VizkopaGuildmageTriggeredAbility ability) { public VizkopaGuildmageDelayedTriggeredAbility(VizkopaGuildmageDelayedTriggeredAbility ability) {
super(ability); super(ability);
} }
@Override
public VizkopaGuildmageTriggeredAbility copy() {
return new VizkopaGuildmageTriggeredAbility(this);
}
@Override @Override
public boolean checkTrigger(GameEvent event, Game game) { public boolean checkTrigger(GameEvent event, Game game) {
if (event.getType() == GameEvent.EventType.GAINED_LIFE && event.getTargetId().equals(controllerId)) { if (event.getType() == GameEvent.EventType.GAINED_LIFE && event.getTargetId().equals(controllerId)) {
@ -114,9 +117,14 @@ class VizkopaGuildmageTriggeredAbility extends TriggeredAbilityImpl<VizkopaGuild
return false; return false;
} }
@Override
public VizkopaGuildmageDelayedTriggeredAbility copy() {
return new VizkopaGuildmageDelayedTriggeredAbility(this);
}
@Override @Override
public String getRule() { public String getRule() {
return "Whenever you gain life this turn, " + super.getRule(); return "Whenever you gain life this turn, " + modes.getText();
} }
} }