From 13b1f01e3b7ec10df2608b77b2f4f2d9c22a065e Mon Sep 17 00:00:00 2001
From: Hanmac <hanmac@cardforge.org>
Date: Sat, 30 Jul 2016 19:01:34 +0000
Subject: [PATCH] AiCostDescision: add collection for discarded and tapped, to
 prevent the ai from using the same card twice

---
 .../src/main/java/forge/ai/AiCostDecision.java | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/forge-ai/src/main/java/forge/ai/AiCostDecision.java b/forge-ai/src/main/java/forge/ai/AiCostDecision.java
index 2ad507feb9d..69a93d90230 100644
--- a/forge-ai/src/main/java/forge/ai/AiCostDecision.java
+++ b/forge-ai/src/main/java/forge/ai/AiCostDecision.java
@@ -27,10 +27,16 @@ public class AiCostDecision extends CostDecisionMakerBase {
     private final SpellAbility ability;
     private final Card source;
     
+    private final CardCollection discarded;
+    private final CardCollection tapped;
+
     public AiCostDecision(Player ai0, SpellAbility sa) {
         super(ai0);
         ability = sa;
         source = ability.getHostCard();
+
+        discarded = new CardCollection();
+        tapped = new CardCollection();
     }
 
     @Override
@@ -90,7 +96,12 @@ public class AiCostDecision extends CostDecisionMakerBase {
         }
         else {
             final AiController aic = ((PlayerControllerAi)player.getController()).getAi();
-            return PaymentDecision.card(aic.getCardsToDiscard(c, type.split(";"), ability));
+
+            CardCollection result = aic.getCardsToDiscard(c, type.split(";"), ability, discarded);
+            if (result != null) {
+                discarded.addAll(result);
+            }
+            return PaymentDecision.card(result);
         }
     }
 
@@ -396,14 +407,13 @@ public class AiCostDecision extends CostDecisionMakerBase {
             return null;
         }
 
-        CardCollectionView totap = ComputerUtil.chooseTapType(player, cost.getType(), source, !cost.canTapSource, c);
-
+        CardCollectionView totap = ComputerUtil.chooseTapType(player, cost.getType(), source, !cost.canTapSource, c, tapped);
 
         if (totap == null) {
             System.out.println("Couldn't find a valid card to tap for: " + source.getName());
             return null;
         }
-
+        tapped.addAll(totap);
         return PaymentDecision.card(totap);
     }