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

Plot flag permissions does not consider Operator // need all permissions for flags #4574

Closed
1 of 2 tasks
SirC4LV1N opened this issue Jan 18, 2025 · 1 comment
Closed
1 of 2 tasks
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@SirC4LV1N
Copy link

SirC4LV1N commented Jan 18, 2025

Server Implementation

Paper

Server Version

1.21.3

Describe the bug

When adding / removing / changing a plot-flag, the plugin writes you need permission, even as an operator with all permissions.

To Reproduce

I needed to set every permission for the plot flags (for example weather):
-plots.flag
-plots.flag.remove
-plots.set.flag.weather
-.....

only then i could change (in that case remove) the plot-flag. befor that it told me i dont have permission.

i also tried plots.* and plots.flag.* and plots.set.* wich did not work

Expected behaviour

When having permission for everything (being op) or having the * permission it should not tell me i dont have permission, but execute the command

Screenshots / Videos

Image

Image

Image

Image

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/221cf75eb57b43fd9b071b120c7977a5

PlotSquared Version

7.4.0-Premium

Checklist

Anything else?

No response

@SirC4LV1N SirC4LV1N added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Jan 18, 2025
@dordsor21
Copy link
Member

This would be up to your permissions plugin, whether it gives OPs all permissions and allows wildcard permissions.

@dordsor21 dordsor21 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"
Projects
None yet
Development

No branches or pull requests

2 participants