Check for Blank OfflinePlayer Names - the internal representation does not allow them, so lets catch it early when the issue lies with the user.

This commit is contained in:
md_5 2014-04-14 17:22:57 +10:00
parent 3e2a7bb17c
commit 72d5f50bb7

View file

@ -0,0 +1,21 @@
From ffc8031afadcab95da7bf7991fa4903265de4e77 Mon Sep 17 00:00:00 2001
From: md_5 <git@md-5.net>
Date: Mon, 14 Apr 2014 17:21:24 +1000
Subject: [PATCH] Check for blank OfflinePlayer Names
diff --git a/src/main/java/org/bukkit/craftbukkit/CraftServer.java b/src/main/java/org/bukkit/craftbukkit/CraftServer.java
index c30197b..9f0517b 100644
--- a/src/main/java/org/bukkit/craftbukkit/CraftServer.java
+++ b/src/main/java/org/bukkit/craftbukkit/CraftServer.java
@@ -1304,6 +1304,7 @@ public final class CraftServer implements Server {
public OfflinePlayer getOfflinePlayer(String name) {
Validate.notNull(name, "Name cannot be null");
+ com.google.common.base.Preconditions.checkArgument( !org.apache.commons.lang.StringUtils.isBlank( name ), "Name cannot be blank" ); // Spigot
OfflinePlayer result = getPlayerExact(name);
if (result == null) {
--
1.8.3.2