diff options
| author | Géry Debongnie <ged@odoo.com> | 2017-04-21 22:43:50 +0200 |
|---|---|---|
| committer | Géry Debongnie <ged@odoo.com> | 2017-04-24 10:09:30 +0200 |
| commit | 329c2ded24af600feaf6d013e6ad71ba41126665 (patch) | |
| tree | 7a44b65e8e618fd5aa3bb474b5b6048b703b40af /addons/web/static/src/js/core | |
| parent | 272e2cb0482b2250a6c54de1c8928d274c6a0b94 (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')
0 files changed, 0 insertions, 0 deletions
