From 206453c83689ad0dc6ceec3686153ab79bf88660 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?G=C3=A9ry=20Debongnie?= <ged@odoo.com>
Date: Mon, 27 Feb 2017 19:39:41 +0100
Subject: [FIX] web: various fixes with one2many/formviewdialogs

We share the basic model record between views, and we have to make sure
that everything is properly coordinated.  The main issue solved by this
commit is the fact that we can now edit a one2many record, then close
the dialog, then edit it again and finally cancel the changes, which
bring us to the previous step.

Also, we had to slightly improve the AbstractView API
---
 .../web/static/src/js/views/basic/basic_model.js   | 27 ++++++++++++++++++++++
 1 file changed, 27 insertions(+)

(limited to 'addons/web/static/src/js/views/basic/basic_model.js')

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 785cf79775b..bc36f454914 100644
--- a/addons/web/static/src/js/views/basic/basic_model.js
+++ b/addons/web/static/src/js/views/basic/basic_model.js
@@ -403,6 +403,7 @@ var BasicModel = AbstractModel.extend({
      *   of all fields by default
      * @param {Object} params.fields contains the description of each field
      * @param {string} [params.type] 'record' or 'list'
+     * @param {string} [params.recordID] an ID for an existing resource.
      * @returns {Deferred -> string} resolves to a local id, or handle
      */
     load: function (params) {
@@ -733,11 +734,22 @@ var BasicModel = AbstractModel.extend({
      * @param {string} record_id local resource
      * @param {Object} [options]
      * @param {boolean} [options.reload=true] if true, data will be reloaded
+     * @param {boolean} [options.localSave=false] if true, the record will only
+     *   be 'locally' saved: its changes will move from the _changes key to the
+     *   data key
      * @returns {Deferred}
      */
     save: function (record_id, options) {
         var self = this;
         options = options || {};
+        if (options.localSave) {
+            var record = self.localData[record_id];
+            if (record._changes) {
+                _.extend(record.data, record._changes);
+                record._changes = null;
+            }
+            return $.when();
+        }
         var shouldReload = 'reload' in options ? options.reload : true;
         return this.mutex.exec(function () {
             var record = self.localData[record_id];
@@ -799,6 +811,21 @@ var BasicModel = AbstractModel.extend({
             }
         });
     },
+    /**
+     * Set and return the fieldattrs on a record element.
+     * It is mostly useful for the cases where a record element is shared
+     * between various views, such as a one2many with a tree and a form view.
+     *
+     * @param {string} recordID a valid element ID
+     * @param {Object} attrs the new field attrs
+     * @return {Object} the previous field attrs
+     */
+    setFieldAttrs: function (recordID, attrs) {
+        var record = this.localData[recordID];
+        var result = record.fieldAttrs;
+        record.fieldAttrs = attrs;
+        return result;
+    },
     /**
      * For list resources, this changes the orderedBy key, then performs the
      * sort directly, in javascript.  This is used for sorting static datasets,
-- 
cgit v1.2.3