From 6efd3eaa27a87fd41046d2ddc9bb96f6508d6851 Mon Sep 17 00:00:00 2001 From: ElgarL Date: Sat, 3 Mar 2012 13:47:59 +0000 Subject: [PATCH] Fix a silly logic error when testing bukkit perms --- .../src/org/anjocaido/groupmanager/data/Group.java | 2 +- .../groupmanager/permissions/AnjoPermissionsHandler.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/Group.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/Group.java index 396240ec6..56ef7e870 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/Group.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/Group.java @@ -52,7 +52,7 @@ public class Group extends DataUnit implements Cloneable { /** * Is this a GlobalGroup * - * @return + * @return true if this is a global group */ public boolean isGlobal() { return (getDataSource() == null); diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java index b9315e849..6b6ae58ea 100644 --- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java +++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java @@ -766,7 +766,7 @@ public class AnjoPermissionsHandler extends PermissionsReaderInterface { // (Heroes). final Player player = user.getBukkitPlayer(); //final Permission bukkitPerm = Bukkit.getPluginManager().getPermission(targetPermission); - if (player != null && player.hasPermission(targetPermission)) { + if ((player != null) && player.hasPermission(targetPermission)) { result.resultType = PermissionCheckResult.Type.FOUND; result.owner = user; return result;