[FIX] web: fetching special data in field widgets

Some field widgets need additional data to render themself. These data
should ideally be fetched in the data fetching phase. In 10.0, these
data were fetched during field rendering which implied that the data
fetching was done sequentially. To limit this effect, before this
commit, these special fields were flagged by the basic view according
to some field properties and the basic model flagged the fields with
the required data. Now, the system has been improved. The field widget
classes can define a key "specialData" which refers to a BasicModel
method. These methods will be used to fetch the required data at the
right time and these data will be available at
`this.record.specialData[this.name]` in the AbstractField instance.

The new system also fixes a bug: the required data was fetched only
at record loading but should in fact be re-fetched at each view
change.

1 files changed, 8 insertions, 4 deletions

diff --git a/addons/web/static/src/js/fields/relational_fields.js b/addons/web/static/src/js/fields/relational_fields.js
index 4d3aaf14e7d..68bd7a52343 100644
--- a/addons/web/static/src/js/fields/relational_fields.js
+++ b/addons/web/static/src/js/fields/relational_fields.js
@@ -1107,10 +1107,11 @@ var FieldMany2ManyCheckBoxes = AbstractField.extend({
     events: _.extend({}, AbstractField.prototype.events, {
         change: '_onChange',
     }),
+    specialData: "_fetchSpecialRelation",
     supportedFieldTypes: ['many2many'],
     init: function () {
         this._super.apply(this, arguments);
-        this.m2mValues = this.field.__many2many_information;
+        this.m2mValues = this.record.specialData[this.name];
     },
 
     //--------------------------------------------------------------------------
@@ -1170,6 +1171,7 @@ var FieldStatus = AbstractField.extend({
     events: {
         'click .o_arrow_button': '_onClickStage',
     },
+    specialData: "_fetchSpecialStatus",
     supportedFieldTypes: ['selection', 'many2one'],
     init: function () {
         this._super.apply(this, arguments);
@@ -1192,7 +1194,7 @@ var FieldStatus = AbstractField.extend({
      */
     _setState: function () {
         if (this.field.type === 'many2one') {
-            this.status_information = this.field.__status_information;
+            this.status_information = this.record.specialData[this.name];
             var status = _.findWhere(this.status_information, {id: this.value.res_id});
             if (status) {
                 status.selected = true;
@@ -1260,6 +1262,7 @@ var FieldStatus = AbstractField.extend({
  */
 var FieldSelection = AbstractField.extend({
     template: 'FieldSelection',
+    specialData: "_fetchSpecialRelation",
     supportedFieldTypes: ['selection', 'many2one'],
     events: _.extend({}, AbstractField.prototype.events, {
         'change': '_onChange',
@@ -1272,7 +1275,7 @@ var FieldSelection = AbstractField.extend({
         this._super.apply(this, arguments);
         this.values = [];
         if (this.field.type === 'many2one') {
-            this.values = this.field.__selection_information;
+            this.values = this.record.specialData[this.name];
         } else {
             this.values = _.reject(this.field.selection, function (v) {
                 return v[0] === false && v[1] === '';
@@ -1357,6 +1360,7 @@ var FieldSelection = AbstractField.extend({
 
 var FieldRadio = FieldSelection.extend({
     template: 'FieldRadio',
+    specialData: "_fetchSpecialMany2ones",
     supportedFieldTypes: ['selection', 'many2one'],
     events: _.extend({}, AbstractField.prototype.events, {
         'click input': '_onInputClick',
@@ -1369,7 +1373,7 @@ var FieldRadio = FieldSelection.extend({
         if (this.field.type === 'selection') {
             this.values = this.field.selection || [];
         } else if (this.field.type === 'many2one') {
-            this.values = _.map(this.field.__status_information, function (val) {
+            this.values = _.map(this.record.specialData[this.name], function (val) {
                 return [val.id, val.display_name];
             });
         }