LuckPerms

LuckPerms

41.4k Downloads

[Feature Request] Disable auto permission organizing.

Kyrobi opened this issue · 6 comments

commented

I don't know what it's called, so I'll just refer to it as "auto permissions organizing".

The request adds a config option to leave comments in flatfile groups without the plugin removing them.

Example:

image

It helps when organizing permissions for each plugin when directly editing the files. Keeps the permission nodes for a group nice and tidy.

Currently, the plugin just alphabetize the perms:
image

It's nice, but I don't want my comments to be removed stating which plugin these permissions are coming from.

commented

It's not recommendet to set permissions directly through files, since there are better alternatives like the web editor.
In addition do file storages artificially increase storage space used.

commented

The files only get reorganized when you change the permissions of what the file referrs to. So if you don't want it overriden, don't change the data through any means but the files.

Though I'd highly urge you not to do that. I mean to what plugin a permission belongs should be clear from the get go, as they start with the plugin name.

commented

Yes, I understand that commenting artificially increases the file of the file.

Regards to the permissions starting with the name of the plugin, yeah I get it. It sounds like a personal problem. But, it does make it looks nicer :D

commented

I talk about the files themself and not if the comments increase storage used.
Compared to H2 or MySQL are file-based storages much heavier in terms of storage used.

I don't think this will ever be added, as you're so far the only one suggesting it and we also recommend using alternatives, rather than editing the files directly.

commented

The main issue with this request boils down to:

  • The currently used YAML library doesn’t allow for persistent comments. As it’s the library shipped by Bukkit, it would require shipping a custom one, which comes within many problems.
  • It’s a personal problem, as you mentioned. And due to the previous point the effort required to implement this minute feature is several hours. Which is just not proportional to the benefit that feature would provide.
commented

Understood. Thanks!