[Identity-dev] svn commit r13771 - trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action

[svn]

Author: prabath
Date: Thu Feb 14 09:16:34 2008
New Revision: 13771

Log:

added a check to avoid empty values being added for optional attributes during 
user registration

Modified:
   
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/UserRegistrationFormSubmitAction.java

Modified: 
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/UserRegistrationFormSubmitAction.java
==============================================================================
--- 
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/UserRegistrationFormSubmitAction.java
     (original)
+++ 
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/UserRegistrationFormSubmitAction.java
     Thu Feb 14 09:16:34 2008
@@ -70,7 +70,9 @@
         for (int i = 0; i < claimDOs.length; i++) {
             String uri = claimDOs[i].getUri();
             if (!uri.equals(IdentityConstants.CLAIM_PPID)) {
-                props.put(uri, request.getParameter(uri));
+                if (request.getParameter(uri) != null
+                        && request.getParameter(uri).trim().length() > 0)
+                    props.put(uri, request.getParameter(uri));
             }
 
             if (uri.equals(IdentityConstants.CLAIM_EMAIL_ADDRESS)) {
@@ -83,7 +85,7 @@
                 
.getParameter(IdentityProviderConstants.PARAM_NAME_ENABLE_OPENID_REGISTRATION) 
!= null;
 
         String openid = null;
-        
+
         if (enableOpenIDRegistration) {
             // We create an OpenID for all registering users
             openid = OpenIDProvider.generateOpenID(this.username);

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