diff options
author | ElgarL <ElgarL@Palmergames.com> | 2014-06-20 08:51:41 +0100 |
---|---|---|
committer | ElgarL <ElgarL@Palmergames.com> | 2014-06-20 08:51:41 +0100 |
commit | d86d72eeb75593c3125f876a8cfcf199eba12352 (patch) | |
tree | 84d717616a691954ddadb15d914707422454b1ae /EssentialsGroupManager | |
parent | f336e54b67b585186a83735f15d6d7f73decc522 (diff) | |
download | Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.gz Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.lz Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.xz Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.zip |
Prevent NPEs on new user creation.
Diffstat (limited to 'EssentialsGroupManager')
-rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java | 26 |
1 files changed, 15 insertions, 11 deletions
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java index 3b0132617..4a1d6506a 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java @@ -162,22 +162,26 @@ public class WorldDataHolder { } // Search for a LastName match - for (String uid : getUUIDLookup(currentName)) { - - User usr = getUsers().get(uid); + + if (nameToUUIDLookup.containsKey(currentName)) { - if (usr.getLastName().equalsIgnoreCase(currentName) && usr.getUUID().equalsIgnoreCase(usr.getLastName())) { + for (String uid : getUUIDLookup(currentName)) { - // Clone this user so we can set it's uUID - user = usr.clone(uUID, currentName); + User usr = getUsers().get(uid); - // Delete it and replace with the new clone. - this.removeUser(usr.getUUID()); - this.addUser(user); + if (usr.getLastName().equalsIgnoreCase(currentName) && usr.getUUID().equalsIgnoreCase(usr.getLastName())) { + + // Clone this user so we can set it's uUID + user = usr.clone(uUID, currentName); + + // Delete it and replace with the new clone. + this.removeUser(usr.getUUID()); + this.addUser(user); + + return getUsers().get(uUID.toLowerCase()); + } - return getUsers().get(uUID.toLowerCase()); } - } |