From 54968d55c243d9890804305915ee72b6fe44c2ae Mon Sep 17 00:00:00 2001 From: Denis Ledoux Date: Wed, 14 Sep 2016 13:04:49 +0200 Subject: [PATCH] [FIX] tools: do not overwrite view priority on update In some cases, users change the sequence of base views in order to have one applied before the other e.g. when there is two inherited views doing ``` ... ``` The priority of the views will have an impact on the fields order in the tree. On update, the change of priority done by the users was overwritten, even if the priority of the view was not defined in the XML code of the view. The priority of the views should be overwritten only in the case were the priority of the view is specifically set in its definition. opw-688470 --- openerp/tools/convert.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/openerp/tools/convert.py b/openerp/tools/convert.py index 314d946cd69..3fcd781e4a0 100644 --- a/openerp/tools/convert.py +++ b/openerp/tools/convert.py @@ -741,7 +741,8 @@ form: module.record_id""" % (xml_id,) record.append(Field(name, name='name')) record.append(Field(full_tpl_id, name='key')) record.append(Field("qweb", name='type')) - record.append(Field(el.get('priority', "16"), name='priority')) + if 'priority' in el.attrib: + record.append(Field(el.get('priority'), name='priority')) if 'inherit_id' in el.attrib: record.append(Field(name='inherit_id', ref=el.get('inherit_id'))) if 'website_id' in el.attrib: