[FIX] table: create dynamic table on #SPILL! errors

Steps to reproduce:
- add an array formula
- add some content on the spill zone
- select the cell with #SPILL! error
- add a table on that cell

=> the created table is not dynamic

closes #7263

Task: 5102771
Signed-off-by: Adrien Minne (adrm) <adrm@odoo.com>

Author: LucasLefevre

src/plugins/ui_core_views/dynamic_tables.ts

@@ -9,6 +9,7 @@ import {
   zoneToXc,
 } from "../../helpers";
 import { createFilter } from "../../helpers/table_helpers";
+import { CellErrorType } from "../../types/errors";
 import {
   CellPosition,
   Command,
@@ -170,7 +171,10 @@ export class DynamicTablesPlugin extends CoreViewPlugin {
 
     const parentSpreadingCell = this.getters.getArrayFormulaSpreadingOn(topLeft);
     if (!parentSpreadingCell) {
-      return false;
+      const evaluatedCell = this.getters.getEvaluatedCell(topLeft);
+      return (
+        evaluatedCell.value === CellErrorType.SpilledBlocked && !evaluatedCell.errorOriginPosition
+      );
     } else if (deepEquals(parentSpreadingCell, topLeft) && getZoneArea(unionZone) === 1) {
       return true;
     }

tests/menus/menu_items_registry.test.ts

@@ -1674,6 +1674,29 @@ describe("Menu Item actions", () => {
         });
       });
 
+      test("Insert -> Table creates a dynamic table if it's called on a #SPILL! error", () => {
+        setCellContent(model, "A1", "=MUNIT(500)");
+        setSelection(model, ["A1"]);
+        expect(getEvaluatedCell(model, "A1")?.value).toBe("#SPILL!");
+        doAction(insertTablePath, env);
+        expect(model.getters.getCoreTable({ sheetId, row: 0, col: 0 })).toMatchObject({
+          range: { zone: toZone("A1") },
+          type: "dynamic",
+        });
+      });
+
+      test("Insert -> Table do not creates a dynamic table if it's called on cell referencing a #SPILL! error", () => {
+        setCellContent(model, "A1", "=A3");
+        setCellContent(model, "A3", "=MUNIT(500)");
+        expect(getEvaluatedCell(model, "A1")?.value).toBe("#SPILL!");
+        setSelection(model, ["A1"]);
+        doAction(insertTablePath, env);
+        expect(model.getters.getCoreTable({ sheetId, row: 0, col: 0 })).toMatchObject({
+          range: { zone: toZone("A1") },
+          type: "static",
+        });
+      });
+
       test("Edit -> Table (topbar)", () => {
         const spyOpenSidePanel = jest.spyOn(env, "openSidePanel");
         createTable(model, "A1:A5");