From 2ba9526d372bee4b9e6ccf2d377c9b2789397bde Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Fri, 31 Jan 2014 22:26:55 +0100
Subject: [PATCH] Prevent unecessary update of record when loading XML

review3061003
---
 trytond/convert.py | 10 ++++++----
 trytond/res/ir.xml |  4 ++--
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/trytond/convert.py b/trytond/convert.py
index 36d5950ac8..4a56439787 100644
--- a/trytond/convert.py
+++ b/trytond/convert.py
@@ -137,6 +137,9 @@ class MenuitemTagHandler:
             else:
                 values['name'] = action_name
 
+        if values.get('sequence'):
+            values['sequence'] = int(values['sequence'])
+
         self.values = values
 
     def characters(self, data):
@@ -752,10 +755,9 @@ class TrytondXmlHandler(sax.handler.ContentHandler):
         actions = iter(args)
         for record, values, old_values, fs_id, mdata_id in zip(
                 *((actions,) * 5)):
-            if module != self.module:
-                temp_values = old_values.copy()
-                temp_values.update(values)
-                values = temp_values
+            temp_values = old_values.copy()
+            temp_values.update(values)
+            values = temp_values
 
             if values != old_values:
                 self.grouped_model_data.extend(([self.ModelData(mdata_id)], {
diff --git a/trytond/res/ir.xml b/trytond/res/ir.xml
index 45c6557306..73c8d96fcc 100644
--- a/trytond/res/ir.xml
+++ b/trytond/res/ir.xml
@@ -787,9 +787,9 @@ this repository contains the full copyright notices and license terms. -->
             <field name="request_user" ref="user_admin"/>
             <field name="user" ref="user_trigger"/>
             <field name="active" eval="True"/>
-            <field name="interval_number">5</field>
+            <field name="interval_number" eval="5"/>
             <field name="interval_type">minutes</field>
-            <field name="number_calls">-1</field>
+            <field name="number_calls" eval="-1"/>
             <field name="repeat_missed" eval="False"/>
             <field name="model">ir.trigger</field>
             <field name="function">trigger_time</field>