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 04a10b46442..508ece9af0d 100644 --- a/addons/web/static/src/js/views/basic/basic_model.js +++ b/addons/web/static/src/js/views/basic/basic_model.js @@ -3208,6 +3208,7 @@ var BasicModel = AbstractModel.extend({ _.each(element._changes, function (command) { if (command.operation === 'DELETE' || command.operation === 'FORGET' || + (command.operation === 'ADD' && !command.isNew)|| command.operation === 'REMOVE_ALL') { return; } diff --git a/addons/web/static/tests/fields/relational_fields_tests.js b/addons/web/static/tests/fields/relational_fields_tests.js index facc8132d7b..87b48526a2f 100644 --- a/addons/web/static/tests/fields/relational_fields_tests.js +++ b/addons/web/static/tests/fields/relational_fields_tests.js @@ -6207,6 +6207,51 @@ QUnit.module('relational_fields', { form.destroy(); }); + QUnit.test('one2many, onchange, edition and multipage...', function (assert) { + assert.expect(7); + + this.data.partner.onchanges = { + turtles: function (obj) { + obj.turtles = [[5]].concat(obj.turtles); + } + }; + + this.data.partner.records[0].turtles = [1,2,3]; + + var form = createView({ + View: FormView, + model: 'partner', + data: this.data, + arch:'
' + + '' + + '' + + '' + + '' + + '' + + '
', + res_id: 1, + mockRPC: function (route, args) { + assert.step(args.method + ' ' + args.model) + return this._super(route, args); + }, + viewOptions: { + mode: 'edit', + }, + }); + form.$('.o_field_x2many_list_row_add a').click(); + form.$('.o_field_x2many_list_row_add a').click(); + + assert.verifySteps([ + 'read partner', + 'read turtle', + 'default_get turtle', + 'onchange partner', + 'default_get turtle', + 'onchange partner', + ]); + form.destroy(); + }); + QUnit.test('one2many with CREATE onchanges correctly refreshed', function (assert) { assert.expect(5);