Implement getHome(s)

Essentials/src/net/ess3/permissions/NoCommandCostPermissions.java

@@ -1,9 +1,9 @@
 package net.ess3.permissions;
 
-import net.ess3.api.IPermission;
 import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
+import net.ess3.api.IPermission;
 
 
 public class NoCommandCostPermissions

Essentials/src/net/ess3/settings/SettingsHolder.java

@@ -63,6 +63,7 @@ public class SettingsHolder extends AsyncStorageObjectHolder<Settings> implement
 		return debug.get();
 	}
 	
+	@Override
 	public void setDebug(final boolean set)
 	{
 		debug.set(set);

Essentials/src/net/ess3/user/User.java

@@ -1,14 +1,5 @@
 package net.ess3.user;
 
-import net.ess3.Console;
-import static net.ess3.I18n._;
-import net.ess3.Teleport;
-import net.ess3.api.*;
-import net.ess3.craftbukkit.InventoryWorkaround;
-import net.ess3.economy.register.Method;
-import net.ess3.permissions.Permissions;
-import net.ess3.utils.DateUtil;
-import net.ess3.utils.Util;
 import java.util.Calendar;
 import java.util.GregorianCalendar;
 import java.util.List;
@@ -18,6 +9,15 @@ import java.util.logging.Logger;
 import lombok.Cleanup;
 import lombok.Getter;
 import lombok.Setter;
+import net.ess3.Console;
+import static net.ess3.I18n._;
+import net.ess3.Teleport;
+import net.ess3.api.*;
+import net.ess3.craftbukkit.InventoryWorkaround;
+import net.ess3.economy.register.Method;
+import net.ess3.permissions.Permissions;
+import net.ess3.utils.DateUtil;
+import net.ess3.utils.Util;
 import org.bukkit.Bukkit;
 import org.bukkit.Location;
 import org.bukkit.Material;
@@ -255,6 +255,7 @@ public class User extends UserBase implements IUser
 		}
 	}
 
+	@Override
 	public void setDisplayNick()
 	{
 		String name = getNick(true);
@@ -366,6 +367,7 @@ public class User extends UserBase implements IUser
 	}
 
 	//Returns true if status expired during this check
+	@Override
 	public boolean checkJailTimeout(final long currentTime)
 	{
 		acquireReadLock();
@@ -535,12 +537,6 @@ public class User extends UserBase implements IUser
 		}
 	}
 	
-	@Override
-	public Location getHome(String name) throws Exception
-	{
-		throw new UnsupportedOperationException("Not supported yet.");
-	}
-
 	@Override
 	public void updateCompass()
 	{
@@ -565,7 +561,7 @@ public class User extends UserBase implements IUser
 	@Override
 	public List<String> getHomes()
 	{
-		throw new UnsupportedOperationException("Not supported yet.");
+		return super.getHomes();
 	}
 
 	@Override
@@ -775,4 +771,10 @@ public class User extends UserBase implements IUser
 		final boolean set = !vanished;
 		this.setVanished(set);
 	}
+
+	@Override
+	public Location getHome(String name) throws Exception
+	{
+		return super.getHome(name);
+	}
 }

Essentials/src/net/ess3/user/UserBase.java

@@ -11,6 +11,8 @@ import net.ess3.storage.Location.WorldNotLoadedException;
 import java.io.File;
 import java.io.IOException;
 import java.util.*;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 import lombok.Cleanup;
 import lombok.Delegate;
 import org.bukkit.Bukkit;
@@ -404,6 +406,30 @@ public abstract class UserBase extends AsyncStorageObjectHolder<UserData> implem
 		}
 	}
 
+	public Location getHome(String name) throws Exception
+	{
+		acquireReadLock();
+		try
+		{
+			if (getData().getHomes() == null)
+			{
+				return null;
+			}
+			try
+			{
+				return getData().getHomes().get(Util.sanitizeFileName(name)).getBukkitLocation();
+			}
+			catch (WorldNotLoadedException ex)
+			{
+				return null;
+			}
+		}
+		finally
+		{
+			unlock();
+		}
+	}
+
 	public Location getHome(Location loc)
 	{
 
@@ -455,4 +481,21 @@ public abstract class UserBase extends AsyncStorageObjectHolder<UserData> implem
 			unlock();
 		}
 	}
+	
+	public List<String> getHomes()
+	{
+		acquireReadLock();
+		try
+		{
+			if (getData().getHomes() == null)
+			{
+				return null;
+			}	
+			return new ArrayList<String>(getData().getHomes().keySet());
+		}
+		finally
+		{
+			unlock();
+		}
+	}
 }