From 99cfd8653310990e86d38d2265c37ff7ed925761 Mon Sep 17 00:00:00 2001
From: LevelX2 <ludwig.hirth@online.de>
Date: Sun, 24 Jun 2018 18:11:17 +0200
Subject: [PATCH] * Some more fixes for ManaOptions #5023

---
 Mage.Sets/src/mage/cards/n/NykthosShrineToNyx.java          | 6 +++---
 .../abilities/mana/CommanderColorIdentityManaAbility.java   | 6 ++++--
 .../java/mage/abilities/mana/ConditionalManaAbility.java    | 4 +---
 Mage/src/main/java/mage/abilities/mana/ManaOptions.java     | 2 +-
 .../main/java/mage/abilities/mana/SimpleManaAbility.java    | 4 ++--
 .../main/java/mage/abilities/mana/TriggeredManaAbility.java | 3 +--
 6 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/Mage.Sets/src/mage/cards/n/NykthosShrineToNyx.java b/Mage.Sets/src/mage/cards/n/NykthosShrineToNyx.java
index 67453e05318b..6b08d876413d 100644
--- a/Mage.Sets/src/mage/cards/n/NykthosShrineToNyx.java
+++ b/Mage.Sets/src/mage/cards/n/NykthosShrineToNyx.java
@@ -66,11 +66,11 @@ public NykthosShrineToNyxManaAbility copy() {
 
     @Override
     public List<Mana> getNetMana(Game game) {
-        netMana.clear();
+        ArrayList<Mana> netManaCopy = new ArrayList<>();
         if (game != null) {
-            netMana.addAll(((ManaEffect) this.getEffects().get(0)).getNetMana(game, this));
+            netManaCopy.addAll(((ManaEffect) this.getEffects().get(0)).getNetMana(game, this));
         }
-        return netMana;
+        return netManaCopy;
     }
 }
 
diff --git a/Mage/src/main/java/mage/abilities/mana/CommanderColorIdentityManaAbility.java b/Mage/src/main/java/mage/abilities/mana/CommanderColorIdentityManaAbility.java
index 054f31593690..44ad62ff2322 100644
--- a/Mage/src/main/java/mage/abilities/mana/CommanderColorIdentityManaAbility.java
+++ b/Mage/src/main/java/mage/abilities/mana/CommanderColorIdentityManaAbility.java
@@ -1,6 +1,6 @@
-
 package mage.abilities.mana;
 
+import java.util.ArrayList;
 import java.util.List;
 import java.util.UUID;
 import mage.Mana;
@@ -42,6 +42,7 @@ public CommanderColorIdentityManaAbility copy() {
 
     @Override
     public List<Mana> getNetMana(Game game) {
+        List<Mana> netManas = new ArrayList<>();
         if (netMana.isEmpty() && game != null) {
             Player controller = game.getPlayer(getControllerId());
             if (controller != null) {
@@ -68,7 +69,8 @@ public List<Mana> getNetMana(Game game) {
                 }
             }
         }
-        return netMana;
+        netManas.addAll(netMana);
+        return netManas;
     }
 
     @Override
diff --git a/Mage/src/main/java/mage/abilities/mana/ConditionalManaAbility.java b/Mage/src/main/java/mage/abilities/mana/ConditionalManaAbility.java
index 06c2919d1754..c2d0a97511fb 100644
--- a/Mage/src/main/java/mage/abilities/mana/ConditionalManaAbility.java
+++ b/Mage/src/main/java/mage/abilities/mana/ConditionalManaAbility.java
@@ -33,8 +33,6 @@ public ConditionalManaAbility copy() {
 
     @Override
     public List<Mana> getNetMana(Game game) {
-        List<Mana> newNetMana = new ArrayList<>();
-        newNetMana.addAll(conditionalManaEffect.getNetMana(game, this));
-        return newNetMana;
+        return new ArrayList<>(conditionalManaEffect.getNetMana(game, this));
     }
 }
diff --git a/Mage/src/main/java/mage/abilities/mana/ManaOptions.java b/Mage/src/main/java/mage/abilities/mana/ManaOptions.java
index 6633be31e8a3..81792fc8c9ac 100644
--- a/Mage/src/main/java/mage/abilities/mana/ManaOptions.java
+++ b/Mage/src/main/java/mage/abilities/mana/ManaOptions.java
@@ -53,7 +53,7 @@ public void addMana(List<ActivatedManaAbilityImpl> abilities, Game game) {
                     boolean hasTapCost = hasTapCost(abilities.get(0));
                     for (Mana netMana : netManas) {
                         for (Mana mana : copy) {
-                            if (!hasTapCost || checkTappedForManaReplacement(abilities.get(0), game, netMana)) {
+                            if (!hasTapCost /* || checkTappedForManaReplacement(abilities.get(0), game, netMana) */) { // Seems to produce endless iterations so deactivated for now:  https://github.com/magefree/mage/issues/5023
                                 Mana newMana = new Mana();
                                 newMana.add(mana);
                                 newMana.add(netMana);
diff --git a/Mage/src/main/java/mage/abilities/mana/SimpleManaAbility.java b/Mage/src/main/java/mage/abilities/mana/SimpleManaAbility.java
index 4998d58b5c58..0877a667c252 100644
--- a/Mage/src/main/java/mage/abilities/mana/SimpleManaAbility.java
+++ b/Mage/src/main/java/mage/abilities/mana/SimpleManaAbility.java
@@ -1,6 +1,6 @@
-
 package mage.abilities.mana;
 
+import java.util.ArrayList;
 import java.util.List;
 import mage.Mana;
 import mage.abilities.costs.Cost;
@@ -55,7 +55,7 @@ public List<Mana> getNetMana(Game game) {
         if (predictable) {
             return super.getNetMana(game);
         }
-        return netMana;
+        return new ArrayList<Mana>(netMana);
     }
 
 }
diff --git a/Mage/src/main/java/mage/abilities/mana/TriggeredManaAbility.java b/Mage/src/main/java/mage/abilities/mana/TriggeredManaAbility.java
index 92054553ac6d..41173683bd10 100644
--- a/Mage/src/main/java/mage/abilities/mana/TriggeredManaAbility.java
+++ b/Mage/src/main/java/mage/abilities/mana/TriggeredManaAbility.java
@@ -1,4 +1,3 @@
-
 package mage.abilities.mana;
 
 import java.util.ArrayList;
@@ -54,7 +53,7 @@ public List<Mana> getNetMana(Game game) {
             }
             return newNetMana;
         }
-        return netMana;
+        return new ArrayList<Mana>(netMana);
     }
 
     /**