diff options
| author | Xavier Morel <xmo@odoo.com> | 2018-11-13 16:15:07 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2018-11-13 16:15:07 +0000 |
| commit | 79728da9d366d1402c07207aaaed45db58b37fc6 (patch) | |
| tree | 5ab991e82a0cd1a6492162dc9f7c5e48eb73f75f /addons/web/static/src | |
| parent | 1e4b3d093e194f24337c436a38e8588dd502e962 (diff) | |
[IMP] core: setup o2m edition views recursively in server-side form
Before this commit, the SSF would only set up edition views on o2ms at
the toplevel. This usually isn't an issue, but in some cases involing
onchanges on o2ms which themselves contain o2m, odd results could
happen as the second level would try to list the sub-o2m's fields in
order to cleanup its (possible absence of) values (cf the `subfields`
local variable in _cleanup_onchange), which would blow up due to the
missing `"edition"` key.
Fix by recursively processing o2ms and setting up their edition view.
Also fix the fields iteration in fvg processing: don't recurse inside
fields. There still is no support for embedded views (I think?) but we
should not try to treat subview fields as view fields, that could have
very bad results.
closes odoo/odoo#28645
Diffstat (limited to 'addons/web/static/src')
0 files changed, 0 insertions, 0 deletions
