# HG changeset patch
# User sylvain.thenault@logilab.fr
# Date 1235556179 -3600
# Node ID ded89d49c0b4c0ef906d3a5b359a0dd9d1b33757
# Parent  8e4da466efc76fb14b4b51956033d51419fcf49d
xhtml fixes: table has no align attribute, avoid inserting empty table

diff -r 8e4da466efc7 -r ded89d49c0b4 web/views/baseviews.py
--- a/web/views/baseviews.py	Wed Feb 25 11:02:14 2009 +0100
+++ b/web/views/baseviews.py	Wed Feb 25 11:02:59 2009 +0100
@@ -244,7 +244,7 @@
         """
         if hasattr(self, 'get_side_boxes_defs'):
             if rset:
-                self.w(u'<table align="right" width="100%">')
+                self.w(u'<table width="100%">')
                 for label, rset in self.get_side_boxes_defs(entity):
                     self.w(u'<tr><td>')
                     self.w(u'<div class="sideRelated">')
@@ -253,7 +253,7 @@
                     self.w(u'</td></tr>')
                 self.w(u'</table>')
         elif siderelations:
-            self.w(u'<table align="right" width="100%">')
+            self.w(u'<table width="100%">')
             self.w(u'<tr><td>')
             self.w(u'<div class="sideRelated">')
             for relatedinfos in siderelations:
@@ -263,19 +263,21 @@
             self.w(u'</div>')
             self.w(u'</td></tr>')
             self.w(u'</table>')
-        self.w(u'<table  align="right" width="100%">')
-        for box in self.vreg.possible_vobjects('boxes', self.req, self.rset,
-                                               row=self.row, view=self,
-                                               context='incontext'):
-            self.w(u'<tr><td>')
-            try:
-                box.dispatch(w=self.w, row=self.row)
-            except NotImplementedError:
-                # much probably a context insensitive box, which only implements
-                # .call() and not cell_call()
-                box.dispatch(w=self.w)
-            self.w(u'</td></tr>')
-        self.w(u'</table>')
+        boxes = list(self.vreg.possible_vobjects('boxes', self.req, self.rset,
+                                                 row=self.row, view=self,
+                                                 context='incontext'))
+        if boxes:
+            self.w(u'<table width="100%">')
+            for box in boxes:
+                self.w(u'<tr><td>')
+                try:
+                    box.dispatch(w=self.w, row=self.row)
+                except NotImplementedError:
+                    # much probably a context insensitive box, which only implements
+                    # .call() and not cell_call()
+                    box.dispatch(w=self.w)
+                self.w(u'</td></tr>')
+            self.w(u'</table>')
                 
     def is_side_related(self, rschema, eschema):
         return rschema.meta and \