Skip to content

Commit ce7d33a

Browse files
committed
Remove command tree method as that'll be done upon registration
1 parent 0553016 commit ce7d33a

File tree

2 files changed

+3
-13
lines changed

2 files changed

+3
-13
lines changed

src/main/java/org/spongepowered/api/command/manager/CommandManager.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ public interface CommandManager {
138138
* aliases
139139
* @param container The {@link PluginContainer} to register the command for
140140
* @param commandTree The {@link CommandTreeBuilder} that represents this command
141-
* structure
141+
* structure.
142142
* @param requirement What a {@link CommandCause} needs to fulfil in order for this
143143
* command to be executed.
144144
* @param primaryAlias The first command alias to register

src/main/java/org/spongepowered/api/command/registrar/CommandRegistrar.java

+2-12
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838

3939
import java.util.List;
4040
import java.util.Optional;
41+
import java.util.function.Predicate;
4142

4243
/**
4344
* {@link CommandRegistrar}s are the entry point for plugins wishing to provide
@@ -47,7 +48,7 @@
4748
* <p>This interface <em>does not</em> have a defined way to register commands,
4849
* it is up to the implementor to provide such a method. Such a registration
4950
* method <strong>must</strong> call
50-
* {@link CommandManager#registerAlias(CommandRegistrar, PluginContainer, String, String...)}
51+
* {@link CommandManager#registerAlias(CommandRegistrar, PluginContainer, CommandTreeBuilder.Basic, Predicate, String, String...)}
5152
* to indicate that they wish to take control of certain aliases. Beyond this
5253
* call, the {@link CommandRegistrar} will only need to retain the link between
5354
* the primary alias and the command to execute, as the {@link CommandManager}
@@ -144,15 +145,4 @@ public interface CommandRegistrar extends CatalogType {
144145
*/
145146
void unregister(CommandMapping mapping);
146147

147-
/**
148-
* Called when an update to the client's command list is requested,
149-
* for client side completion.
150-
*
151-
* @param commandCause The {@link CommandCause} to build the tree for
152-
* @param builder The builder to supply command information to
153-
*/
154-
// TODO: Should this just be a player, as that's the only entity the tree
155-
// will be sent for?
156-
void completeCommandTree(CommandCause commandCause, CommandTreeBuilder.Basic builder);
157-
158148
}

0 commit comments

Comments
 (0)