194 lines
9.2 KiB
Diff
194 lines
9.2 KiB
Diff
From 2e148b75463f007fbfd5c9d19bb2c0dbf5658aa3 Mon Sep 17 00:00:00 2001
|
|
From: DemonWav <demonwav@gmail.com>
|
|
Date: Sat, 30 Jan 2016 18:58:09 -0600
|
|
Subject: [PATCH] Add Location support to tab completers (vanilla feature
|
|
missing in CraftBukkit)
|
|
|
|
|
|
diff --git a/src/main/java/org/bukkit/command/Command.java b/src/main/java/org/bukkit/command/Command.java
|
|
index 548d570..c5b2e34 100644
|
|
--- a/src/main/java/org/bukkit/command/Command.java
|
|
+++ b/src/main/java/org/bukkit/command/Command.java
|
|
@@ -8,6 +8,7 @@ import java.util.Set;
|
|
import org.apache.commons.lang.Validate;
|
|
import org.bukkit.Bukkit;
|
|
import org.bukkit.ChatColor;
|
|
+import org.bukkit.Location;
|
|
import org.bukkit.Server;
|
|
import org.bukkit.entity.Player;
|
|
import org.bukkit.entity.minecart.CommandMinecart;
|
|
@@ -85,6 +86,42 @@ public abstract class Command {
|
|
* @throws IllegalArgumentException if sender, alias, or args is null
|
|
*/
|
|
public List<String> tabComplete(CommandSender sender, String alias, String[] args) throws IllegalArgumentException {
|
|
+ // PaperSpigot - location tab-completes
|
|
+ /*
|
|
+ To prevent infinite recursion, this implementation has been moved down to
|
|
+ tabCompleteImpl(CommandSender sender, String alias, String[] args). The infinite recursion happens when
|
|
+ a subclass calls super.tabComplete(CommandSender sender, String alias, String[] args, Location location),
|
|
+ which would end up calling tabComplete(CommandSender sender, String alias, String[] args), but rather than
|
|
+ this method, it would call the overridden method, which would call super.tabComplete again, etc. To prevent
|
|
+ this we move the actual main logic to a separate private method.
|
|
+ */
|
|
+ return tabCompleteImpl(sender, alias, args);
|
|
+ }
|
|
+
|
|
+ // PaperSpigot start - location tab-completes
|
|
+ /**
|
|
+ * Executed on tab completion for this command, returning a list of options the player can tab through. This method
|
|
+ * returns the {@link Location} of the block the player is looking at at the time of the tab complete.
|
|
+ * <p>
|
|
+ * Commands that want to use the Location information in their tab-complete implementations need to override this
|
|
+ * method. The Location provided by this method is the block that the player is currently looking at when the player
|
|
+ * attempts the tab complete. For this to be valid, the block must be highlighted by the player (i.e. the player is
|
|
+ * close enough to interact with the block).
|
|
+ *
|
|
+ * @param sender Source object which is executing this command
|
|
+ * @param alias the alias being used
|
|
+ * @param args All arguments passed to the command, split via ' '
|
|
+ * @param location the location of the block the player is looking at
|
|
+ * @return a list of tab-completions for the specified arguments. This
|
|
+ * will never be null. List may be immutable.
|
|
+ * @throws IllegalArgumentException if sender, alias, or args is null
|
|
+ */
|
|
+ public List<String> tabComplete(CommandSender sender, String alias, String[] args, Location location) throws IllegalArgumentException {
|
|
+ // Simply default to the standard tab-complete, subclasses can override this if needed
|
|
+ return tabCompleteImpl(sender, alias, args);
|
|
+ }
|
|
+
|
|
+ private List<String> tabCompleteImpl(CommandSender sender, String alias, String[] args) throws IllegalArgumentException {
|
|
Validate.notNull(sender, "Sender cannot be null");
|
|
Validate.notNull(args, "Arguments cannot be null");
|
|
Validate.notNull(alias, "Alias cannot be null");
|
|
@@ -108,6 +145,7 @@ public abstract class Command {
|
|
Collections.sort(matchedPlayers, String.CASE_INSENSITIVE_ORDER);
|
|
return matchedPlayers;
|
|
}
|
|
+ // PaperSpigot end
|
|
|
|
/**
|
|
* Returns the name of this command
|
|
diff --git a/src/main/java/org/bukkit/command/PluginCommand.java b/src/main/java/org/bukkit/command/PluginCommand.java
|
|
index 3bfa31f..e3f8295 100644
|
|
--- a/src/main/java/org/bukkit/command/PluginCommand.java
|
|
+++ b/src/main/java/org/bukkit/command/PluginCommand.java
|
|
@@ -3,6 +3,7 @@ package org.bukkit.command;
|
|
import java.util.List;
|
|
|
|
import org.apache.commons.lang.Validate;
|
|
+import org.bukkit.Location;
|
|
import org.bukkit.plugin.Plugin;
|
|
|
|
/**
|
|
@@ -122,6 +123,15 @@ public final class PluginCommand extends Command implements PluginIdentifiableCo
|
|
*/
|
|
@Override
|
|
public java.util.List<String> tabComplete(CommandSender sender, String alias, String[] args) throws CommandException, IllegalArgumentException {
|
|
+ return tabComplete(sender, alias, args, null); // PaperSpigot - The code from this method has been (slightly modified) moved to the Location method.
|
|
+ }
|
|
+
|
|
+ // PaperSpigot start - location tab-completes
|
|
+ /*
|
|
+ this code was copied, except for the noted changes, from tabComplete(CommandSender sender, String alias, String[] args)
|
|
+ */
|
|
+ @Override
|
|
+ public List<String> tabComplete(CommandSender sender, String alias, String[] args, Location location) throws CommandException, IllegalArgumentException {
|
|
Validate.notNull(sender, "Sender cannot be null");
|
|
Validate.notNull(args, "Arguments cannot be null");
|
|
Validate.notNull(alias, "Alias cannot be null");
|
|
@@ -129,10 +139,10 @@ public final class PluginCommand extends Command implements PluginIdentifiableCo
|
|
List<String> completions = null;
|
|
try {
|
|
if (completer != null) {
|
|
- completions = completer.onTabComplete(sender, this, alias, args);
|
|
+ completions = completer.onTabComplete(sender, this, alias, args, location); // PaperSpigot - add location argument
|
|
}
|
|
if (completions == null && executor instanceof TabCompleter) {
|
|
- completions = ((TabCompleter) executor).onTabComplete(sender, this, alias, args);
|
|
+ completions = ((TabCompleter) executor).onTabComplete(sender, this, alias, args, location); // PaperSpigot - add location argument
|
|
}
|
|
} catch (Throwable ex) {
|
|
StringBuilder message = new StringBuilder();
|
|
@@ -145,10 +155,11 @@ public final class PluginCommand extends Command implements PluginIdentifiableCo
|
|
}
|
|
|
|
if (completions == null) {
|
|
- return super.tabComplete(sender, alias, args);
|
|
+ return super.tabComplete(sender, alias, args, location); // PaperSpigot - add location argument
|
|
}
|
|
return completions;
|
|
}
|
|
+ // PaperSpigot end
|
|
|
|
@Override
|
|
public String toString() {
|
|
diff --git a/src/main/java/org/bukkit/command/SimpleCommandMap.java b/src/main/java/org/bukkit/command/SimpleCommandMap.java
|
|
index a300ae7..12d9232 100644
|
|
--- a/src/main/java/org/bukkit/command/SimpleCommandMap.java
|
|
+++ b/src/main/java/org/bukkit/command/SimpleCommandMap.java
|
|
@@ -12,6 +12,7 @@ import java.util.Map;
|
|
import java.util.regex.Pattern;
|
|
|
|
import org.apache.commons.lang.Validate;
|
|
+import org.bukkit.Location;
|
|
import org.bukkit.Server;
|
|
import org.bukkit.command.defaults.*;
|
|
import org.bukkit.entity.Player;
|
|
@@ -167,6 +168,14 @@ public class SimpleCommandMap implements CommandMap {
|
|
}
|
|
|
|
public List<String> tabComplete(CommandSender sender, String cmdLine) {
|
|
+ return tabComplete(sender, cmdLine, null); // PaperSpigot - location tab-completes, code moved below
|
|
+ }
|
|
+
|
|
+ // PaperSpigot start - location tab-completes
|
|
+ /*
|
|
+ this code was copied, except for the noted change, from tabComplete(CommandSender sender, String cmdLine)
|
|
+ */
|
|
+ public List<String> tabComplete(CommandSender sender, String cmdLine, Location location) {
|
|
Validate.notNull(sender, "Sender cannot be null");
|
|
Validate.notNull(cmdLine, "Command line cannot null");
|
|
|
|
@@ -211,13 +220,14 @@ public class SimpleCommandMap implements CommandMap {
|
|
String[] args = PATTERN_ON_SPACE.split(argLine, -1);
|
|
|
|
try {
|
|
- return target.tabComplete(sender, commandName, args);
|
|
+ return target.tabComplete(sender, commandName, args, location); // PaperSpigot - add location argument
|
|
} catch (CommandException ex) {
|
|
throw ex;
|
|
} catch (Throwable ex) {
|
|
throw new CommandException("Unhandled exception executing tab-completer for '" + cmdLine + "' in " + target, ex);
|
|
}
|
|
}
|
|
+ // PaperSpigot end
|
|
|
|
public Collection<Command> getCommands() {
|
|
return Collections.unmodifiableCollection(knownCommands.values());
|
|
diff --git a/src/main/java/org/bukkit/command/TabCompleter.java b/src/main/java/org/bukkit/command/TabCompleter.java
|
|
index 6d61e3a..2cb971c 100644
|
|
--- a/src/main/java/org/bukkit/command/TabCompleter.java
|
|
+++ b/src/main/java/org/bukkit/command/TabCompleter.java
|
|
@@ -1,5 +1,7 @@
|
|
package org.bukkit.command;
|
|
|
|
+import org.bukkit.Location;
|
|
+
|
|
import java.util.List;
|
|
|
|
/**
|
|
@@ -19,4 +21,10 @@ public interface TabCompleter {
|
|
* to default to the command executor
|
|
*/
|
|
public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args);
|
|
+
|
|
+ // PaperSpigot start - location tab-completes
|
|
+ default List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args, Location location) {
|
|
+ return onTabComplete(sender, command, alias, args);
|
|
+ }
|
|
+ // PaperSpigot end
|
|
}
|
|
--
|
|
2.7.0
|
|
|