From 5cedc3258a999a02f860e69e01c1ada2fe505790 Mon Sep 17 00:00:00 2001
From: Susucre <34709007+Susucre@users.noreply.github.com>
Date: Thu, 24 Aug 2023 12:22:03 +0200
Subject: [PATCH] Cleanup now that server is no longer sending MageObject in
 ClientMessage (#10986)

---
 Mage/src/main/java/mage/abilities/effects/EffectImpl.java | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/Mage/src/main/java/mage/abilities/effects/EffectImpl.java b/Mage/src/main/java/mage/abilities/effects/EffectImpl.java
index c38a5118fce..b3abfdd1e2c 100644
--- a/Mage/src/main/java/mage/abilities/effects/EffectImpl.java
+++ b/Mage/src/main/java/mage/abilities/effects/EffectImpl.java
@@ -4,7 +4,6 @@ import mage.abilities.MageSingleton;
 import mage.abilities.Mode;
 import mage.constants.EffectType;
 import mage.constants.Outcome;
-import mage.players.Player;
 import mage.target.targetpointer.FirstTargetPointer;
 import mage.target.targetpointer.TargetPointer;
 
@@ -105,12 +104,6 @@ public abstract class EffectImpl implements Effect {
                 values = new HashMap<>();
             }
         }
-        if (value instanceof Player) {
-            // If Player are set as value, there might be PlayerImpl serialized in ClientMessage's GameView.
-            // That does cause the message's data to not be unzippable, since the PlayerImpl class are not
-            // client-side.
-            throw new IllegalArgumentException("Players should not be set as value, set the UUID instead.");
-        }
         values.put(key, value);
     }