[Yanel-commits] rev 23667 - public/yanel/trunk/src/resources/add-realm/src/java/org/wyona/yanel/impl/resources

michi at wyona.com michi at wyona.com
Wed Apr 11 22:16:15 CEST 2007


Author: michi
Date: 2007-04-11 22:16:13 +0200 (Wed, 11 Apr 2007)
New Revision: 23667

Modified:
   public/yanel/trunk/src/resources/add-realm/src/java/org/wyona/yanel/impl/resources/AddRealmResource2.java
Log:
validation started

Modified: public/yanel/trunk/src/resources/add-realm/src/java/org/wyona/yanel/impl/resources/AddRealmResource2.java
===================================================================
--- public/yanel/trunk/src/resources/add-realm/src/java/org/wyona/yanel/impl/resources/AddRealmResource2.java	2007-04-11 20:10:12 UTC (rev 23666)
+++ public/yanel/trunk/src/resources/add-realm/src/java/org/wyona/yanel/impl/resources/AddRealmResource2.java	2007-04-11 20:16:13 UTC (rev 23667)
@@ -190,19 +190,25 @@
 
         Element parameterElement = null;
         Parameter para = null;
+        boolean valid = true;
 
         // Parameter "realmid"
         para = getParameterFromResourceConfig("realmid");
         parameterElement = (Element) fromScratchElement.appendChild(doc.createElementNS(NAMESPACE, "parameter"));
         parameterElement.setAttributeNS(NAMESPACE, "name", para.name);
         parameterElement.setAttributeNS(NAMESPACE, "sample-value", para.sampleValue);
-        if (request.getParameter("from-scratch") != null) {
+        if (request.getParameter("submit-from-scratch") != null) {
             String realmIdValue = request.getParameter("realmid");
             if (realmIdValue != null) {
-                // TODO: validate value ...
-                parameterElement.setAttributeNS(NAMESPACE, "value", realmIdValue);
+                valid = valid && validateRealmId(realmIdValue);
+                if (validateRealmId(realmIdValue)) {
+                    parameterElement.setAttributeNS(NAMESPACE, "value", realmIdValue);
+                } else {
+                    parameterElement.setAttributeNS(NAMESPACE, "exception", "Something is wrong ...!");
+                }
             } else {
-                parameterElement.setAttributeNS(NAMESPACE, "exception", "required");
+                parameterElement.setAttributeNS(NAMESPACE, "exception", "NullPointer");
+                valid = valid && false;
             }
         }
 
@@ -265,4 +271,13 @@
             return null;
         }
     }
+
+    /**
+     *
+     */
+    private boolean validateRealmId(String value) {
+        if (value.length() < 1) return false;
+        // TODO: Check for whitespace ...
+        return true;
+    }
 }




More information about the Yanel-commits mailing list