-
-
Notifications
You must be signed in to change notification settings - Fork 497
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
Fixed Blacklist/Whitelist not being resepected #1680
Conversation
…tSelectedRoles() of the same scope and attempts to get the highest role the user has that is present in the selected roles. Changes onRequest in PlaceholderAPIExpansion.java to use this new method rather than just fetching their highest role and ignoring selectedRoles. Additionally removed logic getting a list of selected rules as it is not needed here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work perfectly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After talking about this a bit more it’d be preferable if these were split out into different placeholders (ie top_selected_role)
Sounds good |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR! :)
fix #1557
Added getTopSelectedRole() method to DiscordSRV.java. Method calls getSelectedRoles() of the same scope and attempts to get the highest role the user has that is present in the selected roles.
Changes onRequest in PlaceholderAPIExpansion.java to use this new method rather than just fetching their highest role and ignoring selectedRoles. Additionally, removed logic getting a list of selected rules as it is not needed here.