Change valid name checking to a configurable predicate #382

I see the value in allowing acf users to disable the valid name check.
I think others may want to go a step further and create their own
valid name verification. Those that want to truly accept all names can simply
do `CommandManager#setValidNamePredicate(name -> true)` with the new API.
Another solution I considered was having users override the isValidName method
on their CommandManager, but this functionality seemed more friendly.

bungee/src/main/java/co/aikar/commands/ACFBungeeUtil.java

@@ -27,6 +27,7 @@ import net.md_5.bungee.api.ChatColor;
 import net.md_5.bungee.api.CommandSender;
 import net.md_5.bungee.api.ProxyServer;
 import net.md_5.bungee.api.connection.ProxiedPlayer;
+import org.jetbrains.annotations.Nullable;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -124,7 +125,7 @@ public class ACFBungeeUtil {
         }
 
         if (matches.isEmpty()) {
-            if (!isValidName(name, issuer.getManager())) {
+            if (!issuer.getManager().isValidName(name)) {
                 issuer.sendError(MinecraftMessageKeys.IS_NOT_A_VALID_NAME, "{name}", name);
                 return null;
             }
@@ -149,13 +150,8 @@ public class ACFBungeeUtil {
         throw new IllegalStateException("You may not use the ACFBungeeUtil#findPlayerSmart(CommandSender) async to the command execution.");
     }
 
-    @Deprecated
-    public static boolean isValidName(String name) {
-        return isValidName(name, null);
-    }
-
-    public static boolean isValidName(String name, CommandManager manager) {
-        return name != null && !name.isEmpty() && ((manager != null && manager.isAllowInvalidName()) || ACFPatterns.VALID_NAME_PATTERN.matcher(name).matches());
+    public static boolean isValidName(@Nullable String name) {
+        return name != null && !name.isEmpty() && ACFPatterns.VALID_NAME_PATTERN.matcher(name).matches();
     }
 
     public static <T> T validate(T object, String message, Object... values) {

bungee/src/main/java/co/aikar/commands/BungeeCommandManager.java

@@ -66,6 +66,8 @@ public class BungeeCommandManager extends CommandManager<
 
         getLocales(); // auto load locales
 
+        this.validNamePredicate = ACFBungeeUtil::isValidName;
+
         plugin.getProxy().getPluginManager().registerListener(plugin, new ACFBungeeListener(this, plugin));
 
         //BungeeCord has no event for listening for client setting changes