[Identity-dev] svn commit r13588 - trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/openid

[svn]

Author: prabath
Date: Mon Feb 11 07:16:21 2008
New Revision: 13588

Log:

code review fixes

Modified:
   
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/openid/OpenIDProvider.java

Modified: 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/openid/OpenIDProvider.java
==============================================================================
--- 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/openid/OpenIDProvider.java
     (original)
+++ 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/openid/OpenIDProvider.java
     Mon Feb 11 07:16:21 2008
@@ -286,13 +286,13 @@
             String user = (String) iterator.next();
             mapValues = userStore.getClaimValues(user, null);
 
-            if (mapValues != null) {
+            if (mapValues != null && !mapValues.isEmpty()) {
 
                 // User has defined claims!
                 String claimId = (String) mapValues
                         .get(IdentityConstants.CLAIM_OPENID);
 
-                if (mapValues != null && !mapValues.isEmpty() && 
claimId!=null) {
+                if (claimId != null) {
                     if (openId.indexOf(claimId) >= 0
                             && openId.endsWith(claimId.substring(claimId
                                     .length() - 1))) {

_______________________________________________
Identity-dev mailing list
Identity-dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/identity-dev