small fixes and missing author for waterfall

YouHaveTrouble · Dec 24, 2021 · 7cb94cc · 7cb94cc
1 parent 7f6315b
commit 7cb94cc
Show file tree

Hide file tree

Showing 9 changed files with 14 additions and 14 deletions.
diff --git a/CommandWhitelistBukkit/pom.xml b/CommandWhitelistBukkit/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>eu.endermite.commandwhitelist</groupId>
         <artifactId>CommandWhitelist</artifactId>
-        <version>2.2.2</version>
+        <version>2.2.3</version>
     </parent>
 
     <artifactId>Bukkit</artifactId>

diff --git a/CommandWhitelistCommon/pom.xml b/CommandWhitelistCommon/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>eu.endermite.commandwhitelist</groupId>
         <artifactId>CommandWhitelist</artifactId>
-        <version>2.2.2</version>
+        <version>2.2.3</version>
     </parent>
 
     <artifactId>Common</artifactId>

diff --git a/CommandWhitelistCommon/src/main/java/eu/endermite/commandwhitelist/common/CommandUtil.java b/CommandWhitelistCommon/src/main/java/eu/endermite/commandwhitelist/common/CommandUtil.java
@@ -17,15 +17,17 @@ public class CommandUtil {
     public static List<String> filterSuggestions(String buffer, Collection<String> suggestions, Collection<String> blockedSubCommands) {
         if (buffer.startsWith("/"))
             buffer = buffer.substring(1);
+        List<String> suggestionsList = new ArrayList<>(suggestions);
+        if (suggestions.isEmpty() || blockedSubCommands.isEmpty()) return suggestionsList;
         for (String s : blockedSubCommands) {
-            String slast = getLastArgument(s);
             String scommand = cutLastArgument(s);
             if (buffer.startsWith(scommand)) {
-                while (suggestions.contains(slast))
-                    suggestions.remove(slast);
+                String slast = getLastArgument(s);
+                while (suggestionsList.contains(slast))
+                    suggestionsList.remove(slast);
             }
         }
-        return new ArrayList<>(suggestions);
+        return suggestionsList;
     }
 
     /**

diff --git a/CommandWhitelistVelocity/pom.xml b/CommandWhitelistVelocity/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>eu.endermite.commandwhitelist</groupId>
         <artifactId>CommandWhitelist</artifactId>
-        <version>2.2.2</version>
+        <version>2.2.3</version>
     </parent>
 
     <artifactId>Velocity</artifactId>

diff --git a/CommandWhitelistWaterfall/pom.xml b/CommandWhitelistWaterfall/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>eu.endermite.commandwhitelist</groupId>
         <artifactId>CommandWhitelist</artifactId>
-        <version>2.2.2</version>
+        <version>2.2.3</version>
     </parent>
 
     <artifactId>Waterfall</artifactId>

diff --git a/...ain/java/eu/endermite/commandwhitelist/waterfall/listeners/BungeeTabcompleteListener.java b/...ain/java/eu/endermite/commandwhitelist/waterfall/listeners/BungeeTabcompleteListener.java
@@ -13,15 +13,13 @@ public class BungeeTabcompleteListener implements Listener {
     public void onTabcomplete(net.md_5.bungee.api.event.TabCompleteEvent event) {
         if (!(event.getReceiver() instanceof ProxiedPlayer)) return;
         ProxiedPlayer player = (ProxiedPlayer) event.getReceiver();
+        if (event.getSuggestions().isEmpty()) return;
         if (player.hasPermission(CWPermission.BYPASS.permission())) return;
-
         CommandUtil.filterSuggestions(
                 event.getCursor(),
                 event.getSuggestions(),
                 CommandWhitelistWaterfall.getSuggestions(player)
         );
-
-
     }
 
 }
diff --git a/...va/eu/endermite/commandwhitelist/waterfall/listeners/WaterfallDefineCommandsListener.java b/...va/eu/endermite/commandwhitelist/waterfall/listeners/WaterfallDefineCommandsListener.java
@@ -15,8 +15,7 @@ public class WaterfallDefineCommandsListener implements Listener {
     public void onProxyDefineCommandsEvent(io.github.waterfallmc.waterfall.event.ProxyDefineCommandsEvent event) {
         if (event.getReceiver() instanceof ProxiedPlayer) {
             ProxiedPlayer player = (ProxiedPlayer) event.getReceiver();
-            if (player.hasPermission(CWPermission.BYPASS.permission()))
-                return;
+            if (player.hasPermission(CWPermission.BYPASS.permission())) return;
             HashMap<String, Command> commandHashMap = new HashMap<>();
             CommandWhitelistWaterfall.getCommands(player).forEach(cmdName ->
                     CommandWhitelistWaterfall.getPlugin().getProxy().getPluginManager().getCommands()

diff --git a/CommandWhitelistWaterfall/src/main/resources/bungee.yml b/CommandWhitelistWaterfall/src/main/resources/bungee.yml
@@ -1,4 +1,5 @@
 name: CommandWhitelist
+author: YouHaveTrouble
 version: ${project.version}
 main: eu.endermite.commandwhitelist.waterfall.CommandWhitelistWaterfall
 description: You decide what commands players can use or tab complete on your server!
diff --git a/pom.xml b/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>eu.endermite.commandwhitelist</groupId>
     <artifactId>CommandWhitelist</artifactId>
-    <version>2.2.2</version>
+    <version>2.2.3</version>
     <modules>
         <module>CommandWhitelistCommon</module>
         <module>CommandWhitelistBukkit</module>