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

The Gitrog Monster was not triggering on discard because of a null check. As far as I can tell the null check was there just to prevent null-ref but indadvertently caused OncePerEffect triggers to fail triggering when the stack is empty (such as when discarding a land card to pay a cost)

Browse Source
This commit is contained in:
jje4th
2016-04-29 20:17:44 +00:00
parent 0040b1eab5
commit bc4ee13ca5
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
forge-game/src/main/java/forge/game/trigger/TriggerChangesZone.java
View File

@@ -138,7 +138,7 @@ public class TriggerChangesZone extends Trigger {
// card, rather than keeping track of the host card itself - but it's good enough for now - since there are // card, rather than keeping track of the host card itself - but it's good enough for now - since there are
// no cards with multiple different OncePerEffect triggers. // no cards with multiple different OncePerEffect triggers.
SpellAbilityStackInstance si = (SpellAbilityStackInstance) runParams2.get("SpellAbilityStackInstance"); SpellAbilityStackInstance si = (SpellAbilityStackInstance) runParams2.get("SpellAbilityStackInstance");
return si != null && si.attemptOncePerEffectTrigger(this.getHostCard()); return si == null || si.attemptOncePerEffectTrigger(this.getHostCard());
} }
return true; return true;