odoo/o-spreadsheet#3481
Created by fw-bot
Merged
at 8b1ab8b78d8ce334042fa64a0a047d8eaa51aee2
Statuses:
- label
- odoo:saas-17.1-16.0-fix-format-larg-numer-chaining-adrm-5seS-fw
- head
- 1e3091dca70a1a8025f65e5088ba1917e17d16ff
- merged
- 9 months ago by BI, Adrien Minne (adrm)
| odoo/o-spreadsheet | |
|---|---|
| 16.0 | #3358 |
| saas-16.1 | #3476 |
| saas-16.2 | #3477 |
| saas-16.3 | #3478 |
| saas-16.4 | #3479 |
| 17.0 | #3480 |
| saas-17.1 | #3481 |
| master | #3483 |
[FW][FIX] format: fix chaining of FORMAT.LARGE.NUMBER
Description
Chaining FORMAT.LARGE.NUMBER formulas was adding the postfix multiple times. This commit fixes the issue by adding a check to see if the postfix is already present.
Task: : 3633947
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: #3358