[FW][FIX] evaluation: Ensure dependency invalidation for array formulas

[fw-bot]

How to reproduce: -> see the attached test

What is happening? During the evaluation of SUMIFS, it will fetch its ranges. First it will first fetch an empty range (E4:E7) since MUNIT hasn't been evaluated yet. Afterwards, it will fetch H4, which in turn will request C4 and therefore force the evaluation of the spreaded function MUNIT.

AT this point, there is a disparity of values because the values of the range (E4:E7) have changed. To address those situations, the evaluator invalidates the dependencies of all the spreaded zone. Which means marking A1 to be recomputed.

Unfortunately, A1 is still being evaluated and since #4589 (specifically commit 8d22e86) we remove the cell from the queue after we evaluted it.

This revision changes the approach taken in 8d22e86 to allow legitimate invalidations take place while avoiding useless reevaluations.

Task: 4252800

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

Forward-Port-Of: odoo/o-spreadsheet#5211 Forward-Port-Of: odoo/o-spreadsheet#5209

[robodoo]

[fw-bot]

@rrahir @LucasLefevre cherrypicking of pull request odoo/o-spreadsheet#5209 failed.

• b2aa847b681d2c840b93190f36ce6fdbbb5c424d <- on this commit
• fa4517d0eb5147ec11fa794a8af54ae05add8475

stdout:

Auto-merging src/functions/index.ts
CONFLICT (content): Merge conflict in src/functions/index.ts
Auto-merging src/plugins/ui_core_views/cell_evaluation/evaluator.ts
Auto-merging tests/evaluation/evaluation_formula_array.test.ts
CONFLICT (content): Merge conflict in tests/evaluation/evaluation_formula_array.test.ts

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

:warning: after resolving this conflict, you will need to merge it via @robodoo.

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

[rrahir]

@robodoo r+