odoo/o-spreadsheet#4632
Created by BI, Dhrutik Patel (dhrp)
Merged
at a10569d003f97693d272d73d4f4d65bae5e31570
Statuses:
- label
- odoo:saas-17.3-fix-merge-cells-spill-zone-dhrp
- head
- 746d830816dababf12b4b675469827a8b0809dda
- merged
- 3 months ago by BI, Lucas Lefèvre (lul)
odoo/o-spreadsheet | |
---|---|
saas-17.4 | #4632 |
master | #4670 |
[FIX] Evaluator: merge cells in spill zone
Description:
PR##4205 introduced a #SPILL error when array formulas attempted to spread into merged cells. However, an incorrect logic in retrieving merged cells for specific range caused inconsistent behavior.
This PR corrects the logic to accurately retrieve merged cells within the specified range.
Task: : 4043638
review checklist
- [ ] feature is organized in plugin, or UI components
- [ ] support of duplicate sheet (deep copy)
- [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
- [ ] in model/UI: ranges are strings (to show the user)
- [ ] undo-able commands (uses this.history.update)
- [ ] multiuser-able commands (has inverse commands and transformations where needed)
- [ ] new/updated/removed commands are documented
- [ ] exportable in excel
- [ ] translations (_t("qmsdf %s", abc))
- [ ] unit tested
- [ ] clean commented code
- [ ] track breaking changes
- [ ] doc is rebuild (npm run doc)
- [ ] status is correct in Odoo