From 176eb0236d73fe429e7d893a01abe343a3361ba7 Mon Sep 17 00:00:00 2001
From: swordshine <swordshine@cardforge.org>
Date: Sat, 14 Jun 2014 13:07:09 +0000
Subject: [PATCH] - Fixed Dovescape and friends

---
 .../java/forge/game/ability/effects/CounterEffect.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/forge-game/src/main/java/forge/game/ability/effects/CounterEffect.java b/forge-game/src/main/java/forge/game/ability/effects/CounterEffect.java
index 6b04f6b8cc4..d54f9b44116 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/CounterEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/CounterEffect.java
@@ -94,6 +94,10 @@ public class CounterEffect extends SpellAbilityEffect {
 
         for (final SpellAbility tgtSA : sas) {
             final Card tgtSACard = tgtSA.getHostCard();
+            // should remember even that spell cannot be countered, e.g. Dovescape
+            if (sa.hasParam("RememberCounteredCMC")) {
+                sa.getHostCard().addRemembered((Integer) tgtSACard.getCMC());
+            }
 
             if (tgtSA.isSpell() && !CardFactoryUtil.isCounterableBy(tgtSACard, sa)) {
                 continue;
@@ -108,10 +112,6 @@ public class CounterEffect extends SpellAbilityEffect {
                 continue;
             }
 
-            if (sa.hasParam("RememberCounteredCMC")) {
-                sa.getHostCard().addRemembered((Integer) tgtSACard.getCMC());
-            }
-
             this.removeFromStack(tgtSA, sa, si);
 
             // Destroy Permanent may be able to be turned into a SubAbility