From 02c41195a8c5d8da44099c720dee65366b3a98f1 Mon Sep 17 00:00:00 2001
From: Sloth <sloth@cardforge.org>
Date: Thu, 21 Mar 2013 22:00:18 +0000
Subject: [PATCH] - Fixed a possible UnmodifiableCollection.remove call.

---
 src/main/java/forge/card/ability/effects/DiscardEffect.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/forge/card/ability/effects/DiscardEffect.java b/src/main/java/forge/card/ability/effects/DiscardEffect.java
index d15ab566e7f..4e2d5d8254b 100644
--- a/src/main/java/forge/card/ability/effects/DiscardEffect.java
+++ b/src/main/java/forge/card/ability/effects/DiscardEffect.java
@@ -165,7 +165,7 @@ public class DiscardEffect extends RevealEffectBase {
                 } else if (mode.equals("RevealYouChoose") || mode.equals("RevealOppChoose") || mode.equals("TgtChoose")) {
                     // Is Reveal you choose right? I think the wrong player is
                     // being used?
-                    List<Card> dPHand = p.getCardsIn(ZoneType.Hand);
+                    List<Card> dPHand = new ArrayList<Card>(p.getCardsIn(ZoneType.Hand));
                     if (dPHand.isEmpty())
                         continue; // for loop over players