Round 1 of fixing getPlayer in commands

Essentials/src/net/ess3/commands/Commandban.java

@@ -20,7 +20,7 @@ public class Commandban extends EssentialsCommand
 			throw new NotEnoughArgumentsException();
 		}
 		@Cleanup
-		final IUser user = getPlayer(args, 0, true);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, true);
 		if (!user.isOnline())
 		{
 			if (sender instanceof Player && Permissions.BAN_OFFLINE.isAuthorized(user))

Essentials/src/net/ess3/commands/Commandclearinventory.java

@@ -34,7 +34,7 @@ public class Commandclearinventory extends EssentialsCommand
 			}
 			else
 			{
-				Player p = server.getPlayer(args[0]);
+				Player p = ess.getUserMap().matchUser(args[0], false, false);
 				if (p != null)
 				{
 					p.getInventory().clear();

Essentials/src/net/ess3/commands/Commanddelhome.java

@@ -37,7 +37,7 @@ public class Commanddelhome extends EssentialsCommand
 
 		if (expandedArg.length > 1 && (user == null || Permissions.DELHOME_OTHERS.isAuthorized(user)))
 		{
-			user = getPlayer(expandedArg, 0, true);
+			user = ess.getUserMap().matchUser(expandedArg[1], false, true);
 			name = expandedArg[1];
 		}
 		else if (user == null)

Essentials/src/net/ess3/commands/Commandeco.java

@@ -88,7 +88,7 @@ public class Commandeco extends EssentialsCommand
 		}
 		else
 		{
-			final IUser player = getPlayer(args, 1, true);
+			final IUser player = ess.getUserMap().matchUser(args[0], true, true);
 			switch (cmd)
 			{
 			case GIVE:

Essentials/src/net/ess3/commands/Commandenchant.java

@@ -68,7 +68,7 @@ public class Commandenchant extends EssentialsCommand
 		{
 			stack.addEnchantment(enchantment, level);
 		}
-		user.getInventory().setItemInHand(stack);
+		user.getInventory().setxItemInHand(stack);
 		user.updateInventory();
 		final String enchantmentName = enchantment.getName().toLowerCase(Locale.ENGLISH);
 		if (level == 0)

Essentials/src/net/ess3/commands/Commandgetpos.java

@@ -15,7 +15,7 @@ public class Commandgetpos extends EssentialsCommand
 		if (args.length > 0 && Permissions.GETPOS_OTHERS.isAuthorized(user))
 		{
 			//todo permissions
-			final IUser otherUser = getPlayer(args, 0);
+			final IUser otherUser = ess.getUserMap().matchUser(args[0], false, false);
 			if (!otherUser.isHidden() || Permissions.LIST_HIDDEN.isAuthorized(user))
 			{
 				outputPosition(user, otherUser.getLocation(), user.getLocation());
@@ -33,7 +33,7 @@ public class Commandgetpos extends EssentialsCommand
 		{
 			throw new NotEnoughArgumentsException();
 		}
-		final IUser user = getPlayer(args, 0);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, false);
 		outputPosition(sender, user.getLocation(), null);
 	}
 

Essentials/src/net/ess3/commands/Commandhome.java

@@ -33,7 +33,7 @@ public class Commandhome extends EssentialsCommand
 			}
 			else
 			{
-				player = getPlayer(nameParts, 0, true);
+				player = ess.getUserMap().matchUser(nameParts[0], false, true);
 				if (nameParts.length > 1)
 				{
 					homeName = nameParts[1];

Essentials/src/net/ess3/commands/Commandignore.java

@@ -16,7 +16,7 @@ public class Commandignore extends EssentialsCommand
 		IUser player;
 		try
 		{
-			player = getPlayer(args, 0);
+			player = ess.getUserMap().matchUser(args[0], false, false);
 		}
 		catch (NoSuchFieldException ex)
 		{

Essentials/src/net/ess3/commands/Commandinvsee.java

@@ -21,7 +21,7 @@ public class Commandinvsee extends EssentialsCommand
 		IUser invUser = user;
 		if (args.length == 1)
 		{
-			invUser = getPlayer(args, 0);
+			invUser = ess.getUserMap().matchUser(args[0], false, false);
 		}
 		user.acquireWriteLock();
 		if (invUser == user && user.getData().getInventory() != null)

Essentials/src/net/ess3/commands/Commandkick.java

@@ -18,7 +18,7 @@ public class Commandkick extends EssentialsCommand
 			throw new NotEnoughArgumentsException();
 		}
 
-		final IUser user = getPlayer(args, 0);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, false);
 		if (Permissions.KICK_EXEMPT.isAuthorized(user))
 		{
 			throw new Exception(_("kickExempt"));

Essentials/src/net/ess3/commands/Commandmsg.java

@@ -62,7 +62,7 @@ public class Commandmsg extends EssentialsCommand
 			return;
 		}
 
-		final List<Player> matchedPlayers = server.matchPlayer(args[0]);
+		final List<Player> matchedPlayers = ess.getUserMap().matchUser(args[0], false, false);
 
 		if (matchedPlayers.isEmpty())
 		{

Essentials/src/net/ess3/commands/Commandmute.java

@@ -20,7 +20,7 @@ public class Commandmute extends EssentialsCommand
 		}
 
 		@Cleanup
-		final IUser player = getPlayer(args, 0, true);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, true);
 		player.acquireReadLock(); 
 		if (!player.getData().isMuted() && Permissions.MUTE_EXEMPT.isAuthorized(player))
 		{

Essentials/src/net/ess3/commands/Commandnear.java

@@ -22,7 +22,7 @@ public class Commandnear extends EssentialsCommand
 		{
 			try
 			{
-				otherUser = getPlayer(args, 0);
+				otherUser = ess.getUserMap().matchUser(args[0], false, false);
 			}
 			catch (Exception ex)
 			{
@@ -62,7 +62,7 @@ public class Commandnear extends EssentialsCommand
 		{
 			throw new NotEnoughArgumentsException();
 		}
-		final IUser otherUser = getPlayer(args, 0);
+		final IUser otherUser = ess.getUserMap().matchUser(args[0], false, false);
 		long radius = 200;
 		if (args.length > 1)
 		{

Essentials/src/net/ess3/commands/Commandnick.java

@@ -34,7 +34,7 @@ public class Commandnick extends EssentialsCommand
 			{
 				throw new Exception(_("nickOthersPermission"));
 			}
-			setNickname(getPlayer(args, 0), formatNickname(user, args[1]));
+			setNickname(ess.getUserMap().matchUser(args[0], false, false), formatNickname(user, args[1]));
 			user.sendMessage(_("nickChanged"));
 			return;
 		}
@@ -61,7 +61,7 @@ public class Commandnick extends EssentialsCommand
 		}
 		else
 		{
-			setNickname(getPlayer(args, 0), formatNickname(null, args[1]));
+			setNickname(ess.getUserMap().matchUser(args[0], false, false), formatNickname(null, args[1]));
 		}
 		sender.sendMessage(_("nickChanged"));
 	}

Essentials/src/net/ess3/commands/Commandseen.java

@@ -31,7 +31,7 @@ public class Commandseen extends EssentialsCommand
 		}
 		try
 		{
-			IUser u = getPlayer(args, 0);
+			IUser u = ess.getUserMap().matchUser(args[0], false, false);
 			sender.sendMessage(_("seenOnline", u.getDisplayName(), DateUtil.formatDateDiff(u.getTimestamp(TimestampType.LOGIN))));
 		}
 		catch (NoSuchFieldException e)

Essentials/src/net/ess3/commands/Commandspawn.java

@@ -20,7 +20,7 @@ public class Commandspawn extends EssentialsCommand
 		charge.isAffordableFor(user);
 		if (args.length > 0 && Permissions.SPAWN_OTHERS.isAuthorized(user))
 		{
-			final IUser otherUser = getPlayer(args, 0);
+			final IUser otherUser = ess.getUserMap().matchUser(args[0], false, false);
 			respawn(otherUser, null);
 			if (!otherUser.equals(user))
 			{
@@ -41,7 +41,7 @@ public class Commandspawn extends EssentialsCommand
 		{
 			throw new NotEnoughArgumentsException();
 		}
-		final IUser user = getPlayer(args, 0);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, false);
 		respawn(user, null);
 		user.sendMessage(_("teleportAtoB", user.getDisplayName(), "spawn"));
 		sender.sendMessage(_("teleporting"));

Essentials/src/net/ess3/commands/Commandsudo.java

@@ -16,7 +16,7 @@ public class Commandsudo extends EssentialsCommand
 			throw new NotEnoughArgumentsException();
 		}
 
-		final IUser user = getPlayer(args, 0, false);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, false);
 		final String command = args[1];
 		final String[] arguments = new String[args.length - 2];
 		if (arguments.length > 0)

Essentials/src/net/ess3/commands/Commandtempban.java

@@ -20,7 +20,7 @@ public class Commandtempban extends EssentialsCommand
 		{
 			throw new NotEnoughArgumentsException();
 		}
-		final IUser user = getPlayer(args, 0, true);
+		final IUser user = ess.getUserMap().matchUser(args[0], false, true);
 		if (!user.isOnline())
 		{
 			if (Permissions.TEMPBAN_OFFLINE.isAuthorized(sender))

Essentials/src/net/ess3/commands/Commandtogglejail.java

@@ -21,7 +21,7 @@ public class Commandtogglejail extends EssentialsCommand
 		}
 
 		@Cleanup
-		final IUser player = getPlayer(args, 0, true);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, true);
 		player.acquireReadLock();
 
 		if (args.length >= 2 && !player.getData().isJailed())

Essentials/src/net/ess3/commands/Commandtp.java

@@ -28,7 +28,7 @@ public class Commandtp extends EssentialsCommand
 
 		case 1:
 			@Cleanup
-			final IUser player = getPlayer(args, 0);
+			final IUser player = ess.getUserMap().matchUser(args[0], false, false);
 			player.acquireReadLock();
 			if (!player.getData().isTeleportEnabled())
 			{
@@ -52,9 +52,9 @@ public class Commandtp extends EssentialsCommand
 			}
 			user.sendMessage(_("teleporting"));
 			@Cleanup
-			final IUser target = getPlayer(args, 0);
+			final IUser target = ess.getUserMap().matchUser(args[0], false, false);
 			@Cleanup
-			final IUser toPlayer = getPlayer(args, 1);
+			final IUser toPlayer = ess.getUserMap().matchUser(args[1], false, false);
 			target.acquireReadLock();
 			toPlayer.acquireReadLock();
 

Essentials/src/net/ess3/commands/Commandtpa.java

@@ -18,7 +18,7 @@ public class Commandtpa extends EssentialsCommand
 		}
 
 		@Cleanup
-		IUser player = getPlayer(args, 0);
+		IUser player = ess.getUserMap().matchUser(args[0], false, false);
 		player.acquireReadLock();
 		if (!player.getData().isTeleportEnabled())
 		{

Essentials/src/net/ess3/commands/Commandtpaall.java

@@ -24,7 +24,7 @@ public class Commandtpaall extends EssentialsCommand
 			throw new NotEnoughArgumentsException();
 		}
 
-		final IUser player = getPlayer(args, 0);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, false);
 		teleportAAllPlayers(sender, player);
 	}
 

Essentials/src/net/ess3/commands/Commandtpahere.java

@@ -18,7 +18,7 @@ public class Commandtpahere extends EssentialsCommand
 		}
 
 		@Cleanup
-		final IUser player = getPlayer(args, 0);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, false);
 		player.acquireReadLock();
 		if (!player.getData().isTeleportEnabled())
 		{

Essentials/src/net/ess3/commands/Commandtpall.java

@@ -26,7 +26,7 @@ public class Commandtpall extends EssentialsCommand
 			throw new NotEnoughArgumentsException();
 		}
 
-		final IUser player = getPlayer(args, 0);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, false);
 		teleportAllPlayers(sender, player);
 	}
 

Essentials/src/net/ess3/commands/Commandtphere.java

@@ -14,7 +14,7 @@ public class Commandtphere extends EssentialsCommand
 	public void run(final IUser user, final String commandLabel, final String[] args) throws Exception
 	{
 		@Cleanup
-		final IUser player = getPlayer(args, 0);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, false);
 		player.acquireReadLock();
 		if (!player.getData().isTeleportEnabled())
 		{

Essentials/src/net/ess3/commands/Commandtpo.java

@@ -18,7 +18,7 @@ public class Commandtpo extends EssentialsCommand
 		}
 
 		//Just basically the old tp command
-		final IUser player = getPlayer(args, 0, true);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, true);
 		// Check if user is offline
 		if (!player.isOnline())
 		{

Essentials/src/net/ess3/commands/Commandtpohere.java

@@ -21,7 +21,7 @@ public class Commandtpohere extends EssentialsCommand
 		}
 
 		//Just basically the old tphere command
-		final IUser player = getPlayer(args, 0, true);
+		final IUser player = ess.getUserMap().matchUser(args[0], false, true);
 
 		// Check if user is offline
 		if (!player.isOnline())

Essentials/src/net/ess3/commands/Commandtppos.java

@@ -2,6 +2,7 @@ package net.ess3.commands;
 
 import static net.ess3.I18n._;
 import net.ess3.api.IUser;
+import net.ess3.api.server.CommandSender;
 import net.ess3.api.server.Location;
 import net.ess3.economy.Trade;
 //TODO: remove bukkit

Essentials/src/net/ess3/commands/Commandunban.java

@@ -19,7 +19,7 @@ public class Commandunban extends EssentialsCommand
 		try
 		{
 			@Cleanup
-			final IUser player = getPlayer(args, 0, true);
+			final IUser player = ess.getUserMap().matchUser(args[0], false, true);
 			player.acquireWriteLock();
 			player.getData().setBan(null);
 			player.setBanned(false);

Essentials/src/net/ess3/commands/Commandunbanip.java

@@ -18,7 +18,7 @@ public class Commandunbanip extends EssentialsCommand
 		try
 		{
 			@Cleanup
-			final IUser user = getPlayer(args, 0, true);
+			final IUser user = ess.getUserMap().matchUser(args[0], false, true);
 			user.acquireReadLock();
 			ess.getServer().unbanIP(user.getData().getIpAddress());
 		}

Essentials/src/net/ess3/commands/Commandunlimited.java

@@ -26,7 +26,7 @@ public class Commandunlimited extends EssentialsCommand
 
 		if (args.length > 1 && Permissions.UNLIMITED_OTHERS.isAuthorized(user))
 		{
-			target = getPlayer(args, 1);
+			target = ess.getUserMap().matchUser(args[1], false, false);
 			target.acquireReadLock();
 		}
 

Essentials/src/net/ess3/commands/Commandworth.java

@@ -5,6 +5,7 @@ import net.ess3.utils.Util;
 import net.ess3.api.IUser;
 import net.ess3.api.server.ItemStack;
 import java.util.Locale;
+import net.ess3.api.server.CommandSender;
 
 
 public class Commandworth extends EssentialsCommand