[FIX] evaluation: evaluate new dependencies

Commit 43f4fdd is wrong, the dependencies should be evaluated,
not the position.

closes #4183

Task: 0
Signed-off-by: Pierre Rousseau (pro) <pro@odoo.com>

src/plugins/ui_core_views/cell_evaluation/evaluator.ts

@@ -91,7 +91,15 @@ export class Evaluator {
 
   private addDependencies(position: CellPosition, dependencies: Range[]) {
     this.formulaDependencies().addDependencies(position, dependencies);
-    this.computeAndSave(position);
+    for (const range of dependencies) {
+      const sheetId = range.sheetId;
+      const { left, bottom, right, top } = range.zone;
+      for (let col = left; col <= right; col++) {
+        for (let row = top; row <= bottom; row++) {
+          this.computeAndSave({ sheetId, col, row });
+        }
+      }
+    }
   }
 
   private updateCompilationParameters() {

tests/functions/module_lookup.test.ts

@@ -1567,6 +1567,23 @@ describe("INDIRECT formula", () => {
     expect(grid.A4).toBe(6);
   });
 
+  test("Dependencies are correctly evaluated", () => {
+    const model = new Model({
+      sheets: [
+        {
+          cells: {
+            A1: { content: '=INDIRECT("B1")' },
+            B1: { content: "hello" },
+            A2: { content: '=INDIRECT("B2")' },
+            B2: { content: "=1+1" },
+          },
+        },
+      ],
+    });
+    expect(getCellContent(model, "A1")).toBe("hello");
+    expect(getCellContent(model, "A2")).toBe("2");
+  });
+
   test("Reference to a cell and range of a different sheet (A1 notation)", () => {
     const model = new Model();
     const sheetId = model.getters.getActiveSheetId();