Skip to content
This repository was archived by the owner on Dec 4, 2024. It is now read-only.

Add permission context#24

Closed
Tobi406 wants to merge 2 commits into
LuckPerms:masterfrom
Tobi406:add-permission-calculator
Closed

Add permission context#24
Tobi406 wants to merge 2 commits into
LuckPerms:masterfrom
Tobi406:add-permission-calculator

Conversation

@Tobi406
Copy link
Copy Markdown

@Tobi406 Tobi406 commented Jun 1, 2020

Adds a permission= context.

This only works for nodes the user has explicitly set and eg. not wildcard ones.

@lucko
Copy link
Copy Markdown
Member

lucko commented Jun 1, 2020

What's the use-case? I don't really think something like this is a good idea to be honest!

@Tobi406
Copy link
Copy Markdown
Author

Tobi406 commented Jun 1, 2020

I sometimes (not very often too) see people asking on Discord for that.

A use case I can imagine is the following:

Two groups:

  • vip
  • helper

In the helper group there is a test.permission permission=group.vip, so to give a group a permission, but only if the player has the other group.

Cases found on Discord:
image
image

@lucko
Copy link
Copy Markdown
Member

lucko commented Jun 1, 2020

Hmm, honestly I think this is better implemented by just using groups properly. Including all permissions the player has set in their contexts feels really dirty.

@Tobi406
Copy link
Copy Markdown
Author

Tobi406 commented Jun 1, 2020

Probably

@Tobi406 Tobi406 closed this Jun 1, 2020
@niqofdez
Copy link
Copy Markdown

Hi @Tobi406! You can provide a download link of the compiled plugin with this modification?

@Tobi406
Copy link
Copy Markdown
Author

Tobi406 commented Jul 19, 2020

You can build it yourself, I haven't deleted the branch yet.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants