pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] web: BasicModel: always reload x2many fields

Browse Source 
Before this commit, the relational data of a record was only
fetched once, and then retrieved from the cache.

This was premature optimization, that can lead to bugs as those
relational data displayed in the webclient might be desynchronized
with the actual data in DB.
This commit is contained in:
Aaron Bohy
2017-04-11 19:44:38 +02:00
parent ce36786038
commit 1e9b695ef1
2 changed files with 31 additions and 46 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/web/static/src/js/views/basic/basic_model.js
+18 -31
View File
@@ -1300,7 +1300,6 @@ var BasicModel = AbstractModel.extend({
_fetchRecord: function (record, fieldNames) {
var self = this;
fieldNames = fieldNames || _.uniq(record.fieldNames.concat(['display_name']));
var oldData = record.data;
return this._rpc(record.model, 'read')
.args([[record.res_id], fieldNames])
// .withContext({ bin_size: true }) // FIXME: when editing a subrecord in the partner form view, it tries to write the bin_size on the image field
@@ -1313,7 +1312,7 @@ var BasicModel = AbstractModel.extend({
self._parseServerData(fieldNames, record.fields, record.data);
})
.then(function () {
return self._fetchX2Manys(record, oldData, fieldNames).then(function () {
return self._fetchX2Manys(record, fieldNames).then(function () {
return self._postprocess(record);
});
});
@@ -1591,44 +1590,32 @@ var BasicModel = AbstractModel.extend({
* useless rpcs.
*
* @param {Object} record local resource
* @param {Object} oldData the data fetched previously
* @param {string[]} [fieldNames] list of fields to fetch
* @returns {Deferred}
*/
_fetchX2Manys: function (record, oldData, fieldNames) {
_fetchX2Manys: function (record, fieldNames) {
var self = this;
var defs = [];
fieldNames = fieldNames || record.fieldNames;
_.each(fieldNames, function (fieldName) {
var field = record.fields[fieldName];
if (field.type === 'one2many' || field.type === 'many2many') {
var list;
// a previously loaded list exists
if (oldData[fieldName]) {
list = self.localData[oldData[fieldName]];
list.res_ids = record.data[fieldName];
list.count = list.res_ids.length;
list.offset = list.offset < list.count ? list.offset : 0;
} else {
// need to create a list from scratch
var attrs = record.fieldAttrs[fieldName];
var rawContext = attrs && attrs.context;
var ids = record.data[fieldName] || [];
list = self._makeDataPoint({
count: ids.length,
fieldAttrs: field.fieldAttrs,
fields: field.relatedFields,
limit: field.limit,
modelName: field.relation,
res_ids: ids,
static: true,
type: 'list',
parentID: record.id,
rawContext: rawContext,
relationField: field.relation_field,
});
}
var attrs = record.fieldAttrs[fieldName];
var rawContext = attrs && attrs.context;
var ids = record.data[fieldName] || [];
var list = self._makeDataPoint({
count: ids.length,
fieldAttrs: field.fieldAttrs,
fields: field.relatedFields,
limit: field.limit,
modelName: field.relation,
res_ids: ids,
static: true,
type: 'list',
parentID: record.id,
rawContext: rawContext,
relationField: field.relation_field,
});
record.data[fieldName] = list.id;
if (!field.__no_fetch) {
var def = self._readUngroupedList(list).then(function () {
addons/web/static/test/fields/relational_fields_tests.js
+13 -15
View File
@@ -711,33 +711,31 @@ QUnit.module('relational_fields', {
'one2many kanban should contain 3 cards for record 2');
// move back to record 1, which should contain again its first 40 related
// records, but that is already in cache so only 1 RPC (the read for the
// main record) should have been done
// records
form.pager.previous();
assert.strictEqual(count, 5, 'one RPC should have been done');
assert.strictEqual(count, 6, 'two RPCs should have been done');
assert.strictEqual(form.$('.o_kanban_record:not(".o_kanban_ghost")').length, 40,
'one2many kanban should contain 40 cards for record 1');
// move to the second page of the o2m: 1 RPC should have been done to fetch
// the 2 subrecords of page 2, and those records should now be displayed
form.$('.o_x2m_control_panel .o_pager_next').click();
assert.strictEqual(count, 6, 'one RPC should have been done');
assert.strictEqual(count, 7, 'one RPC should have been done');
assert.strictEqual(form.$('.o_kanban_record:not(".o_kanban_ghost")').length, 2,
'one2many kanban should contain 2 cards for record 1 at page 2');
// move to record 2 again and check that everything is correctly updated
// (only one RPC should be done, to read the record 2)
form.pager.next();
assert.strictEqual(count, 7, 'one RPC should have been done');
assert.strictEqual(count, 9, 'two RPCs should have been done');
assert.strictEqual(form.$('.o_kanban_record:not(".o_kanban_ghost")').length, 3,
'one2many kanban should contain 3 cards for record 2');
// move back to record 1 and move to page 2 again: all o2m records should
// be in cache
// move back to record 1 and move to page 2 again: all data should have
// been correctly reloaded
form.pager.previous();
assert.strictEqual(count, 8, 'one RPC should have been done'); // the read of record 1
assert.strictEqual(count, 11, 'two RPCs should have been done');
form.$('.o_x2m_control_panel .o_pager_next').click();
assert.strictEqual(count, 8, 'no more RPC should have been done');
assert.strictEqual(count, 12, 'one RPC should have been done');
assert.strictEqual(form.$('.o_kanban_record:not(".o_kanban_ghost")').length, 2,
'one2many kanban should contain 2 cards for record 1 at page 2');
form.destroy();
@@ -792,7 +790,7 @@ QUnit.module('relational_fields', {
});
QUnit.test('one2many list field edition', function (assert) {
assert.expect(7);
assert.expect(6);
this.data.partner.records.push({
id: 3,
@@ -844,10 +842,10 @@ QUnit.module('relational_fields', {
form.$('.o_field_one2many tbody td').first().find('input').val("new value").trigger('input');
form.$el.click();
form.$buttons.find('.o_form_button_save').click();
// FIXME: this next test doesn't actually test that the save works, because the relational
// data isn't reloaded when clicking on save (it doesn't work for now actually)
assert.strictEqual(form.$('.o_field_one2many tbody td').first().text(), 'new value',
"display name of first record in o2m list should be 'new value'");
// FIXME: this next test doesn't pass as the save of updates of
// relational data is temporarily disabled
// assert.strictEqual(form.$('.o_field_one2many tbody td').first().text(), 'new value',
// "display name of first record in o2m list should be 'new value'");
form.destroy();
});