Author: tyrell
Date: Tue Mar 18 08:55:43 2008
New Revision: 14967

Log:

Applying performance fixes.

Modified:
   
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASStrategyImpl.java
   
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreReader.java

Modified: 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASStrategyImpl.java
==============================================================================
--- 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASStrategyImpl.java
 (original)
+++ 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASStrategyImpl.java
 Tue Mar 18 08:55:43 2008
@@ -232,24 +232,13 @@
     
 
     public synchronized String[] getUserRoles(String userName) throws 
UserStoreException {
-        String[] names = new String[0];
+        String[] roles;
         try {
-            UserAdmin admin = new UserAdmin();
-            UserData[] data =  admin.getUserNames();
-            for(int i=0;i<data.length;i++){
-                if(data[i].getUserName().equals(userName)){
-                    RoleData[] roles = data[i].getRoles();
-                    names = new String[roles.length];
-                    for(int j=0; j<roles.length; j++){
-                        names[j] = roles[j].getRole();
-                    }
-                    break;
-                }
-            }
+            roles = (new UserAdmin()).getUserRoles(userName);
         } catch (AxisFault e) {
             throw new UserStoreException(e.getMessage());
         }
-        return names;
+        return roles;
     }
 
 }

Modified: 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreReader.java
==============================================================================
--- 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreReader.java
      (original)
+++ 
trunk/mashup/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreReader.java
      Tue Mar 18 08:55:43 2008
@@ -215,24 +215,13 @@
     }
 
     public String[] getUserRoles(String userName) throws UserStoreException {
-        String[] names = new String[0];
+        String[] roles;
         try {
-            UserAdmin admin = new UserAdmin();
-            UserData[] data =  admin.getUserNames();
-            for(int i=0;i<data.length;i++){
-                if(data[i].getUserName().equals(userName)){
-                    RoleData[] roles = data[i].getRoles();
-                    names = new String[roles.length];
-                    for(int j=0; j<roles.length; j++){
-                        names[j] = roles[j].getRole();
-                    }
-                    break;
-                }
-            }
+            roles = (new UserAdmin()).getUserRoles(userName);
         } catch (AxisFault e) {
             throw new UserStoreException(e.getMessage());
         }
-        return names;
+        return roles;
     }
 
     public String[] getUsersInRole(String roleName) throws UserStoreException {

_______________________________________________
Mashup-dev mailing list
[email protected]
http://www.wso2.org/cgi-bin/mailman/listinfo/mashup-dev

Reply via email to