LuckPerms

LuckPerms

41.4k Downloads

Various PermissionHolder permission lookup methods not following contract

bloodmc opened this issue ยท 1 comments

commented

Using LP 4.4.26, I was originally trying to use getPermanentPermissionNodes() but I discovered the following javadocs

https://github.com/lucko/LuckPerms/blob/master/api/src/main/java/me/lucko/luckperms/api/PermissionHolder.java#L149-L225

This was causing both persisted nodes as well as transient nodes to be included in result.

Although this method is named getPermanentPermissionNodes, it will actually return all types of node.

Any plans on making this follow the intended contract?

As a workaround, I changed code to just call getNodes() which seems to work.

commented

I see this has been addressed in v5 of API.