Add permissions to commands that do not have them
mcmuffindk opened this issue ยท 1 comments
Versions
WorldEdit version: Bukkit-Official(7.2.2+fc3c499)
Platform version: Paper version git-Paper-520 (MC: 1.16.5)
Describe the bug
Some WE commands either misses permissions or ignores them when doing tab completion and/or executing them wrongfully
To Reproduce
- Create a fresh install of Paper
- Install WE and LuckPerms
- Set up permissions to deny all WE commands
- Try
//br
in various forms, both correct and incorrect
Expected behavior
It is expected that when you have denied all WE permissions, the commands should not show up in the tab completion list also the command help should not be returned when the user is not permitted to use the command
I guess this can be the official tracking issue for adding permissions to these commands, since we never made one.
Notes:
This is technically not a bug, but a feature request. Some commands such as /brush none
do not have permissions associated with them, so //br
should show up.
We have a few options:
(A) Add permissions to these commands.
- Pros: works like everything else
- Cons: these commands will gain permissions after 10 years, breaking a lot of expected behavior. They will need to be specially granted to players if other plugins hook into the worldedit tool system, or e.g. re-use the selection wand. Not a great experience.
(B) Add "default-on" permissions to these commands.
- Pros: better migration flow, no breaking behavior
- Cons: a lot more work to implement, won't work on all platforms and may need a config override.