From 9a55d32b7569d520e3b0365df8a08405be1ac6d8 Mon Sep 17 00:00:00 2001
From: ElgarL <ElgarL@Palmergames.com>
Date: Fri, 20 Jun 2014 10:56:52 +0100
Subject: Convert all User lookups and commands to use UUIDs where possible.

---
 .../groupmanager/dataholder/WorldDataHolder.java   | 35 +++++++++-------------
 1 file changed, 14 insertions(+), 21 deletions(-)

(limited to 'EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder')

diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java
index 4a1d6506a..1e1ae96c3 100644
--- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java
+++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/dataholder/WorldDataHolder.java
@@ -108,8 +108,10 @@ public class WorldDataHolder {
 	}
 
 	/**
-	 * Search for a user. If it doesn't exist, create a new one with default
-	 * group.
+	 * Search for a user. If it doesn't exist, create a new one with default group.
+	 * 
+	 * If this is called passing a player name with mantogglevalidate off
+	 * it can return the wrong user object (offline/online UUID match).
 	 * 
 	 * @param userId the UUID String or name of the user
 	 * @return class that manage that user permission
@@ -162,28 +164,19 @@ public class WorldDataHolder {
 		}
 		
 		// Search for a LastName match
+		user = getUsers().get(currentName.toLowerCase());
 		
-		if (nameToUUIDLookup.containsKey(currentName)) {
+		if ((user != null) && user.getLastName().equalsIgnoreCase(currentName) && user.getUUID().equalsIgnoreCase(user.getLastName())) {
 			
-			for (String uid : getUUIDLookup(currentName)) {
-				
-				User usr = getUsers().get(uid);
-				
-				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());
-				}
-				
-			}
-		}
+			// Clone this user so we can set it's uUID
+			User usr = user.clone(uUID, currentName);
+			
+			// Delete it and replace with the new clone.
+			this.removeUser(user.getUUID());
+			this.addUser(usr);
 			
+			return getUsers().get(uUID.toLowerCase());
+		}
 		
 		// No user account found so create a new one.
 		User newUser = createUser(uUID);
-- 
cgit v1.2.3