CivMC/Civ
1
0
Fork 0
mirror of https://github.com/CivMC/Civ.git synced 2026-07-18 00:20:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

remove comment

Browse Source
This commit is contained in:
okx-code
2026-01-08 00:45:19 +00:00
parent 5b65e90c51
commit 2869c0f591
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
plugins/namelayer-paper/src/main/java/vg/civcraft/mc/namelayer/GroupManager.java
View File

@@ -458,12 +458,6 @@ public class GroupManager {
*/
private boolean hasPlayerInheritsPerms(Group group, UUID player, PermissionType perm) {
while (group != null) {
// TODO: Revisit this. Being the primary owner of a group should arguably work like OP where you, in-effect,
// have all grantable permissions. However, this would break certain civ-mechanics that use NameLayer
// permissions as feature-toggles, notably JukeAlert with snitch immunity where having the permission
// by definition means you *are* immune to snitches. Contrast this with SuperVanish where having the
// permission means you *can* vanish. Thus the .isOwnerPermission() check has been added to only give
// the OP-like behaviour to existential group permissions like "DELETE", "MERGE", and "PERMS".
if (group.isOwner(player) && perm.isOwnerPermission()) {
return true;
}