You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by le...@apache.org on 2010/04/07 03:06:14 UTC

svn commit: r931393 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

Author: lektran
Date: Wed Apr  7 01:06:14 2010
New Revision: 931393

URL: http://svn.apache.org/viewvc?rev=931393&view=rev
Log:
Cleaned up renderContainerFindField [sic] to use the writer directly

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?rev=931393&r1=931392&r2=931393&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java Wed Apr  7 01:06:14 2010
@@ -2909,11 +2909,14 @@ public class HtmlFormRenderer extends Ht
     }
     
     public void renderContainerFindField(Appendable writer, Map<String, Object> context, ContainerField containerField) throws IOException {
-        String id = "";
-        if (UtilValidate.isNotEmpty(containerField.getId())) {
-            id = containerField.getId();
+        writer.append("<div ");
+        String id = containerField.getId();
+        if (UtilValidate.isNotEmpty(id)) {
+            writer.append("id=\"");
+            writer.append(id);
+            writer.append("\"");
         }
-        writer.append("<div id=\"" + id + "\"/>");
+        writer.append("/>");
     }
 
     /** Create an ajaxXxxx JavaScript CSV string from a list of UpdateArea objects. See