odoo/o-spreadsheet#7267

Created by fw-bot
Merged at 13cf6886a7f2e6f07218b8be162f88fe0785d1cf

Statuses:

label
odoo:saas-18.3-saas-18.2-fix-data-chart-labels-adrm-466662-fw
head
cc55592298142cdfc1ee6631244229272b57c746
merged
2 days ago by BI, Rémi Rahir (rar)
odoo/o-spreadsheet
saas-18.2 #7227
saas-18.3 #7267
saas-18.4 #7268
19.0 #7270
master #7274

[FIX] chart: wrong label format for date chart

Description

Date charts would always have a datetime format for the labels, no matter the format of the data. They would also be truncated, which we probably don't want for dates.

The issue is that the date charts had a callback for the ticks using truncateLabel, and getLabelForValue which seems bugged in date charts[1].

This commit removes the tick callback for date charts, letting the luxon time adapter format the dates.

[1] chartjs/Chart.js#12128

Task: 4908471

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: #7227