odoo/o-spreadsheet#7417
Created by fw-bot
Merged
at d9f2982fb440d5485af5df61ddfdc76e11dfd5d4
Statuses:
- label
- odoo:saas-18.2-18.0-fix-scorecard-gauge-empty-range-dhrp-473897-fw
- head
- 71bcb2627aa98158f27e2179a781b67e06b7dd36
- merged
- 2 months ago by BI, Dhrutik Patel (dhrp)
| odoo/o-spreadsheet | |
|---|---|
| 18.0 | #7385 |
| saas-18.2 | #7417 |
| saas-18.3 | #7423 |
| saas-18.4 | #7424 |
| 19.0 | #7425 |
| saas-19.1 | |
| master | #7426 |
[FIX] charts: crash when converting empty chart to scorecard/gauge
Description:
When creating a Scorecard or Gauge from an empty carousel/chart, context.range can be an empty array.
Accessing context.range[0].dataRange crashed.
This PR adds a check to ensure that context.range has at least one element before accessing context.range[0].dataRange.
Task: 5181741
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: #7385