summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/core/context.js
diff options
context:
space:
mode:
authorGéry Debongnie <ged@odoo.com>2017-04-21 22:43:50 +0200
committerGéry Debongnie <ged@odoo.com>2017-04-24 10:09:30 +0200
commit329c2ded24af600feaf6d013e6ad71ba41126665 (patch)
tree7a44b65e8e618fd5aa3bb474b5b6048b703b40af /addons/web/static/src/js/core/context.js
parent272e2cb0482b2250a6c54de1c8928d274c6a0b94 (diff)
[FIX] web: in x2manys, do not do onchange if invalid line
In general, with the new views, we execute immediately and recursively all onchanges, even from a x2many in a form view. It allows for nice interactivity, for example, recomputing immediately a total or a tax in a sale order when an order line changed. However, we are used to treat x2manys in a slightly weird way: for example, if you add a line in a one2many, then click elsewhere, most of the time, it is validated. But if the line is 'incomplete' (I mean, if a required field is not set), the line will not be validated, but be automatically discarded. If it was dirty, a confirm dialog will open. So, to keep in line with that semantic, we have decided that onchange will just not be done until the line is 'valid'. This will avoid trouble when someone add a line, it triggers some onchange which changes some values in the form, then click somewhere else, the line is discarded, but the changed values persist. Most of the time, it probably won't be a problem, because another onchange will be triggered, but it is some useless work anyway.
Diffstat (limited to 'addons/web/static/src/js/core/context.js')
0 files changed, 0 insertions, 0 deletions