summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorsnowleo <schneeleo@gmail.com>2012-02-14 20:29:45 +0100
committersnowleo <schneeleo@gmail.com>2012-02-14 20:30:06 +0100
commitc475db78a87cdce93f1691a9739cc5b3388d76a0 (patch)
tree6f821759da2f1651e3b65606326678d81c17637f
parent713024c8731d78c8769fc161d4afef868852c710 (diff)
downloadEssentials-c475db78a87cdce93f1691a9739cc5b3388d76a0.tar
Essentials-c475db78a87cdce93f1691a9739cc5b3388d76a0.tar.gz
Essentials-c475db78a87cdce93f1691a9739cc5b3388d76a0.tar.lz
Essentials-c475db78a87cdce93f1691a9739cc5b3388d76a0.tar.xz
Essentials-c475db78a87cdce93f1691a9739cc5b3388d76a0.zip
Don't return null, throw an exception
-rw-r--r--Essentials/src/com/earth2me/essentials/UserMap.java17
1 files changed, 13 insertions, 4 deletions
diff --git a/Essentials/src/com/earth2me/essentials/UserMap.java b/Essentials/src/com/earth2me/essentials/UserMap.java
index f15e5edd9..94b504241 100644
--- a/Essentials/src/com/earth2me/essentials/UserMap.java
+++ b/Essentials/src/com/earth2me/essentials/UserMap.java
@@ -77,8 +77,17 @@ public class UserMap extends CacheLoader<String, User> implements IConf
public User load(final String name) throws Exception
{
String sanitizedName = Util.sanitizeFileName(name);
- if (!sanitizedName.equals(name)) {
- return getUser(sanitizedName);
+ if (!sanitizedName.equals(name))
+ {
+ User user = getUser(sanitizedName);
+ if (user == null)
+ {
+ throw new Exception("User not found!");
+ }
+ else
+ {
+ return user;
+ }
}
for (Player player : ess.getServer().getOnlinePlayers())
{
@@ -119,12 +128,12 @@ public class UserMap extends CacheLoader<String, User> implements IConf
{
return keys.size();
}
-
+
public File getUserFile(final String name)
{
return getUserFile2(Util.sanitizeFileName(name));
}
-
+
private File getUserFile2(final String name)
{
final File userFolder = new File(ess.getDataFolder(), "userdata");