From 4a74c936c2134eb4242b8dd708de9d83ab2536a9 Mon Sep 17 00:00:00 2001
From: asvitkine <alexei.svitkine@gmail.com>
Date: Thu, 22 Dec 2022 22:04:30 -0700
Subject: [PATCH] Return null modes when all invalid.

---
 .../java/forge/ai/simulation/SpellAbilityChoicesIterator.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
index 0650c9c98bb..46ef273c6b9 100644
--- a/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
+++ b/forge-ai/src/main/java/forge/ai/simulation/SpellAbilityChoicesIterator.java
@@ -61,8 +61,8 @@ public class SpellAbilityChoicesIterator {
             }
             // TODO: Do we need to do something special to support cards that have extra costs
             // when choosing more modes, like Blessed Alliance?
-            if (modesMap.size() == 0) {
-                modeIterator = Collections.emptyIterator();
+            if (modesMap.isEmpty()) {
+                return null;
             } else if (!allowRepeat) {
                 modeIterator = CombinatoricsUtils.combinationsIterator(modesMap.size(), num);
             } else {