[FIX] web: apply onchange even if warning

The onchange route returns field values, but it can also return
a warning, that needs to be displayed in a dialog. When it does,
the new values must be applied anyway, which wasn't the case
before this rev.

We generalized the 'warning' custom event handled by the webclient
so that it can display a warning in a dialog (and not only through
the NotificationManager).

2 files changed, 5 insertions, 7 deletions

diff --git a/addons/web/static/src/js/views/basic/basic_model.js b/addons/web/static/src/js/views/basic/basic_model.js
index 5f1e6ccbb9c..b16bf1e10b0 100644
--- a/addons/web/static/src/js/views/basic/basic_model.js
+++ b/addons/web/static/src/js/views/basic/basic_model.js
@@ -969,7 +969,11 @@ var BasicModel = AbstractModel.extend({
             var args = [idList, currentData, fields, onchange_spec, context];
             return self.performModelRPC(record.model, 'onchange', args, {}).then(function (result) {
                 if (result.warning) {
-                    return $.Deferred().reject(result.warning);
+                    self.trigger_up('warning', {
+                        message: result.warning.message,
+                        title: result.warning.title,
+                        type: 'dialog',
+                    });
                 }
                 var defs = [];
                 _.each(result.value, function (val, name) {
diff --git a/addons/web/static/src/js/views/field_manager_mixin.js b/addons/web/static/src/js/views/field_manager_mixin.js
index f110f9be8bf..00607157a1f 100644
--- a/addons/web/static/src/js/views/field_manager_mixin.js
+++ b/addons/web/static/src/js/views/field_manager_mixin.js
@@ -86,12 +86,6 @@ var FieldManagerMixin = {
             } else {
                 self._confirmChange(dataPointID, result, event);
             }
-        }).fail(function (warning) {
-            new Dialog(self, {
-                size: 'medium',
-                title: warning.title,
-                $content: qweb.render("CrashManager.warning", warning)
-            }).open();
         });
     },
     /**