svn commit: r550179 - /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: r550179 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java

jacopoc
Author: jacopoc
Date: Sun Jun 24 01:35:45 2007
New Revision: 550179

URL: http://svn.apache.org/viewvc?view=rev&rev=550179
Log:
The header cells of fields where the title attribute is explicitly set to empty are not rendered.

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?view=diff&rev=550179&r1=550178&r2=550179
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelForm.java Sun Jun 24 01:35:45 2007
@@ -1043,9 +1043,7 @@
                 // only want to display the title header for the main row
                 String modelFormFieldTitle = modelFormField.getTitle(context);
                 if ("".equals(modelFormFieldTitle)) {
-                    // TODO: for now this is commented: there are too many form definitions
-                    //       that needs to be fixed.
-                    //continue;
+                    continue;
                 }
                 // don't do any header for hidden or ignored fields
                 if (fieldInfo.getFieldType() == ModelFormField.FieldInfo.HIDDEN || fieldInfo.getFieldType() == ModelFormField.FieldInfo.IGNORED) {