Fixed potential packet loss issue with vanish API. Fixes BUKKIT-1125

This commit is contained in:
Kevin Ingersoll 2012-03-10 14:58:20 -08:00 committed by EvilSeph
parent bdd2139c78
commit 49ff166d15

View File

@ -599,6 +599,8 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
} }
public void hidePlayer(Player player) { public void hidePlayer(Player player) {
Validate.notNull(player, "hidden player cannot be null");
if (equals(player)) return;
if (hiddenPlayers.containsKey(player.getName())) return; if (hiddenPlayers.containsKey(player.getName())) return;
hiddenPlayers.put(player.getName(), player); hiddenPlayers.put(player.getName(), player);
@ -615,6 +617,8 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
} }
public void showPlayer(Player player) { public void showPlayer(Player player) {
Validate.notNull(player, "shown player cannot be null");
if (equals(player)) return;
if (!hiddenPlayers.containsKey(player.getName())) return; if (!hiddenPlayers.containsKey(player.getName())) return;
hiddenPlayers.remove(player.getName()); hiddenPlayers.remove(player.getName());