mfnalex / ChestSort

Best sorting plugin for Minecraft / Spigot 1.14+
https://www.chestsort.de
GNU General Public License v3.0
91 stars 34 forks source link

Unresolved needs #128

Closed iillililil closed 3 years ago

iillililil commented 3 years ago

Give the default group chestsort.use and chestsort.use.inventory They can use /isort on and /sort on which is not what I want. I think it can be simpler. By default, all players can use the buttons without setting any permissions. For groups or players with chestsort.use and chestsort.use.inventory, automatic sorting is allowed

noramibuu commented 3 years ago

Can you please send output of "/ver" and /ver chestsort" commands?

iillililil commented 3 years ago

image

mfnalex commented 3 years ago

Many people need the permissions feature because they only allow sorting for VIPs etc. You can use the chestsort.automatic permission to allow/disallow automatic sorting while still allowing hotkeys

Just give chestsort.use and chestsort.use.inventory to all players, and deny chestsort.automatic for players who should only be allowed to use hotkeys.

iillililil commented 3 years ago

I give players chestsort.use and chestsort.use.inventory they can use the /isort on and /sort on commands to turn on automatic sorting

iillililil commented 3 years ago

image

mfnalex commented 3 years ago

I give players chestsort.use and chestsort.use.inventory they can use the /isort on and /sort on commands to turn on automatic sorting

You have to deny the angelchest.automatic permission for those players

mfnalex commented 3 years ago

E.g. /luckperms group default permission set angelchest.automatic false

iillililil commented 3 years ago

Solved, I still want to say that the button should be owned by the player by default. If the player is not allowed to use the button, they will also download the mods for use. So chestsort.use and chestsort.use.inventory should not contain /isort on and /sort on. We should set the two commands /isort auto and /sort auto to control the switch of automatic sorting and set permissions for them.

mfnalex commented 3 years ago

I don't think so, in my opinion admins should have 100% control over the features a player has access to. Giving permissions by default can result in angry admins that want to allow features only for VIPs. The behaviour you wanted to have can be achieved by using exactly three permissions: chestsort.use (allow), chestsort.use.inventory (allow) and chestsort.automatic (deny).

Also I don't see any reason to change the current commands. /sort (and /isort on|off) toggles automatic sorting, hotkeys can be used to sort manually.