Skip to content

Fix default failure handling so unauthorized subcommands are not exposed in "Did you mean" suggestions - #156

Open
RareMan666 wants to merge 1 commit into
Revxrsal:v4from
RareMan666:v4
Open

Fix default failure handling so unauthorized subcommands are not exposed in "Did you mean" suggestions#156
RareMan666 wants to merge 1 commit into
Revxrsal:v4from
RareMan666:v4

Conversation

@RareMan666

Copy link
Copy Markdown
  1. Register multiple subcommands under the same root, and give the test player no permission for at least one of them.
  2. Join with a normal player account and input the root command incorrectly or incompletely.
  3. Before this fix, the failure message could include unauthorized subcommands in Failed to find a suitable command
    for your input... Did you mean:.
  4. After this fix, only commands visible to the player are suggested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant