From 020f0b2ad6869cb3867b62a06b575ab771472a29 Mon Sep 17 00:00:00 2001
From: ElgarL <ElgarL@Palmergames.com>
Date: Thu, 27 Mar 2014 03:48:20 +0000
Subject: Fix manucheckp to understand and report correctly on Exception nodes.

---
 .../permissions/AnjoPermissionsHandler.java        | 39 ++++++++++++++++------
 1 file changed, 28 insertions(+), 11 deletions(-)

(limited to 'EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java')

diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
index 8f8f4c599..6eda3f1cc 100644
--- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
+++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/permissions/AnjoPermissionsHandler.java
@@ -898,32 +898,49 @@ public class AnjoPermissionsHandler extends PermissionsReaderInterface {
 
 		PermissionCheckResult resultUser = checkUserOnlyPermission(user, targetPermission);
 		if (resultUser.resultType != PermissionCheckResult.Type.NOTFOUND) {
+			
 			resultUser.accessLevel = targetPermission;
-			return resultUser;
+			
+			if (resultUser.resultType == PermissionCheckResult.Type.EXCEPTION) {
+				return resultUser;
+			}
+			
+			result = resultUser;
+			
 		}
 
-		// IT ONLY CHECKS GROUPS PERMISSIONS IF RESULT FOR USER IS NOT FOUND
+		// IT ONLY CHECKS GROUPS PERMISSIONS IF RESULT FOR USER IS NOT AN EXCEPTION
 		PermissionCheckResult resultGroup = checkGroupPermissionWithInheritance(user.getGroup(), targetPermission);
 		if (resultGroup.resultType != PermissionCheckResult.Type.NOTFOUND) {
+			
 			resultGroup.accessLevel = targetPermission;
-			return resultGroup;
+			
+			if (resultGroup.resultType == PermissionCheckResult.Type.EXCEPTION) {
+				return resultGroup;
+			}
+			
+			result = resultGroup;
+			
 		}
 
 		// SUBGROUPS CHECK
 		for (Group subGroup : user.subGroupListCopy()) {
-			PermissionCheckResult resultSubGroup = checkGroupPermissionWithInheritance(subGroup, targetPermission);
-			
 			
+			PermissionCheckResult resultSubGroup = checkGroupPermissionWithInheritance(subGroup, targetPermission);
 			if (resultSubGroup.resultType != PermissionCheckResult.Type.NOTFOUND) {
 				
-				if ((resultSubGroup.resultType == PermissionCheckResult.Type.FOUND) && (result.resultType != PermissionCheckResult.Type.NEGATION)) {
-					resultSubGroup.accessLevel = targetPermission;
-					result = resultSubGroup;
-				} else if (resultSubGroup.resultType == PermissionCheckResult.Type.EXCEPTION) {
-					resultSubGroup.accessLevel = targetPermission;
+				resultSubGroup.accessLevel = targetPermission;
+				
+				if (resultSubGroup.resultType == PermissionCheckResult.Type.EXCEPTION) {
+					
 					return resultSubGroup;
+					
+				} else if ((resultSubGroup.resultType == PermissionCheckResult.Type.FOUND) && (result.resultType != PermissionCheckResult.Type.NEGATION)) {
+					
+					result = resultSubGroup;
+					
 				} else if (resultSubGroup.resultType == PermissionCheckResult.Type.NEGATION) {
-					resultSubGroup.accessLevel = targetPermission;
+					
 					result = resultSubGroup;
 				}
 				
-- 
cgit v1.2.3