Author: sohil.shah(a)jboss.com Date: 2009-08-02 10:42:03 -0400 (Sun, 02 Aug 2009) New Revision: 13652 Modified: jbossexo/branches/security-integration-sandbox/portal/trunk/component/portal/src/main/java/org/exoplatform/portal/jboss/security/provisioning/ExoPolicyProvisioner.java Log: UserACL adapted to switch between jboss security impl and exo impl via configuration * bug fix Modified: jbossexo/branches/security-integration-sandbox/portal/trunk/component/portal/src/main/java/org/exoplatform/portal/jboss/security/provisioning/ExoPolicyProvisioner.java =================================================================== --- jbossexo/branches/security-integration-sandbox/portal/trunk/component/portal/src/main/java/org/exoplatform/portal/jboss/security/provisioning/ExoPolicyProvisioner.java 2009-08-01 20:40:35 UTC (rev 13651) +++ jbossexo/branches/security-integration-sandbox/portal/trunk/component/portal/src/main/java/org/exoplatform/portal/jboss/security/provisioning/ExoPolicyProvisioner.java 2009-08-02 14:42:03 UTC (rev 13652) @@ -377,11 +377,17 @@ for(String portalCreatorGroup: this.portalCreatorGroups) { - portalCreators.addName(portalCreatorGroup); + if(portalCreatorGroup != null && portalCreatorGroup.trim().length()!=0) + { + portalCreators.addName(portalCreatorGroup); + } } - context.addPolicyRule(Effect.PERMIT, action, portalCreators, - "allowExpression"); + if(!portalCreators.isEmpty()) + { + context.addPolicyRule(Effect.PERMIT, action, portalCreators, + "allowExpression"); + } } this.policyProvisioner.composeAndDeploy(context);