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

* Fixed a bug that a dies ability did still trigger as the stack was not cleared meanwhile.

Browse source
This commit is contained in:
LevelX2 2015-11-28 12:52:13 +01:00
parent 732d0f86a1
commit d94a2712ba
2 changed files with 33 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

29
Mage.Tests/src/test/java/org/mage/test/cards/triggers/dies/OmnathLocusOfRageTest.java
View file

@ -66,4 +66,33 @@ public class OmnathLocusOfRageTest extends CardTestPlayerBase {
} }
@Test
public void testDiesTriggeredAbilityOnlyIfPresent() {
// <i>Landfall</i> - Whenever a land enters the battlefield under your control, put a 5/5 red and green Elemental creature token onto the battlefield.
// Whenever Omnath, Locus of Rage or another Elemental you control dies, Omnath deals 3 damage to target creature or player.
addCard(Zone.BATTLEFIELD, playerA, "Omnath, Locus of Rage", 1);
addCard(Zone.BATTLEFIELD, playerA, "Lightning Elemental", 1); // 4/1 Elemental - Haste
// Blastfire Bolt deals 5 damage to target creature. Destroy all Equipment attached to that creature.
addCard(Zone.HAND, playerB, "Blastfire Bolt", 1); // {5}{R}
addCard(Zone.HAND, playerB, "Lightning Bolt", 1); // {R}
addCard(Zone.BATTLEFIELD, playerB, "Mountain", 7);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerB, "Lightning Bolt", "Lightning Elemental"); // Dying Lightning Elemental does no longer trigger ability of Omnath
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerB, "Blastfire Bolt", "Omnath, Locus of Rage", "Lightning Bolt");
addTarget(playerA, playerB);
addTarget(playerA, playerB);
setStopAt(1, PhaseStep.BEGIN_COMBAT);
execute();
assertGraveyardCount(playerB, "Lightning Bolt", 1);
assertGraveyardCount(playerB, "Blastfire Bolt", 1);
assertGraveyardCount(playerA, "Omnath, Locus of Rage", 1);
assertGraveyardCount(playerA, "Lightning Elemental", 1);
assertLife(playerA, 20);
assertLife(playerB, 17);
}
} }

6
Mage/src/mage/abilities/common/DiesThisOrAnotherCreatureTriggeredAbility.java
View file

@ -70,11 +70,13 @@ public class DiesThisOrAnotherCreatureTriggeredAbility extends TriggeredAbilityI
@Override @Override
public boolean isInUseableZone(Game game, MageObject source, GameEvent event) { public boolean isInUseableZone(Game game, MageObject source, GameEvent event) {
Permanent sourcePermanent; Permanent sourcePermanent = null;
if (game.getState().getZone(getSourceId()) == Zone.BATTLEFIELD) { if (game.getState().getZone(getSourceId()) == Zone.BATTLEFIELD) {
sourcePermanent = game.getPermanent(getSourceId()); sourcePermanent = game.getPermanent(getSourceId());
} else { } else {
sourcePermanent = (Permanent) game.getLastKnownInformation(getSourceId(), Zone.BATTLEFIELD); if (game.getShortLivingLKI(sourceId, Zone.BATTLEFIELD)) {
sourcePermanent = (Permanent) game.getLastKnownInformation(getSourceId(), Zone.BATTLEFIELD);
}
} }
if (sourcePermanent == null) { if (sourcePermanent == null) {
return false; return false;