odoo/o-spreadsheet#7046
Created by fw-bot
Merged
at cb952fde22364118c3e2ee5ed89c1239e49a9b1e
Statuses:
- label
- odoo:saas-18.3-17.0-evaluation-prevent_operation_on_empty_matrixes-anhe-460418-fw
- head
- 18ba38c67a158a89eb2e217a38e9d093769e58d6
- merged
- 5 days ago by BI, Hendrickx Anthony (anhe)
| odoo/o-spreadsheet | |
|---|---|
| 17.0 | #6930 |
| 18.0 | #7034 |
| saas-18.1 | #7042 |
| saas-18.2 | #7045 |
| saas-18.3 | #7046 |
| saas-18.4 | #7048 |
| master | #7057 |
[FIX] evaluation: fix operation with empty matrices
Task Description
When using empty matrices for math operation (multiplication or inversion), as in the FORECAST formula, we got a traceback and the error message was not clear for the user. This commits aims to fix it by checking that the matrices used in theses operations are not empty and returning an understandable error if it's not the case.
Related Task
- Task: 5001658
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