[Erp5-report] r31214 fabien - /erp5/trunk/products/ERP5OOo/FormPrintout.py

[nobody at svn.erp5.org]

Author: fabien
Date: Thu Dec 10 11:14:53 2009
New Revision: 31214

URL: http://svn.erp5.org?rev=31214&view=rev
Log:
fix a mistake (s/render_odt/render_odg/g) and replace the node only if it's not
a None node. Currently, this can exists because render_odg is under developpment,
so there is no render_odg for all fields yet (if render_odg is not overloaded
for each field, the default method return None).

Modified:
    erp5/trunk/products/ERP5OOo/FormPrintout.py

Modified: erp5/trunk/products/ERP5OOo/FormPrintout.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5OOo/FormPrintout.py?rev=31214&r1=31213&r2=31214&view=diff
==============================================================================
--- erp5/trunk/products/ERP5OOo/FormPrintout.py [utf8] (original)
+++ erp5/trunk/products/ERP5OOo/FormPrintout.py [utf8] Thu Dec 10 11:14:53 2009
@@ -957,5 +957,7 @@
       for target_node in node_list:
         attr_dict = {}
         attr_dict.update(target_node.attrib)
-        new_node = field.render_odt(attr_dict=attr_dict)
-        parent_node = target_node.getparent().replace(target_node, new_node)
+        new_node = field.render_odg(attr_dict=attr_dict)
+        if new_node is not None: # XXX None node exists because render_odg is
+                                 # not yet writed for all fields, but it should
+          parent_node = target_node.getparent().replace(target_node, new_node)