From 493bfc9374d7c9adfd7dcf9625dc1d6024e7cb77 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Fri, 9 Aug 2013 15:45:24 +0200 Subject: [PATCH] [FIX] omagerd don't completely misuse lxml in retarded ways bzr revid: xmo@openerp.com-20130809134524-wldxjgda13sl21fa --- openerp/tools/convert.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/openerp/tools/convert.py b/openerp/tools/convert.py index deed121604f..5447aa2d4a0 100644 --- a/openerp/tools/convert.py +++ b/openerp/tools/convert.py @@ -49,7 +49,7 @@ except: from datetime import datetime, timedelta from dateutil.relativedelta import relativedelta -from lxml import etree +from lxml import etree, builder import misc from config import config from translate import _ @@ -850,7 +850,6 @@ form: module.record_id""" % (xml_id,) el.attrib.pop('id', None) el.tag = 't' - record = etree.Element('record') record_attrs = { 'id': tpl_id, 'model': 'ir.ui.view', @@ -859,11 +858,10 @@ form: module.record_id""" % (xml_id,) if att in el.attrib: record_attrs[att] = el.attrib.pop(att) - record.attrib.update(record_attrs) - record.append(etree.fromstring('%s' % tpl_id)) - record.append(etree.fromstring('qweb')) - record.append(etree.fromstring('')) - record[-1].append(el) + record = etree.Element('record', attrib=record_attrs) + record.append(builder.E.field(str(tpl_id), name='name')) + record.append(builder.E.field("qweb", name='type')) + record.append(builder.E.field(el, name="arch", type="xml")) return self._tag_record(cr, record, data_node) def id_get(self, cr, id_str):