From 1ca1c35fc864647fa5c9a6d9bc7ea0d647475211 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Fri, 4 Dec 2020 18:37:36 +0100
Subject: [PATCH] Return false when modified is not saved and not created

The return of modified_save method is used to decide to execute an action or
not. When a new record is cancelled, the client select the previous record so
we must not run the action on this one.

issue9881
review337111002
---
 src/tab.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/tab.js b/src/tab.js
index 81225b3726..83d4dc861d 100644
--- a/src/tab.js
+++ b/src/tab.js
@@ -734,7 +734,13 @@
                             case 'ok':
                                 return this.save();
                             case 'ko':
-                                return this.reload(false);
+                                var record_id = this.screen.current_record.id;
+                                return this.reload(false).then(function() {
+                                    if (record_id !=
+                                        this.screen.current_record.id) {
+                                        return jQuery.Deferred().reject();
+                                    }
+                                }.bind(this));
                             default:
                                 return jQuery.Deferred().reject();
                         }