svn commit: r739195 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r739195 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

hansbak-2
Author: hansbak
Date: Fri Jan 30 07:02:59 2009
New Revision: 739195

URL: http://svn.apache.org/viewvc?rev=739195&view=rev
Log:
another error message improvement on the form type which can be confused with a service attribute type

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java?rev=739195&r1=739194&r2=739195&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java Fri Jan 30 07:02:59 2009
@@ -833,7 +833,11 @@
         } else if ("upload".equals(this.type)) {
             this.renderSingleFormString(writer, context, formStringRenderer, positions);
         } else {
-            throw new IllegalArgumentException("The type " + this.getType() + " is not supported for form with name " + this.getName());
+         if (UtilValidate.isEmpty(this.getType())) {
+                throw new IllegalArgumentException("The form 'type' tag is missing or empty on the form with the name " + this.getName());
+         } else {
+         throw new IllegalArgumentException("The form type " + this.getType() + " is not supported for form with name " + this.getName());
+         }
         }
     }