[FIX] web: better handling of isDirty state

When the view is saved (either directly or via a widget in readonly
mode), the flag isDirty was not reset to false.

also, use the $ function on widgets insted of $el.find and remove some
unused imports

2 files changed, 3 insertions, 4 deletions

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 4e61d25f468..10d0c06ad76 100644
--- a/addons/web/static/src/js/views/field_manager_mixin.js
+++ b/addons/web/static/src/js/views/field_manager_mixin.js
@@ -9,10 +9,6 @@ odoo.define('web.FieldManagerMixin', function (require) {
  */
 
 var BasicModel = require('web.BasicModel');
-var Dialog = require('web.Dialog');
-var core = require('web.core');
-
-var qweb = core.qweb;
 
 var FieldManagerMixin = {
     custom_events: {
diff --git a/addons/web/static/src/js/views/form/form_controller.js b/addons/web/static/src/js/views/form/form_controller.js
index b4ea40faa3c..f27a47ababc 100644
--- a/addons/web/static/src/js/views/form/form_controller.js
+++ b/addons/web/static/src/js/views/form/form_controller.js
@@ -242,6 +242,7 @@ var FormController = BasicController.extend({
                         if (!stayInEdit) {
                             self._toReadOnlyMode();
                         }
+                        self.isDirty = false;
                     });
             }
         }
@@ -313,6 +314,7 @@ var FormController = BasicController.extend({
      * @param {string} id
      */
     _confirmSave: function (id) {
+        this.isDirty = false;
         if (id === this.handle) {
             this.reload();
         } else {
@@ -461,6 +463,7 @@ var FormController = BasicController.extend({
         var self = this;
         this.canBeDiscarded().then(function () {
             self.model.discardChanges(self.handle);
+            self.isDirty = false;
             if (!self.model.isNew(self.handle)) {
                 self._toReadOnlyMode();
             } else {