Skip to content

[FIX] format: large number format with no digits#8397

Closed
fw-bot wants to merge 1 commit into
masterfrom
master-saas-19.1-format-large-number-crash-adrm-516770-fw
Closed

[FIX] format: large number format with no digits#8397
fw-bot wants to merge 1 commit into
masterfrom
master-saas-19.1-format-large-number-crash-adrm-516770-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 _createLargeNumberFormat would crash if there were no digits.

Task: 6010376

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

Since e8b38a a format with no date part and no digit is (correctly)
detected as a number format. But `_createLargeNumberFormat` would
crash if there were no digits.

Task: 6010376
X-original-commit: 269bec6
@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 `_createLargeNumberFormat` would
crash if there were no digits.

closes #8397

Task: 6010376
X-original-commit: 269bec6
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-format-large-number-crash-adrm-516770-fw branch April 21, 2026 06:48
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