Skip to content
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

Revise how we handle permissions. #928

Open
dumptruckman opened this issue Oct 23, 2012 · 4 comments
Open

Revise how we handle permissions. #928

dumptruckman opened this issue Oct 23, 2012 · 4 comments
Assignees
Labels
PR: Better Coding Improvements in code, but not introducing new features. Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it.

Comments

@dumptruckman
Copy link
Member

Not so much change the permission nodes, just make handling them a bit saner.

The current way we are handling them is very likely causing permission issues for many users. However, many users are also able to use them successfully so this is not going to be fixed in 2.5. If you're having any kind of permission issues then just know that they will be probably be fixed after we release 2.5-AB.

@ghost ghost assigned main-- Oct 23, 2012
@ammaraskar
Copy link
Contributor

btw permissions for command should be set up in plugin.yml like so

commands:
  awesomecommand:
    usage: blarg
    permission: permission.node

This allows craftbukkit to display the command only if you have the permission in /help and only tab completes it if you have the permission

@hades200082
Copy link

The Gamemode bypass permission issue is a real PITA for my builders. It only seems to be an issue when travelling to a "non-primary" world. i.e. any world that is not specified in server.properties file.

I use GroupManager if that helps.

@fernferret
Copy link
Member

This allows craftbukkit to display the command only if you have the permission in /help and only tab completes it if you have the permission

Can't do this for all, dynamic permission allocation (world perms) :( unless it will read the permission structure that is injected at runtime, but I doubt it. That seems like a rabbit hole I wouldn't go down.

@ammaraskar
Copy link
Contributor

Actually, it will if you are adding those permissions with parenting and such and registering them internally to bukkit, it will work.

@benwoo1110 benwoo1110 added the Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it. label Nov 7, 2020
@benwoo1110 benwoo1110 mentioned this issue Feb 10, 2023
75 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Better Coding Improvements in code, but not introducing new features. Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it.
Projects
None yet
Development

No branches or pull requests

6 participants