From af43f27785d75c99d5b35f078d1112f0dbf96358 Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Wed, 23 Apr 2014 18:18:58 -0500
Subject: [PATCH] IDEMPIERE-1912 Document Action Access bug / thanks to hieplq

---
 .../src/org/compiere/model/MRole.java               | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/org.adempiere.base/src/org/compiere/model/MRole.java b/org.adempiere.base/src/org/compiere/model/MRole.java
index 93a8ee1f4d..9a19c7c78c 100644
--- a/org.adempiere.base/src/org/compiere/model/MRole.java
+++ b/org.adempiere.base/src/org/compiere/model/MRole.java
@@ -2581,12 +2581,15 @@ public final class MRole extends X_AD_Role
 				while (rs.next()) {
 					String op = rs.getString(1);
 					String active=rs.getString(2);
-					if ("N".equals(active) && validOptions.contains(op)) {
-						validOptions.remove(op);
+					if ("N".equals(active)) {
+						if (validOptions.contains(op)) {
+							validOptions.remove(op);
+						}
 					} else {
-					  if (!validOptions.contains(op))
-						  validOptions.add(op);
-					} 				
+						if (!validOptions.contains(op)) {
+							validOptions.add(op);
+						}
+					} 
 				}
 			}