From 7884644a4f639d521e74b052b6df5d4444bcdec7 Mon Sep 17 00:00:00 2001
From: Agetian <stavdev@mail.ru>
Date: Sun, 4 Oct 2015 14:11:28 +0000
Subject: [PATCH] - A more appropriate location for the num of targets test in
 HumanCostDecision (Tap).

---
 forge-gui/src/main/java/forge/player/HumanCostDecision.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/forge-gui/src/main/java/forge/player/HumanCostDecision.java b/forge-gui/src/main/java/forge/player/HumanCostDecision.java
index 07d874d31e9..cd46bbfda4c 100644
--- a/forge-gui/src/main/java/forge/player/HumanCostDecision.java
+++ b/forge-gui/src/main/java/forge/player/HumanCostDecision.java
@@ -1156,10 +1156,11 @@ public class HumanCostDecision extends CostDecisionMakerBase {
             return PaymentDecision.card(inp.getSelected());
         }
 
-        final InputSelectCardsFromList inp = new InputSelectCardsFromList(controller, c, c, typeList);
         if (c > typeList.size()) {
             return null; // not enough targets anymore (e.g. Crackleburr + Smokebraider tapped to get mana)
         }
+
+        final InputSelectCardsFromList inp = new InputSelectCardsFromList(controller, c, c, typeList);
         inp.setCancelAllowed(true);
         inp.setMessage("Select a " + cost.getDescriptiveType() + " to tap (%d left)");
         inp.showAndWait();