diff options
| author | Khoi Nguyen <bng@odoo.com> | 2017-10-11 10:00:29 +0200 |
|---|---|---|
| committer | Khoi Nguyen <khoi@nguyen.me.uk> | 2017-10-16 11:34:39 +0200 |
| commit | 9f7af9f9b742d08ca69a4d4d0b963f625c4b4bba (patch) | |
| tree | c317468abba71a502c47bd271a1b2523bebae832 /addons/web/static/src/js/views/basic | |
| parent | bb692c76d41bb308ef24e729f907d707c44b99ca (diff) | |
[FIX] web: ticking a checkbox in an x2many after onchange
This commits fixes a traceback that appears when ticking a checkbox in
an x2many which triggers an onchange. The root cause of the issue is
twofold:
- confirmUpdate requires currentFieldIndex to be defined, but might be
needed when setting currentFieldIndex;
- confirmUpdate attempts to restore the selection range even if we are
not dealing with a char field (e.g. checkbox).
We solve the issue by ensuring that _selectCell gives a temporary value
to currentFieldIndex. This value is chosen so that confirmUpdate calls
_selectCell with the right parameters. Moreover, we only restore the
selection range when the range is defined.
Diffstat (limited to 'addons/web/static/src/js/views/basic')
0 files changed, 0 insertions, 0 deletions
