From e09d57562ff829d2c2a292b9e2a8c24feae331ca Mon Sep 17 00:00:00 2001 From: Agetian Date: Sat, 17 Dec 2016 14:35:45 +0000 Subject: [PATCH] - A better idea: for non-targeted AF ChangeZoneAll, just return "true" for now without trying to target it. This allows to keep Paradigm Shift as non-RemAIDeck. --- forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java index a434ff5ee41..9f870cb2f74 100644 --- a/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java +++ b/forge-ai/src/main/java/forge/ai/ability/ChangeZoneAllAi.java @@ -128,8 +128,8 @@ public class ChangeZoneAllAi extends SpellAbilityAi { // spBounceAll has some AI we can compare to. if (origin.equals(ZoneType.Hand) || origin.equals(ZoneType.Library)) { if (!sa.usesTargeting()) { - // TODO: the AI can't currently handle non-targeting abilities of this type (e.g. Memory Jar, Timetwister, etc.) - return false; + // TODO: improve logic for non-targeted SAs of this type (most are currently RemAIDeck, e.g. Memory Jar, Timetwister) + return true; } // search targetable Opponents