-
Notifications
You must be signed in to change notification settings - Fork 34
Open
Description
It looks like the expansion does not properly handle case insensitivity. The argument (sometimes a perm node) passed to a placeholder is set to lowercase:
https://github.com/LuckPerms/placeholders/blob/master/common/src/main/java/me/lucko/luckperms/placeholders/LPPlaceholderProvider.java#L415
placeholder = placeholder.toLowerCase(Locale.ROOT);Yet in most placeholders, the lowercased node is being compared against nodes in their original case, failing on nodes which were originally in a case like camel.
placeholders/common/src/main/java/me/lucko/luckperms/placeholders/LPPlaceholderProvider.java
Lines 166 to 170 in d8fe626
| builder.addDynamic("inherits_permission", (player, user, userData, queryOptions, node) -> | |
| user.resolveInheritedNodes(queryOptions).stream() | |
| .filter(n -> n.getContexts().isSatisfiedBy(queryOptions.context())) | |
| .anyMatch(n -> n.getKey().equals(node)) | |
| ); |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels