# HG changeset patch
# User Stéphanie Marcu <steph@logilab.fr>
# Date 1277297781 -7200
# Node ID 23ea56123670d02fc5939700ca4c699f9ee1f49c
# Parent  f898991c6de5c9dfa2fd281d8c533773b69dd61f
EntityFormRenderer redefine open_form and add a div section, so redefine close_form to add the closing div tag

diff -r f898991c6de5 -r 23ea56123670 web/views/formrenderers.py
--- a/web/views/formrenderers.py	Wed Jun 23 14:17:57 2010 +0200
+++ b/web/views/formrenderers.py	Wed Jun 23 14:56:21 2010 +0200
@@ -397,10 +397,6 @@
     _options = FormRenderer._options + ('main_form_title',)
     main_form_title = _('main informations')
 
-    def render(self, form, values):
-        rendered = super(EntityFormRenderer, self).render(form, values)
-        return rendered + u'</div>' # close extra div introducted by open_form
-
     def open_form(self, form, values):
         attrs_fs_label = ''
         if self.main_form_title:
@@ -409,6 +405,13 @@
         attrs_fs_label += '<div class="formBody">'
         return attrs_fs_label + super(EntityFormRenderer, self).open_form(form, values)
 
+    def close_form(self, form, values):
+        """seems dumb but important for consistency w/ close form, and necessary
+        for form renderers overriding open_form to use something else or more than
+        and <form>
+        """
+        return super(EntityFormRenderer, self).close_form(form, values) + '</div>'
+
     def render_buttons(self, w, form):
         if len(form.form_buttons) == 3:
             w("""<table width="100%%">