[seam-commits] Seam SVN: r8174 - trunk/src/main/org/jboss/seam/security/permission.

[seam-commits at lists.jboss.org]

Author: shane.bryzak at jboss.com
Date: 2008-05-12 19:03:14 -0400 (Mon, 12 May 2008)
New Revision: 8174

Modified:
   trunk/src/main/org/jboss/seam/security/permission/PersistentPermissionResolver.java
Log:
don't return if there are more permissions to check

Modified: trunk/src/main/org/jboss/seam/security/permission/PersistentPermissionResolver.java
===================================================================
--- trunk/src/main/org/jboss/seam/security/permission/PersistentPermissionResolver.java	2008-05-12 22:54:07 UTC (rev 8173)
+++ trunk/src/main/org/jboss/seam/security/permission/PersistentPermissionResolver.java	2008-05-12 23:03:14 UTC (rev 8174)
@@ -96,7 +96,7 @@
             if (role.isConditional())
             {
                RuleBasedPermissionResolver resolver = RuleBasedPermissionResolver.instance();
-               return resolver.checkConditionalRole(role.getName(), target, action);
+               if (resolver.checkConditionalRole(role.getName(), target, action)) return true;               
             }
             else if (identity.hasRole(role.getName()))
             {