From 3a5085073489b63104c7bf0ecaab18eeda0afb4b Mon Sep 17 00:00:00 2001
From: KHobbits <rob@khobbits.co.uk>
Date: Sun, 1 Jun 2014 21:50:24 +0100
Subject: Use strings for cachemap key, should prevent accidental key issues.

---
 Essentials/src/com/earth2me/essentials/UserMap.java | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/Essentials/src/com/earth2me/essentials/UserMap.java b/Essentials/src/com/earth2me/essentials/UserMap.java
index 84ae92aee..567c1a0d4 100644
--- a/Essentials/src/com/earth2me/essentials/UserMap.java
+++ b/Essentials/src/com/earth2me/essentials/UserMap.java
@@ -18,14 +18,13 @@ import java.util.concurrent.ConcurrentSkipListMap;
 import java.util.concurrent.ConcurrentSkipListSet;
 import java.util.concurrent.ExecutionException;
 import net.ess3.api.IEssentials;
-import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
 
 
-public class UserMap extends CacheLoader<UUID, User> implements IConf
+public class UserMap extends CacheLoader<String, User> implements IConf
 {
 	private final transient IEssentials ess;
-	private final transient Cache<UUID, User> users;
+	private final transient Cache<String, User> users;
 	private final transient ConcurrentSkipListSet<UUID> keys = new ConcurrentSkipListSet<UUID>();
 	private final transient ConcurrentSkipListMap<String, UUID> names = new ConcurrentSkipListMap<String, UUID>();
 	private final transient ConcurrentSkipListMap<UUID, ArrayList<String>> history = new ConcurrentSkipListMap<UUID, ArrayList<String>>();
@@ -92,7 +91,7 @@ public class UserMap extends CacheLoader<UUID, User> implements IConf
 			if (names.containsKey(sanitizedName))
 			{
 				final UUID uuid = names.get(sanitizedName);
-				return users.get(uuid);
+				return getUser(uuid);
 			}
 			
 			final File userFile = getUserFileFromString(sanitizedName);
@@ -105,10 +104,6 @@ public class UserMap extends CacheLoader<UUID, User> implements IConf
 			}
 			return null;
 		}
-		catch (ExecutionException ex)
-		{
-			return null;
-		}
 		catch (UncheckedExecutionException ex)
 		{
 			return null;
@@ -119,7 +114,7 @@ public class UserMap extends CacheLoader<UUID, User> implements IConf
 	{
 		try
 		{
-			return users.get(uuid);
+			return users.get(uuid.toString());
 		}
 		catch (ExecutionException ex)
 		{
@@ -149,8 +144,9 @@ public class UserMap extends CacheLoader<UUID, User> implements IConf
 	}
 
 	@Override
-	public User load(final UUID uuid) throws Exception
+	public User load(final String stringUUID) throws Exception
 	{
+		UUID uuid = UUID.fromString(stringUUID);
 		Player player = ess.getServer().getPlayer(uuid);
 		if (player != null)
 		{
@@ -206,7 +202,7 @@ public class UserMap extends CacheLoader<UUID, User> implements IConf
 	{
 		return keys.size();
 	}
-
+	
 	protected ConcurrentSkipListMap<String, UUID> getNames()
 	{
 		return names;
-- 
cgit v1.2.3