diff options
| author | Géry Debongnie <ged@odoo.com> | 2017-07-06 09:34:25 +0200 |
|---|---|---|
| committer | Martin Geubelle <mge@odoo.com> | 2017-07-06 15:04:04 +0200 |
| commit | 56dae80994ceb27a80714bed78f4e17450151831 (patch) | |
| tree | 514a351b1b0b969ebc60b656947982afff0cd7e5 /addons/web/static/src/js/fields/relational_fields.js | |
| parent | 7dc731db8308febb6426ee4c6d61b2a80d40c021 (diff) | |
[FIX] web: prevent crash with invisible x2m and onchanges
When an invisible x2many field is modified by an onchange, we had a
crash because the reset function did not return a deferred in that case.
This happens because the signature of _render in AbstractField allow for
an undefined return value, but the reset method should return a
deferred.
This was an issue for example in stock: editing the
pack_operation_product_ids one2many field in a stock.picking could
trigger an onchange on another invisible one2many (with no inline views).
Diffstat (limited to 'addons/web/static/src/js/fields/relational_fields.js')
| -rw-r--r-- | addons/web/static/src/js/fields/relational_fields.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/web/static/src/js/fields/relational_fields.js b/addons/web/static/src/js/fields/relational_fields.js index df7f018369d..4ddc674255a 100644 --- a/addons/web/static/src/js/fields/relational_fields.js +++ b/addons/web/static/src/js/fields/relational_fields.js @@ -684,7 +684,7 @@ var FieldX2Many = AbstractField.extend({ * * @override * @private - * @returns {Deferred} + * @returns {Deferred|undefined} */ _render: function () { if (!this.view) { |