TotalFreedomMC/OpenInv
1
0
Fork 0
mirror of https://github.com/TotalFreedomMC/OpenInv.git synced 2024-09-28 04:34:50 +00:00
Code Issues Projects Releases Wiki Activity

Null pointer check

Browse source 
Players who haven't joined your server since the UUID conversion will have a null name. As a result, if you try to /openinv a player who *has* joined your server since then, the loop can break due to a NPE while checking other players. This null check should fix that problem.
This commit is contained in:
Fysac 2014-08-01 13:36:18 -04:00
parent c2c7996da5
commit bbacb04636
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/com/lishid/openinv/internal/v1_7_R4/PlayerDataManager.java
View file

@ -80,7 +80,8 @@ public class PlayerDataManager implements IPlayerDataManager {
OfflinePlayer[] offlinePlayers = Bukkit.getOfflinePlayers(); OfflinePlayer[] offlinePlayers = Bukkit.getOfflinePlayers();
for (OfflinePlayer player : offlinePlayers) { for (OfflinePlayer player : offlinePlayers) {
String name = player.getName(); String name = player.getName();
if (name == null)
continue;
if (name.equalsIgnoreCase(search)) if (name.equalsIgnoreCase(search))
return player.getUniqueId(); return player.getUniqueId();