diff options
| author | ElgarL <ElgarL@palmergames.com> | 2012-01-27 13:09:11 +0000 |
|---|---|---|
| committer | ElgarL <ElgarL@palmergames.com> | 2012-01-27 13:09:11 +0000 |
| commit | 7695c9f5c70380623868ce60900c87f7d75327a5 (patch) | |
| tree | 1d2958ff5703fdf4ac7688011a4f21b03796d1cb /EssentialsGroupManager/src/org | |
| parent | 25806e7fa8f6d1852624f57ec40ed88b46fc36e6 (diff) | |
| download | Essentials-7695c9f5c70380623868ce60900c87f7d75327a5.tar Essentials-7695c9f5c70380623868ce60900c87f7d75327a5.tar.gz Essentials-7695c9f5c70380623868ce60900c87f7d75327a5.tar.lz Essentials-7695c9f5c70380623868ce60900c87f7d75327a5.tar.xz Essentials-7695c9f5c70380623868ce60900c87f7d75327a5.zip | |
Properly fix inherited negated nodes.
Diffstat (limited to 'EssentialsGroupManager/src/org')
| -rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java index 429a9553f..002558968 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java @@ -134,12 +134,13 @@ public class AnjoPermissionsHandler extends PermissionsReaderInterface { groupPermArray = populatePerms(ph.getGroup(group).getPermissionList(), includeChildren); } - // Add all group permissions, unless negated by direct player perms. + // Add all group permissions, unless negated by earlier permissions. for (String perm : groupPermArray) { + boolean negated = (perm.startsWith("-")); // Perm doesn't already exists and there is no negation for it - // It's not a negated perm where a normal perm exists (don't allow inheritance to negate higher perms) - if ((!playerPermArray.contains(perm)) && (!playerPermArray.contains("-" + perm)) - && (!perm.startsWith("-") && (!playerPermArray.contains(perm.substring(1))))) + // or It's a negated perm where a normal perm doesn't exists (don't allow inheritance to negate higher perms) + if ((!negated && !playerPermArray.contains(perm) && !playerPermArray.contains("-" + perm)) + || (negated && !playerPermArray.contains(perm.substring(1)))) playerPermArray.add(perm); } |