summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNecrodoom <doomed.war@gmail.com>2013-06-14 10:00:43 +0300
committerKHobbits <rob@khobbits.co.uk>2014-05-04 12:50:04 +0100
commit454ef4016fb822fff55ef83bf9153c0609535f3a (patch)
treedd463822cb626c6f5bd5ff47ff434bfe6433beaa
parent188c2548fb3a9ce883eaeac5108b0bc19c4537ac (diff)
downloadEssentials-454ef4016fb822fff55ef83bf9153c0609535f3a.tar
Essentials-454ef4016fb822fff55ef83bf9153c0609535f3a.tar.gz
Essentials-454ef4016fb822fff55ef83bf9153c0609535f3a.tar.lz
Essentials-454ef4016fb822fff55ef83bf9153c0609535f3a.tar.xz
Essentials-454ef4016fb822fff55ef83bf9153c0609535f3a.zip
Improve Javadocs, clean ' strip checks, update changelog.
-rw-r--r--EssentialsGroupManager/src/Changelog.txt5
-rw-r--r--EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java45
-rw-r--r--EssentialsGroupManager/src/plugin.yml8
3 files changed, 16 insertions, 42 deletions
diff --git a/EssentialsGroupManager/src/Changelog.txt b/EssentialsGroupManager/src/Changelog.txt
index 885aff9ca..fcdd930f0 100644
--- a/EssentialsGroupManager/src/Changelog.txt
+++ b/EssentialsGroupManager/src/Changelog.txt
@@ -212,7 +212,7 @@ v 2.0:
- Add support for BukkitForge using 'overworld' as the main world name.
- Prevent '*' permissions granting the 'groupmanager.noofflineperms' permission.
- Added '/mancheckw <world>' to inspect which permission files a world is referencing.
- - Add config potion to set if GM commands should be allowed on CommnandBlocks.
+ - Add config potion to set if GM commands should be allowed on CommnandBlocks.
- Catch the error when using an out of date config for 'allow_commandblocks' So it doesn't kill the whole config.
- '/manselect' will no longer list duplicate worlds.
- Added a new mirroring option in the config of 'all_unnamed_worlds'. This will cause all new or unnamed worlds to use this mirroring.
@@ -220,4 +220,5 @@ v 2.0:
- Warn when adding a node where an exception already exist.
- Only prevent adding nodes with '/manuaddp' and '/mangaddp' if they are exact matches (not wildcards).
- Store worldSelection indexed on the senders name rather than the object (fixes commandblocks using manselect).
- - Check subgroup permissions with an equal priority so no one subgroup is higher ranked than another. \ No newline at end of file
+ - Check subgroup permissions with an equal priority so no one subgroup is higher ranked than another.
+ - add recursive permission adding/deleting
diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java
index dbed96909..7552c2758 100644
--- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java
+++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/GroupManager.java
@@ -738,11 +738,7 @@ public class GroupManager extends JavaPlugin {
for (int i = 1; i < args.length; i++)
{
- auxString = args[i];
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = args[i].replace("'", "");
permissionResult = permissionHandler.checkFullUserPermission(senderUser, auxString);
if (!isConsole && !isOpOverride && (permissionResult.resultType.equals(PermissionCheckResult.Type.NOTFOUND) || permissionResult.resultType.equals(PermissionCheckResult.Type.NEGATION))) {
@@ -791,11 +787,7 @@ public class GroupManager extends JavaPlugin {
for (int i = 1; i < args.length; i++)
{
- auxString = args[i];
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = args[i].replace("'", "");
if (!isConsole && !isOpOverride && (senderGroup != null ? permissionHandler.inGroup(auxUser.getName(), senderGroup.getName()) : false)) {
sender.sendMessage(ChatColor.RED + "You can't modify a player with same group as you, or higher.");
@@ -950,11 +942,7 @@ public class GroupManager extends JavaPlugin {
return true;
}
- auxString = args[1];
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = args[1].replace("'", "");
if ((validateOnlinePlayer) && ((match = validatePlayer(args[0], sender)) == null)) {
return false;
@@ -1019,11 +1007,7 @@ public class GroupManager extends JavaPlugin {
for (int i = 1; i < args.length; i++)
{
- auxString = args[i];
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = args[i].replace("'", "");
// Validating your permissions
permissionResult = permissionHandler.checkFullUserPermission(senderUser, auxString);
@@ -1065,11 +1049,7 @@ public class GroupManager extends JavaPlugin {
}
for (int i = 1; i < args.length; i++)
{
- auxString = args[i];
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = args[i].replace("'", "");
// Validating your permissions
permissionResult = permissionHandler.checkFullUserPermission(senderUser, auxString);
@@ -1333,10 +1313,7 @@ public class GroupManager extends JavaPlugin {
auxString += " ";
}
}
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+ auxString = auxString.replace("'", "");
auxUser.getVariables().addVar(args[1], Variables.parseVariableValue(auxString));
sender.sendMessage(ChatColor.YELLOW + "Variable " + ChatColor.GOLD + args[1] + ChatColor.YELLOW + ":'" + ChatColor.GREEN + auxString + ChatColor.YELLOW + "' added to the user " + auxUser.getName());
@@ -1485,10 +1462,8 @@ public class GroupManager extends JavaPlugin {
auxString += " ";
}
}
- if (auxString.startsWith("'") && auxString.endsWith("'"))
- {
- auxString = auxString.substring(1, auxString.length() - 1);
- }
+
+ auxString = auxString.replace("'", "");
auxGroup.getVariables().addVar(args[1], Variables.parseVariableValue(auxString));
sender.sendMessage(ChatColor.YELLOW + "Variable " + ChatColor.GOLD + args[1] + ChatColor.YELLOW + ":'" + ChatColor.GREEN + auxString + ChatColor.YELLOW + "' added to the group " + auxGroup.getName());
@@ -2097,7 +2072,7 @@ public class GroupManager extends JavaPlugin {
}
/**
- * Checks if a permission exists and of a lower priority.
+ * Checks if a permission exists and of a lower or same priority.
*/
private boolean checkPermissionExists(CommandSender sender, String newPerm, PermissionCheckResult oldPerm, String type) {
@@ -2132,13 +2107,11 @@ public class GroupManager extends JavaPlugin {
{
sender.sendMessage(ChatColor.RED + "The " + type + " already has an exception for this node.");
sender.sendMessage(ChatColor.RED + "Node: " + oldPerm.accessLevel);
- return true;
}
else if (oldPerm.resultType.equals(PermissionCheckResult.Type.NEGATION))
{
sender.sendMessage(ChatColor.RED + "The " + type + " already has a matching negated node.");
sender.sendMessage(ChatColor.RED + "Node: " + oldPerm.accessLevel);
- return true;
}
else if (oldPerm.resultType.equals(PermissionCheckResult.Type.FOUND))
{
diff --git a/EssentialsGroupManager/src/plugin.yml b/EssentialsGroupManager/src/plugin.yml
index c3443f3bd..9e4e4e910 100644
--- a/EssentialsGroupManager/src/plugin.yml
+++ b/EssentialsGroupManager/src/plugin.yml
@@ -34,11 +34,11 @@ commands:
permissions: groupmanager.mangdel
manuaddp:
description: Add permissions directly to the player.
- usage: /<command> <player> <permissions>
+ usage: /<command> <player> <permission> [permission2] [permission3]..
permissions: groupmanager.manuaddp
manudelp:
description: Removes permissions directly from the player.
- usage: /<command> <player> <permissions>
+ usage: /<command> <player> <permission> [permission2] [permission3]..
permissions: groupmanager.manudelp
manuclearp:
description: Removes all permissions of a player.
@@ -54,11 +54,11 @@ commands:
permissions: groupmanager.manucheckp
mangaddp:
description: Add permissions to a group.
- usage: /<command> <group> <permissions>
+ usage: /<command> <group> <permission> [permission2] [permission3]..
permissions: groupmanager.mangaddp
mangdelp:
description: Removes permissions from a group.
- usage: /<command> <group> <permissions>
+ usage: /<command> <group> <permission> [permission2] [permission3]..
permissions: groupmanager.mangdelp
mangclearp:
description: Removes all permissions of a group.