Skip to content

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

Closed
fw-bot wants to merge 1 commit into
masterfrom
master-saas-19.1-fix-format-no-number-adrm-516757-fw
Closed

[FIX] format: number format with no digit#8395
fw-bot wants to merge 1 commit into
masterfrom
master-saas-19.1-fix-format-no-number-adrm-516757-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

@hokolomopo @LucasLefevre this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@robodoo r+

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

@hokolomopo
Copy link
Copy Markdown
Contributor

@robodoo r+

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 #8395

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 added the 19.3 label Apr 14, 2026
@robodoo robodoo closed this Apr 14, 2026
@fw-bot fw-bot deleted the master-saas-19.1-fix-format-no-number-adrm-516757-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