Skip to content

[FIX] formulas: add IFERROR second argument when exporting data#8768

Open
fw-bot wants to merge 1 commit into
masterfrom
master-17.0-formula_fix_iferror_arguments-anhe-528493-fw
Open

[FIX] formulas: add IFERROR second argument when exporting data#8768
fw-bot wants to merge 1 commit into
masterfrom
master-17.0-formula_fix_iferror_arguments-anhe-528493-fw

Conversation

@fw-bot
Copy link
Copy Markdown
Collaborator

@fw-bot fw-bot commented May 20, 2026

Description

In our implementation of the IFERROR formula, the second argument (value if error) is optionnal, defaulting to an empty string, which results to a 0 value in the cell when evaluated. In the excel documentation, the second argument of IFERROR is required leading to issue when exporting from spreadsheet a formula with IFERROR and then importing it to excel or Gsheet. This PR fixes this issue by adding a second argument whenn needed in the exporting process.

Related Task

Task: 5993405

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

Description

In our implementation of the IFERROR formula, the second argument
(value if error) is optionnal, defaulting to an empty string, which
results to a 0 value in the cell when evaluated. In the excel
documentation, the second argument of IFERROR is required leading
to issue when exporting from spreadsheet a formula with IFERROR and
then importing it to excel or Gsheet. This PR fixes this issue by
adding a second argument whenn needed in the exporting process.

Related Task

Task: 5993405
X-original-commit: 57e3404
@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented May 20, 2026

Pull request status dashboard

@rrahir
Copy link
Copy Markdown
Collaborator

rrahir commented May 20, 2026

@anhe-odoo careful, don't r+ it before owl3 is merged

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.

4 participants