[FIX] pivot: skip cells on manipulated pivot array formulas

Steps to reproduce in odoo:

- insert a pivot with more rows than columns
- write in a cell =TRANSPOSE(ODOO.PIVOT.TABLE(1))
- right-click on the grand total value

=> boom

When computing which cell of the pivot table is clicked, we assume the matrix
comes directly from the ODOO.PIVOT.TABLE(...) function to compute the offsets
from the array formula. But it's completely wrong as the cell could at a
completely different place if the matrix is manipulated by other functions
before being outputted to the grid.

closes #5162

X-original-commit: 3605505
Task: 4292134
Signed-off-by: Lucas Lefèvre (lul) <[email protected]>
Signed-off-by: Rémi Rahir (rar) <[email protected]>

src/plugins/ui_core_views/pivot_ui.ts

@@ -209,6 +209,12 @@ export class PivotUIPlugin extends UIPlugin {
     if (!pivot.isValid()) {
       return EMPTY_PIVOT_CELL;
     }
+    if (
+      functionName === "PIVOT" &&
+      !cell.content.replaceAll(" ", "").toUpperCase().startsWith("=PIVOT")
+    ) {
+      return EMPTY_PIVOT_CELL;
+    }
     if (functionName === "PIVOT") {
       const includeTotal = args[2] === false ? false : undefined;
       const includeColumnHeaders = args[3] === false ? false : undefined;

tests/pivots/pivot_plugin.test.ts

@@ -165,6 +165,26 @@ describe("Pivot plugin", () => {
     expect(thirdCalculatedName).toBe("Calculated measure 1");
   });
 
+  test("getPivotCellFromPosition cannot get the pivot cell when the table is manipulated by other functions", () => {
+    // prettier-ignore
+    const grid = {
+      A1: "Customer", B1: "Price", C1: '=PIVOT.VALUE(1,"Price","5","Bob")',
+      A2: "Alice",    B2: "10",
+      A3: "Bob",      B3: "30",
+      A4: "=TRANSPOSE(PIVOT(1))",
+    };
+    const model = createModelFromGrid(grid);
+    addPivot(model, "A1:B3", {
+      columns: [],
+      rows: [{ fieldName: "Customer" }],
+      measures: [{ id: "price:sum", fieldName: "Price", aggregator: "sum" }],
+    });
+    selectCell(model, "C5");
+    expect(model.getters.getPivotCellFromPosition(model.getters.getActivePosition())).toEqual(
+      EMPTY_PIVOT_CELL
+    );
+  });
+
   test("forbidden characters are removed from new sheet name when duplicating a pivot", () => {
     const grid = {
       A1: "Customer",