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.

Sign up for GitHub

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

Jump to bottom

Have "perm set" override #1198

Closed
Kowaman opened this issue Sep 4, 2018 · 0 comments
Closed

Have "perm set" override #1198

Kowaman opened this issue Sep 4, 2018 · 0 comments
Labels
type: suggestion The issue is suggesting a new feature or enhancement.

Comments

@Kowaman
Copy link

Kowaman commented Sep 4, 2018

Hi, I think it would be better if when doing /lp user/group <user/group> perm set ture/false, if the user/group already has the perm, and the new value is different than the current value, override it rather than having it tell you they already have the per node. This saves you time when you need to swap a node as you don't have to type "unset" then "set".
@lucko lucko added the type: suggestion The issue is suggesting a new feature or enhancement. label Sep 5, 2018
lucko added a commit that referenced this issue Sep 20, 2018
@lucko
Operations to add new nodes with differing values should just overrid…
d28170c 
…e existing entries, instead of failing with the ALREADY_HAS result (#1198)
@lucko lucko closed this as completed Sep 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: suggestion The issue is suggesting a new feature or enhancement.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Kowaman @lucko