From bc4ee13ca5cea37b35badb201943550a4fa46998 Mon Sep 17 00:00:00 2001 From: jje4th Date: Fri, 29 Apr 2016 20:17:44 +0000 Subject: [PATCH] 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) --- .../src/main/java/forge/game/trigger/TriggerChangesZone.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/forge-game/src/main/java/forge/game/trigger/TriggerChangesZone.java b/forge-game/src/main/java/forge/game/trigger/TriggerChangesZone.java index a8b1feee7f8..58134b68418 100644 --- a/forge-game/src/main/java/forge/game/trigger/TriggerChangesZone.java +++ b/forge-game/src/main/java/forge/game/trigger/TriggerChangesZone.java @@ -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 // no cards with multiple different OncePerEffect triggers. SpellAbilityStackInstance si = (SpellAbilityStackInstance) runParams2.get("SpellAbilityStackInstance"); - return si != null && si.attemptOncePerEffectTrigger(this.getHostCard()); + return si == null || si.attemptOncePerEffectTrigger(this.getHostCard()); } return true;