odoo/o-spreadsheet#3492
Created by fw-bot
Merged
at 6bf697601c63d4f2d3777c36d3e1c9d85da7018e
Statuses:
- label
- odoo:saas-15.2-15.0-fix-invalid-range-args-khpa-UVEK-fw
- head
- 2a2c07f962942cd952419108bd569a51564cf277
- merged
- 2 years ago by BI, Pierre Rousseau (pro)
| odoo/o-spreadsheet | |
|---|---|
| saas-15.2 | #3492 |
| saas-16.1 | #3515 |
| saas-16.2 | #3517 |
| saas-16.3 | #3520 |
| 15.0 | #3361 |
| saas-16.4 | #3523 |
| 16.0 | #3511 |
| 17.0 | #3524 |
| saas-17.1 | #3539 |
| 18.0 | |
| saas-18.2 | |
| saas-18.3 | |
| saas-18.4 | |
| 19.0 | |
| saas-19.1 | |
| saas-19.2 | |
| master | #3540 |
[FW][FIX] evaluation: behavior for invalid range arguments
Description:
When user provides an invalid sheet name in range arguments of a function, it performs computations on
data of specified range from active sheet by default and returns misleading results.
This PR changes the behavior to throw an error instead of performing any kind of computations.
Task: : 3619144
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
Forward-Port-Of: #3361