From eb95cbeeff916486c3538f38474a7ef597682dc1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Mon, 21 Nov 2016 20:59:33 +0100
Subject: [PATCH] Remove old record in One2Many._set_value only if it is not in
 the new value

Always removing previous record makes the parent record being wrongly
considered as modified.

issue6044
review34451002
---
 src/model.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/model.js b/src/model.js
index b3544ec5e7..6f7d802488 100644
--- a/src/model.js
+++ b/src/model.js
@@ -1898,7 +1898,9 @@
                 if (mode == 'list ids') {
                     for (var i = 0, len = group.length; i < len; i++) {
                         var old_record = group[i];
-                        group.remove(old_record, true);
+                        if (!~value.indexOf(old_record.id)) {
+                            group.remove(old_record, true);
+                        }
                     }
                     group.load(value);
                 } else {