[Bleeding] Don't allow null display names. Fixes BUKKIT-5254

Currently, plugins can set a player's display name to null, which could
cause issues for other calls to getDisplayName that aren't expecting a null
value. This changes setDisplayName to follow the same logic as
setPlayerListName, which sets the player's name back to their unmodified
"vanilla" name if it receives a null value as a parameter.

By: GJ <gjmcferrin@gmail.com>
This commit is contained in:
CraftBukkit/Spigot 2014-01-31 09:42:03 -05:00
parent c9d41122d1
commit 25c4255d7c

View File

@ -169,7 +169,7 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
@Override @Override
public void setDisplayName(final String name) { public void setDisplayName(final String name) {
getHandle().displayName = name; getHandle().displayName = name == null ? getName() : name;
} }
@Override @Override