external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-20 12:48:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

- Experimental: for Dingus Egg, do not check the time stamp of the card that changed zone (fixes interaction with Sacred Ground). Should this be the default behavior for ChangesZone triggers that do not seem to care for how many times the card had changed zones before the trigger resolves?

Browse Source
This commit is contained in:
Agetian
2017-09-13 14:52:09 +00:00
parent 7a92712f0b
commit 0b7fc67c3f
3 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
forge-game/src/main/java/forge/game/trigger/WrappedAbility.java
View File

@@ -451,7 +451,10 @@ public class WrappedAbility extends Ability {
Card card = (Card) o;
Card current = game.getCardState(card);
if (current.getTimestamp() != card.getTimestamp()) {
sa.getTriggerRemembered().remove(o);
// TODO: figure out if NoTimestampCheck should be the default for ChangesZone triggers
if (!triggerParams.containsKey("NoTimestampCheck")) {
sa.getTriggerRemembered().remove(o);
}
}
}
}
@@ -461,7 +464,10 @@ public class WrappedAbility extends Ability {
Card card = (Card) ev.getValue();
Card current = game.getCardState(card);
if (card.isInPlay() && current.isInPlay() && current.getTimestamp() != card.getTimestamp()) {
sa.getTriggeringObjects().remove(ev.getKey());
// TODO: figure out if NoTimestampCheck should be the default for ChangesZone triggers
if (!triggerParams.containsKey("NoTimestampCheck")) {
sa.getTriggeringObjects().remove(ev.getKey());
}
}
}
}