summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorElgarL <ElgarL@Palmergames.com>2014-06-20 08:51:41 +0100
committerElgarL <ElgarL@Palmergames.com>2014-06-20 08:51:41 +0100
commitd86d72eeb75593c3125f876a8cfcf199eba12352 (patch)
tree84d717616a691954ddadb15d914707422454b1ae
parentf336e54b67b585186a83735f15d6d7f73decc522 (diff)
downloadEssentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar
Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.gz
Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.lz
Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.tar.xz
Essentials-d86d72eeb75593c3125f876a8cfcf199eba12352.zip
Prevent NPEs on new user creation.
-rw-r--r--EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java26
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());
}
-
}