odoo/odoo#198749
Created by fw-bot
Merged
at ef7e56d37fa2205b51779cab20c5c7c818cbcc12
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- label
- odoo-dev:master-16.0-prefetch-new-lines-rco-417556-fw
- head
- 329cc71c5a83754e83b310ff35ceaa66316c0aad
- merged
- 8 months ago by Framework (ORM), Raphael Collet
| odoo/odoo | |
|---|---|
| 16.0 | #198124 |
| 17.0 | #198202 |
| saas-17.4 | #198443 |
| 18.0 | #198448 |
| saas-18.1 | #198454 |
| saas-18.2 | #198542 |
| saas-18.3 | |
| saas-18.4 | |
| 19.0 | |
| master | #198749 |
[FW][FIX] core: prefetching of secondary new records
Consider a recordset of new records, and a loop like
for record in records:
for line in record.line_ids:
line.value
The implementation of record.line_ids does not actually prefetch all records. It actually fetches the field on the records' origin (their corresponding real records), but only assigns the current new record in cache. As the prefetching relies on the cached values of line_ids, the prefetching mechanism is actually broken on line.
The fix consists in assigning all the records to prefetch in this case. This does not add unexpected prefetching, since the origin records are prefetched as one batch anyway.