Skip to content

Commit 9ec0ca2

Browse files
committed
User loading cleanup
1 parent b0e1655 commit 9ec0ca2

File tree

2 files changed

+7
-16
lines changed

2 files changed

+7
-16
lines changed

Essentials/src/com/earth2me/essentials/Essentials.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -632,12 +632,14 @@ public User getUser(final Object base)
632632
return null;
633633
}
634634

635+
//This will return null if there is not a match.
635636
@Override
636637
public User getUser(final String base)
637638
{
638639
return getOfflineUser((String)base);
639640
}
640641

642+
//This will return null if there is not a match.
641643
@Override
642644
public User getOfflineUser(final String name)
643645
{
@@ -649,6 +651,7 @@ public User getOfflineUser(final String name)
649651
return user;
650652
}
651653

654+
//This will create a new user if there is not a match.
652655
@Override
653656
public User getUser(final Player base)
654657
{

Essentials/src/com/earth2me/essentials/UserMap.java

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -78,24 +78,12 @@ public User getUser(final String name)
7878
}
7979

8080
@Override
81-
public User load(final String name) throws Exception
81+
public User load(final String sanitizedName) throws Exception
8282
{
83-
String sanitizedName = StringUtil.sanitizeFileName(name);
84-
if (!sanitizedName.equals(name))
85-
{
86-
User user = getUser(sanitizedName);
87-
if (user == null)
88-
{
89-
throw new Exception("User not found!");
90-
}
91-
else
92-
{
93-
return user;
94-
}
95-
}
9683
for (Player player : ess.getServer().getOnlinePlayers())
97-
{
98-
if (player.getName().equalsIgnoreCase(name))
84+
{
85+
String sanitizedPlayer = StringUtil.sanitizeFileName(player.getName());
86+
if (sanitizedPlayer.equalsIgnoreCase(sanitizedName))
9987
{
10088
keys.add(sanitizedName);
10189
return new User(player, ess);

0 commit comments

Comments
 (0)