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

[svn]

Author: prabath
Date: Wed Feb 20 09:54:45 2008
New Revision: 13919

Log:

required field validation 

Modified:
   
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/AddUserProfileSubmitAction.java
   
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/AddUserProfileSubmitAction.java
==============================================================================
--- 
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/AddUserProfileSubmitAction.java
   (original)
+++ 
trunk/solutions/identity/modules/user-ui/src/main/java/org/wso2/solutions/identity/user/ui/action/AddUserProfileSubmitAction.java
   Wed Feb 20 09:54:45 2008
@@ -16,6 +16,7 @@
 import org.wso2.usermanager.UserManagerException;
 
 import com.opensymphony.xwork2.ActionContext;
+import com.opensymphony.xwork2.validator.annotations.RequiredStringValidator;
 
 public class AddUserProfileSubmitAction extends ManagedAction {
 
@@ -47,6 +48,11 @@
             return ERROR;
         }
 
+        if (profileName == null || profileName.trim().length() == 0) {
+            this.addErrorMessage(getText("profile_name_required"));
+            return ERROR;
+        }
+
         claimsAdmin = new ClaimsAdmin();
         claimDOs = claimsAdmin.getAllMappedEnabledClaims();
 
@@ -54,6 +60,14 @@
         for (int i = 0; i < claimDOs.length; i++) {
             String uri = claimDOs[i].getUri();
             if (!uri.equals(IdentityConstants.CLAIM_PPID)) {
+
+                if (!claimDOs[i].isOptional()
+                        && (request.getParameter(uri) == null || request
+                                .getParameter(uri).trim().length() == 0)) {
+                    this.addErrorMessage(getText("required_feilds_missing"));
+                    return ERROR;
+                }
+
                 if (request.getParameter(uri) != null
                         && request.getParameter(uri).trim().length() > 0)
                     props.put(uri, request.getParameter(uri));
@@ -88,6 +102,7 @@
         return SUCCESS;
     }
 
+    @RequiredStringValidator(message = "", key = "profileName")
     public String getProfileName() {
         return profileName;
     }

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
     Wed Feb 20 09:54:45 2008
@@ -70,6 +70,14 @@
         for (int i = 0; i < claimDOs.length; i++) {
             String uri = claimDOs[i].getUri();
             if (!uri.equals(IdentityConstants.CLAIM_PPID)) {
+
+                if (!claimDOs[i].isOptional()
+                        && (request.getParameter(uri) == null || request
+                                .getParameter(uri).trim().length() == 0)) {
+                    this.addErrorMessage(getText("required_feilds_missing"));
+                    return ERROR;
+                }
+
                 if (request.getParameter(uri) != null
                         && request.getParameter(uri).trim().length() > 0)
                     props.put(uri, request.getParameter(uri));

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