Skip to content

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

Closed
hokolomopo wants to merge 1 commit into
saas-19.1from
saas-19.1-format-large-number-crash-adrm
Closed

[FIX] format: large number format with no digits#8198
hokolomopo wants to merge 1 commit into
saas-19.1from
saas-19.1-format-large-number-crash-adrm

Conversation

@hokolomopo
Copy link
Copy Markdown
Contributor

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

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Mar 25, 2026

Pull request status dashboard

@hokolomopo hokolomopo force-pushed the saas-19.1-format-large-number-crash-adrm branch from d36d330 to d04ffeb Compare April 1, 2026 07:44
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
@hokolomopo hokolomopo force-pushed the saas-19.1-format-large-number-crash-adrm branch from d04ffeb to c4fe04f Compare April 9, 2026 08:18
Copy link
Copy Markdown
Collaborator

@LucasLefevre LucasLefevre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

robodoo r+

robodoo pushed a commit that referenced this pull request Apr 13, 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 #8198

Task: 6010376
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
@robodoo robodoo closed this Apr 13, 2026
@fw-bot fw-bot deleted the saas-19.1-format-large-number-crash-adrm branch April 20, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants