github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-14 19:19:32 +00:00
Code Issues Projects Releases Packages Wiki Activity

* Fixed that the dies trigger of Ashen Rider also triggers if Ashen Rider is copied by other creatures (e.g. Volrath the Shapestealer) (fixes #7107).

Browse source
This commit is contained in:
LevelX2 2020-12-23 18:02:00 +01:00
parent 2daa5aae50
commit 2ce90b1e0a
3 changed files with 77 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

73
Mage.Tests/src/test/java/org/mage/test/cards/triggers/dies/AshenRiderTest.java Normal file
View file

@ -0,0 +1,73 @@
/*
* To change this license header, choose License Headers in Project Properties.
* To change this template file, choose Tools | Templates
* and open the template in the editor.
*/
package org.mage.test.cards.triggers.dies;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
*
* @author LevelX2
*/
public class AshenRiderTest extends CardTestPlayerBase {
/*
* Volrath, the Shapestealer and Ashen Rider, Ashen Rider has a counter on it:
Turn Volrath into Ashen Rider:
Destroy the Volrath (who's the Ashen Rider) with Putrefy:
The death trigger for the Volrath copying Ashen Rider did not trigger.
*/
@Test
public void cartelAristrocraftInteractionOpponentDoesNotPayLife() {
setStrictChooseMode(true);
addCard(Zone.BATTLEFIELD, playerB, "Silvercoat Lion", 1);
addCard(Zone.BATTLEFIELD, playerA, "Forest", 2);
addCard(Zone.BATTLEFIELD, playerA, "Island", 2);
addCard(Zone.BATTLEFIELD, playerA, "Swamp", 3);
// Flying
// When Ashen Rider enters the battlefield or dies, exile target permanent.
addCard(Zone.BATTLEFIELD, playerA, "Ashen Rider"); // Creature {4}{W}{W}{B}{B}
// At the beginning of combat on your turn, put a -1/-1 counter on up to one target creature.
// {1}: Until your next turn, Volrath, the Shapestealer becomes a copy of target creature with a counter on it, except it's 7/5 and it has this ability.
addCard(Zone.HAND, playerA, "Volrath, the Shapestealer"); // Creature {2}{B}{G}{U}
addTarget(playerA, "Ashen Rider");
// Destroy target artifact or creature. It can't be regenerated.
addCard(Zone.HAND, playerA, "Putrefy"); // Instant {1}{B}{G}
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Volrath, the Shapestealer");
activateAbility(3, PhaseStep.PRECOMBAT_MAIN, playerA, "{1}: Until your next turn");
addTarget(playerA, "Ashen Rider");
waitStackResolved(3, PhaseStep.PRECOMBAT_MAIN);
castSpell(3, PhaseStep.PRECOMBAT_MAIN, playerA, "Putrefy", "Ashen Rider[only copy]");
addTarget(playerA, "Silvercoat Lion"); // Dies trigger of Volrath, the Shapestealer copied from Ashen Rider
setStopAt(3, PhaseStep.BEGIN_COMBAT);
execute();
assertAllCommandsUsed();
assertPowerToughness(playerA, "Ashen Rider", 4,4);
assertGraveyardCount(playerA, "Putrefy", 1);
assertGraveyardCount(playerA, "Volrath, the Shapestealer", 1);
assertExileCount(playerB, "Silvercoat Lion", 1);
}
}

4
Mage/src/main/java/mage/abilities/TriggeredAbilityImpl.java
View file

@ -203,7 +203,9 @@ public abstract class TriggeredAbilityImpl extends AbilityImpl implements Trigge
}
/*
603.6c,603.6d
603.6c Leaves-the-battlefield abilities, 603.6d
if true the game “looks back in time” to determine if those abilities trigger,
using the existence of those abilities and the appearance of objects immediately prior to the event (603.10)
*/
@Override
public boolean isLeavesTheBattlefieldTrigger() {

1
Mage/src/main/java/mage/abilities/common/EntersBattlefieldOrDiesSourceTriggeredAbility.java
View file

@ -19,6 +19,7 @@ public class EntersBattlefieldOrDiesSourceTriggeredAbility extends TriggeredAbil
public EntersBattlefieldOrDiesSourceTriggeredAbility(Effect effect, boolean optional) {
super(Zone.BATTLEFIELD, effect, optional);
setLeavesTheBattlefieldTrigger(true);
}
public EntersBattlefieldOrDiesSourceTriggeredAbility(final EntersBattlefieldOrDiesSourceTriggeredAbility ability) {