Skip to content

[FIX] format: number format with no digit#8391

Closed
fw-bot wants to merge 1 commit into
saas-19.2from
saas-19.2-saas-19.1-fix-format-no-number-adrm-511943-fw
Closed

[FIX] format: number format with no digit#8391
fw-bot wants to merge 1 commit into
saas-19.2from
saas-19.2-saas-19.1-fix-format-no-number-adrm-511943-fw

Conversation

@fw-bot
Copy link
Copy Markdown
Collaborator

@fw-bot fw-bot commented Apr 13, 2026

Description

Since e8b38a a format with no date part and no digit is (correctly) detected as a number format. But the implementation of a format with no digit applied to a non-zero number was not correct. 123 formatted with [$str] was formatted as str123 instead of str.

Task: 6068824

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

Since e8b38a a format with no date part and no digit is (correctly)
detected as a number format. But the implementation of a format
with no digit applied to a non-zero number was not correct.
`123` formatted with `[$str]` was formatted as `str123` instead of `str`.

Task: 6068824
X-original-commit: c438eb3
@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Apr 13, 2026

Pull request status dashboard

@fw-bot
Copy link
Copy Markdown
Collaborator Author

fw-bot commented Apr 13, 2026

This PR targets saas-19.2 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

robodoo pushed a commit that referenced this pull request Apr 14, 2026
Since e8b38a a format with no date part and no digit is (correctly)
detected as a number format. But the implementation of a format
with no digit applied to a non-zero number was not correct.
`123` formatted with `[$str]` was formatted as `str123` instead of `str`.

closes #8391

Task: 6068824
X-original-commit: c438eb3
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Signed-off-by: Adrien Minne (adrm) <adrm@odoo.com>
@robodoo robodoo closed this Apr 14, 2026
@fw-bot fw-bot deleted the saas-19.2-saas-19.1-fix-format-no-number-adrm-511943-fw branch April 21, 2026 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants