[FIX] functions: ignore vectorized SUBTOTAL

SUBTOTAL should ignore SUBTOTAL values from a spilled formula

A1: "1"
A2: "1"
A3: "=SUBTOTAL(9, A1:A2) + A1:A2"
A5: "=SUBTOTAL(9, A4)"

=> A5 should be zero because the value in A4 comes from a SUBTOTAL
formula and it should be ignored

closes #7503

Task: 5261987
X-original-commit: c23e492
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>

Author: LucasLefevre

packages/o-spreadsheet-engine/src/functions/module_math.ts

@@ -1409,7 +1409,7 @@ export const SUBTOTAL = {
         if (!acceptHiddenCells && this.getters.isRowHiddenByUser(sheetId, row)) continue;
 
         for (let col = left; col <= right; col++) {
-          const cell = this.getters.getCell({ sheetId, col, row });
+          const cell = this.getters.getCorrespondingFormulaCell({ sheetId, col, row });
           if (!cell || !isSubtotalCell(cell)) {
             evaluatedCellToKeep.push(this.getters.getEvaluatedCell({ sheetId, col, row }));
           }

tests/functions/module_math.test.ts

@@ -3129,6 +3129,18 @@ describe("SUBTOTAL formula", () => {
     expect(gridResult.A7).toBe(4);
   });
 
+  test("SUBTOTAL ignores spilled SUBTOTAL", () => {
+    const grid = {
+      A1: "1",
+      A2: "1",
+      A3: "=SUBTOTAL(9, A1:A2) + A1:A2",
+      A5: "=SUBTOTAL(9, A4)",
+    };
+    const gridResult = evaluateGrid(grid);
+    expect(gridResult.A3).toBe(3);
+    expect(gridResult.A5).toBe(0);
+  });
+
   describe("ignoring filtered rows", () => {
     test("write the SUBTOTAL formula after updating filter", () => {
       const model = new Model();