TotalFreedomMC/TF-EssentialsX
1
0
Fork 0
mirror of https://github.com/TotalFreedomMC/TF-EssentialsX.git synced 2024-09-29 21:35:23 +00:00
Code Issues Projects Releases Wiki Activity

Don't return null, throw an exception

Browse source
This commit is contained in:
snowleo 2012-02-14 20:29:45 +01:00
parent cfdb2a36c1
commit 3c98718387
1 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
Essentials/src/com/earth2me/essentials/UserMap.java
View file

@ -77,8 +77,17 @@ public class UserMap extends CacheLoader<String, User> implements IConf
public User load(final String name) throws Exception public User load(final String name) throws Exception
{ {
String sanitizedName = Util.sanitizeFileName(name); String sanitizedName = Util.sanitizeFileName(name);
if (!sanitizedName.equals(name)) { if (!sanitizedName.equals(name))
return getUser(sanitizedName); {
User user = getUser(sanitizedName);
if (user == null)
{
throw new Exception("User not found!");
}
else
{
return user;
}
} }
for (Player player : ess.getServer().getOnlinePlayers()) for (Player player : ess.getServer().getOnlinePlayers())
{ {
@ -119,12 +128,12 @@ public class UserMap extends CacheLoader<String, User> implements IConf
{ {
return keys.size(); return keys.size();
} }
public File getUserFile(final String name) public File getUserFile(final String name)
{ {
return getUserFile2(Util.sanitizeFileName(name)); return getUserFile2(Util.sanitizeFileName(name));
} }
private File getUserFile2(final String name) private File getUserFile2(final String name)
{ {
final File userFolder = new File(ess.getDataFolder(), "userdata"); final File userFolder = new File(ess.getDataFolder(), "userdata");