Upstream/Minestom
1
0
Fork 0
mirror of https://github.com/Minestom/Minestom.git synced 2025-01-03 23:17:48 +01:00
Code Issues Projects Releases Wiki Activity

Fixed inconsistency between hasPermission(Permission) and hasPermission(Class)

Browse Source
This commit is contained in:
jglrxavpok 2020-07-31 22:37:33 +02:00
parent 1772b94114
commit 4782d6e899
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/net/minestom/server/command/CommandSender.java
View File

@ -55,17 +55,17 @@ public interface CommandSender {
/**
* Checks if the given permission is possessed by this command sender.
* Simple shortcut to <pre>getAllPermissions().contains(permission)</pre> for readability.
* Simple shortcut to <pre>getAllPermissions().contains(permission) && permission.isValidFor(this)</pre> for readability.
* @param p permission to check against
* @return
*/
default boolean hasPermission(Permission p) {
return getAllPermissions().contains(p);
return getAllPermissions().contains(p) && p.isValidFor(this);
}
/**
* Checks if the given permission is possessed by this command sender.
* Will call {@link #hasPermission(Permission)} on all permissions that are an instance of permissionClass.
* Will call {@link Permission#isValidFor(Permission)} on all permissions that are an instance of permissionClass.
* If no matching permission is found, this result returns false.
*
* @param permissionClass