diff options
| author | Aaron Bohy <aab@odoo.com> | 2018-02-23 13:48:20 +0100 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2018-02-26 08:14:08 +0100 |
| commit | a16013be6636db58a4252fafd67033c6643a1e48 (patch) | |
| tree | fd6450966b2ca13fee73d202408a0baf6478e3a0 /addons/mail/static/src/xml/thread.xml | |
| parent | 5b769650d6416be123e972aa8f4b2b2c25ce438f (diff) | |
[FIX] web: multi pages ordered o2m with onchange
Let's assume a one2many displayed over several pages with a default
order such that a record, say R, that would normally be on page 1
is actually on another page. Moreover, there is an onchange on that
one2many which converts all commands 4 (LINK_TO) into commands 1
(UPDATE), which is standard in the ORM.
When the first line of the one2many was edited, the record R was
fully loaded, whereas it shouldn't have been (as it is not on the
current page). So basically, a useless read was performed.
Moreover, if there was an x2many field in the one2many list, a
crash occurred because relational data of R that shouldn't have
been read wasn't postprocessed by the BasicModel.
The useless read occurred because the one2many subrecords weren't
correctly sorted before checking if some of them/some of their
fields should be read. This is not the case anymore.
OPW-1816800
Diffstat (limited to 'addons/mail/static/src/xml/thread.xml')
0 files changed, 0 insertions, 0 deletions
