svn commit: r1437804 - /archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java

[olamy]

Author: olamy
Date: Wed Jan 23 23:51:18 2013
New Revision: 1437804

URL: http://svn.apache.org/viewvc?rev=1437804&view=rev
Log:
add more caching for ldap roles

Modified:
    
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java

Modified: 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java?rev=1437804&r1=1437803&r2=1437804&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/LdapRbacManager.java
 Wed Jan 23 23:51:18 2013
@@ -364,9 +364,11 @@ public class LdapRbacManager
             {
                 for ( String roleName : roleNames )
                 {
-                    Role role = getRole( roleName );
+                    Role role = this.rbacImpl.getRole( roleName );
+                    role = ( role == null ) ? new RoleImpl( roleName ) : role;
                     if ( role != null )
                     {
+                        rolesCache.put( role.getName(), role );
                         roles.add( role );
                     }
                 }