diff options
author | ElgarL <ElgarL@Palmergames.com> | 2013-07-28 09:10:23 +0100 |
---|---|---|
committer | KHobbits <rob@khobbits.co.uk> | 2014-05-04 12:50:09 +0100 |
commit | b4a90f3b175c796fa87d1c3dd17ebb6f513717bb (patch) | |
tree | 635b6f3f8b4a488545fe548d33bbfa081207db0e /EssentialsGroupManager/src/org | |
parent | 601b3621598a50df65cf17d42928da130a0dd4dc (diff) | |
download | Essentials-b4a90f3b175c796fa87d1c3dd17ebb6f513717bb.tar Essentials-b4a90f3b175c796fa87d1c3dd17ebb6f513717bb.tar.gz Essentials-b4a90f3b175c796fa87d1c3dd17ebb6f513717bb.tar.lz Essentials-b4a90f3b175c796fa87d1c3dd17ebb6f513717bb.tar.xz Essentials-b4a90f3b175c796fa87d1c3dd17ebb6f513717bb.zip |
Prevent adding sub groups for ranks the granting player doesn't have access to.
Diffstat (limited to 'EssentialsGroupManager/src/org')
-rw-r--r-- | EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java index dcbfbd326..24a70d465 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java @@ -611,6 +611,14 @@ public class GroupManager extends JavaPlugin { sender.sendMessage(ChatColor.RED + "You can't modify a player with same permissions as you, or higher."); return true; } + if (!isConsole && !isOpOverride && (permissionHandler.hasGroupInInheritance(auxGroup, senderGroup.getName()))) { + sender.sendMessage(ChatColor.RED + "The sub-group can't be the same as yours, or higher."); + return true; + } + if (!isConsole && !isOpOverride && (!permissionHandler.inGroup(senderUser.getName(), auxUser.getGroupName()) || !permissionHandler.inGroup(senderUser.getName(), auxGroup.getName()))) { + sender.sendMessage(ChatColor.RED + "You can't modify a player involving a group that you don't inherit."); + return true; + } // Seems OK if (auxUser.addSubGroup(auxGroup)) sender.sendMessage(ChatColor.YELLOW + "You added subgroup '" + auxGroup.getName() + "' to player '" + auxUser.getName() + "'."); @@ -993,7 +1001,7 @@ public class GroupManager extends JavaPlugin { } // Validating arguments if (args.length < 2) { - sender.sendMessage(ChatColor.RED + "Review your arguments count! (/mangaaddp <group> <permission> [permission2] [permission3]...)"); + sender.sendMessage(ChatColor.RED + "Review your arguments count! (/mangaddp <group> <permission> [permission2] [permission3]...)"); return true; } |