Resolve (most) issues flagged by @hugmanrique in 65e1b7be87
This still needs to be configurable, but this is a first step towards shifting towards a more intuitive behavior.
This commit is contained in:
@@ -102,11 +102,9 @@ public class VelocityCommandManager implements CommandManager {
|
||||
|
||||
if (!(command instanceof BrigadierCommand)) {
|
||||
if (!meta.getHints().isEmpty()) {
|
||||
// If the user specified a hint, then allow the hint to take precedence over the catch-all
|
||||
// argument.
|
||||
node.getChildren().clear();
|
||||
// If the user specified a hint, then add the hints to the command node directly.
|
||||
for (CommandNode<CommandSource> hint : meta.getHints()) {
|
||||
node.addChild(BrigadierUtils.wrapForHinting(hint, node.getCommand()));
|
||||
node.addChild(hint);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user