[FIX] functions: cast to matrix

[LucasLefevre]

=SUMIF(CHOOSECOLS(A1,-1),A2) crashes because visitMatchingRanges blindly index args[0][0] even if args[0] is not a matrix. In this case, it's an error object because CHOOSECOLS(A1,-1) is not valid.

Task: 3918882

Description:

description of this task, what is implemented and why it is implemented that way.

Task: : TASK_ID

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]

Pull request status dashboard.

[laa-odoo]

@LucasLefevre are you okay to fixup the suggestion ?

[rrahir]

the ref should be done in master

[laa-odoo]

the ref should be done in master

Should be okay

[rrahir]

@robodoo r+

[robodoo]

@LucasLefevre @rrahir because this PR has multiple commits, I need to know how to merge it:

• merge to merge directly, using the PR as merge commit message
• rebase-merge to rebase and merge, using the PR as merge commit message
• rebase-ff to rebase and fast-forward

[rrahir]

@robodoo rebase-ff

[robodoo]

Merge method set to rebase and fast-forward.