odoo/o-spreadsheet#6436

Created by fw-bot
Merged at 9600da59c3f86c9f46b70b032d2555e5cfabc7c6

Statuses:

label
odoo:18.0-saas-17.4-pivot-month-granularity-sorting-adrm-434810-fw
head
df82e7a53bdec519a38044eececb49ea67f3d145
merged
1 week ago by BI, Rémi Rahir (rar)
odoo/o-spreadsheet
saas-17.4 #6335
18.0 #6436
saas-18.1 #6486
saas-18.2 #6487
saas-18.3 #6488
master #6489

[FW][FIX] pivot: `month` granularity sorting

Description

The month granularity sorting wasn't working. The issue was that we were using Number(value) instead of toNumber(value) to convert the field value to a number.

Task: 4781810

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