From 85554c2b6e432193f7d6a8e3fd110f65e6ccdfd1 Mon Sep 17 00:00:00 2001
From: Aikar <aikar@aikar.co>
Date: Wed, 2 Aug 2017 22:44:11 -0400
Subject: [PATCH] remove extra error feedback on OnlinePlayer's

---
 .../main/java/co/aikar/commands/BukkitCommandContexts.java   | 2 --
 .../main/java/co/aikar/commands/BungeeCommandContexts.java   | 5 +----
 .../main/java/co/aikar/commands/SpongeCommandContexts.java   | 2 --
 3 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/bukkit/src/main/java/co/aikar/commands/BukkitCommandContexts.java b/bukkit/src/main/java/co/aikar/commands/BukkitCommandContexts.java
index 8918cdf4..415e1cc6 100644
--- a/bukkit/src/main/java/co/aikar/commands/BukkitCommandContexts.java
+++ b/bukkit/src/main/java/co/aikar/commands/BukkitCommandContexts.java
@@ -204,14 +204,12 @@ public class BukkitCommandContexts extends CommandContexts<BukkitCommandExecutio
 
     @Nullable
     OnlinePlayer getOnlinePlayer(BukkitCommandIssuer issuer, String lookup, boolean allowMissing) throws InvalidCommandArgument {
-        CommandSender sender = issuer.getIssuer();
         Player player = ACFBukkitUtil.findPlayerSmart(issuer, lookup);
         //noinspection Duplicates
         if (player == null) {
             if (allowMissing) {
                 return null;
             }
-            this.manager.sendMessage(sender, MessageType.ERROR, MessageKeys.COULD_NOT_FIND_PLAYER, "{search}", lookup);
             throw new InvalidCommandArgument(false);
         }
         return new OnlinePlayer(player);
diff --git a/bungee/src/main/java/co/aikar/commands/BungeeCommandContexts.java b/bungee/src/main/java/co/aikar/commands/BungeeCommandContexts.java
index bacddfea..1f189a85 100644
--- a/bungee/src/main/java/co/aikar/commands/BungeeCommandContexts.java
+++ b/bungee/src/main/java/co/aikar/commands/BungeeCommandContexts.java
@@ -38,14 +38,11 @@ public class BungeeCommandContexts extends CommandContexts<BungeeCommandExecutio
     BungeeCommandContexts(CommandManager manager) {
         super(manager);
         registerContext(OnlineProxiedPlayer.class, (c) -> {
-            final String playercheck = c.popFirstArg();
-            ProxiedPlayer proxiedPlayer = ACFBungeeUtil.findPlayerSmart(c.getIssuer(), playercheck);
+            ProxiedPlayer proxiedPlayer = ACFBungeeUtil.findPlayerSmart(c.getIssuer(), c.popFirstArg());
             if (proxiedPlayer == null) {
                 if (c.hasAnnotation(Optional.class)) {
                     return null;
                 }
-                c.getIssuer().sendMessage(MessageType.ERROR, MessageKeys.COULD_NOT_FIND_PLAYER,
-                        "{search}", playercheck);
                 throw new InvalidCommandArgument(false);
             }
             return new OnlineProxiedPlayer(proxiedPlayer);
diff --git a/sponge/src/main/java/co/aikar/commands/SpongeCommandContexts.java b/sponge/src/main/java/co/aikar/commands/SpongeCommandContexts.java
index cb67163a..ebe3aa9d 100644
--- a/sponge/src/main/java/co/aikar/commands/SpongeCommandContexts.java
+++ b/sponge/src/main/java/co/aikar/commands/SpongeCommandContexts.java
@@ -141,14 +141,12 @@ public class SpongeCommandContexts extends CommandContexts<SpongeCommandExecutio
 
     @Nullable
     OnlinePlayer getOnlinePlayer(SpongeCommandIssuer issuer, String lookup, boolean allowMissing) throws InvalidCommandArgument {
-        CommandSource sender = issuer.getIssuer();
         Player player = ACFSpongeUtil.findPlayerSmart(issuer, lookup);
         //noinspection Duplicates
         if (player == null) {
             if (allowMissing) {
                 return null;
             }
-            this.manager.sendMessage(sender, MessageType.ERROR, MessageKeys.COULD_NOT_FIND_PLAYER, "{search}", lookup);
             throw new InvalidCommandArgument(false);
         }
         return new OnlinePlayer(player);