-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New features and fixes #19
Conversation
HiddenCommands can be disabled from execution or displaying with "help" command Moved/Replaced logic for CommandExecutor#print to Console#printCommands
HiddenCommand allows Commands to be hidden from help and command completer.
@@ -115,6 +115,8 @@ public void setCommandExecutor (CommandExecutor commandExec) { | |||
*/ | |||
@Override | |||
public void execCommand(String command) { | |||
if(disabled) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just curious, does this fix some usecase?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This enforces the disabled flag, whether it is a GUIConsole, HeadlessConsole, or even a custom Console extended from AbstractConsole, it is disabled from executing commands.
Thanks again man, this is some good work you're doing |
No problem. I appreciate your acceptance of my code. |
Implemented HiddenCommand annotation for CommandExecutor methods
Moved logic for CommandExecutor.print() to Console.printCommands()
Fixed console not being disabled while not using GUIConsole if disabled flag was set
Refactored and updated method and argument determination/parsing code
Updated Box2DTest with HiddenCommand methods
Added ConsoleTest