odoo/o-spreadsheet#7426

Created by fw-bot
Merged at ff132df10bfab79e3b80013befb00ee748f60b6f

Statuses:

label
odoo:master-18.0-fix-scorecard-gauge-empty-range-dhrp-475803-fw
head
333259fbd4aa16b9d47e973828867df8deb6e1b9
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: #7417
Forward-Port-Of: #7385