aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch')
-rw-r--r--patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch26
1 files changed, 26 insertions, 0 deletions
diff --git a/patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch b/patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch
new file mode 100644
index 0000000000..9a7359c010
--- /dev/null
+++ b/patches/unapplied/server/0271-Dont-block-Player-remove-if-the-handle-is-a-custom-p.patch
@@ -0,0 +1,26 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Zach Brown <[email protected]>
+Date: Mon, 4 Feb 2019 23:33:24 -0500
+Subject: [PATCH] Dont block Player#remove if the handle is a custom player
+
+Upstream throws UOE if you try to call remove on a Player.
+We just add a check to ensure that the CraftPlayer's handle
+is a ServerPlayer
+
+diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+index 5cdd18e071f069adc8137561c164bd533dea7c7c..7c7f66ea38babab7b04e76d14e19cc0dac9c4b54 100644
+--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
++++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+@@ -229,8 +229,12 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
+
+ @Override
+ public void remove() {
++ if (this.getHandle().getClass().equals(ServerPlayer.class)) { // special case for NMS plugins inheriting
+ // Will lead to an inconsistent player state if we remove the player as any other entity.
+ throw new UnsupportedOperationException(String.format("Cannot remove player %s, use Player#kickPlayer(String) instead.", this.getName()));
++ } else {
++ super.remove();
++ }
+ }
+
+ @Override