diff options
| author | Géry Debongnie <ged@odoo.com> | 2017-04-01 14:38:32 +0200 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2017-04-11 19:44:38 +0200 |
| commit | 0f96189b7fbc481b21f0b7b7a2fdcb82916d6bae (patch) | |
| tree | dcab5ac6540bf3e0ba0cb77853f496333e3a9084 /addons/web/static/src/js/fields/relational_fields.js | |
| parent | cef1b273c06ba277c6d47a538c203872c4594898 (diff) | |
[FIX] web: update all fields in onchange in one2many
Before this commit, we did not properly update all fields in a one2many.
I assume that it was because the list renderer crashed if it tried to
display a field that did not exist in the list view, but in that case,
we simply can ignore these fields.
Diffstat (limited to 'addons/web/static/src/js/fields/relational_fields.js')
| -rw-r--r-- | addons/web/static/src/js/fields/relational_fields.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/addons/web/static/src/js/fields/relational_fields.js b/addons/web/static/src/js/fields/relational_fields.js index e4ea88db789..c29f958cc06 100644 --- a/addons/web/static/src/js/fields/relational_fields.js +++ b/addons/web/static/src/js/fields/relational_fields.js @@ -746,9 +746,9 @@ var FieldOne2Many = FieldX2Many.extend({ if (event && event.target === this && self.view.arch.tag === 'tree' && this.editable) { var command = event.data.changes[this.name]; if (command.operation === 'UPDATE') { - var fieldsChanged = _.keys(command.data); - // FIXME: other fields might have change with onchange - this.renderer.confirmChange(record.data[this.name], command.id, fieldsChanged); + var state = record.data[this.name]; + var fieldNames = state.getFieldNames(); + this.renderer.confirmChange(state, command.id, fieldNames); return $.when(); } } |
