[FIX] Tables: invalidate computed style on DELETE_CONTENT

The command `DELETE_CONTENT` can sometimes dispatch a subcommand that
deletes a table, which implies that the table style and therefore the
cell computed style need to be invalidated. Since the subcommand does
not reach Ui feature plugins when replaying commands, we need to
explicitely invalidate the table & cell styles on `DELETE_CONTENT`.

closes #8839

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

Author: rrahir

src/plugins/ui_feature/table_computed_style.ts

@@ -276,6 +276,7 @@ const invalidateTableStyleCommands = [
   "RESIZE_TABLE",
   "CREATE_TABLE_STYLE",
   "REMOVE_TABLE_STYLE",
+  "DELETE_CONTENT",
 ] as const;
 const invalidateTableStyleCommandsSet = new Set<CommandTypes>(invalidateTableStyleCommands);
 

tests/collaborative/collaborative.test.ts

@@ -70,7 +70,7 @@ import {
   getMerges,
   getStyle,
 } from "../test_helpers/getters_helpers";
-import { addToRegistry, getDataValidationRules } from "../test_helpers/helpers";
+import { addToRegistry, getDataValidationRules, toCellPosition } from "../test_helpers/helpers";
 import { addPivot, updatePivot } from "../test_helpers/pivot_helpers";
 import { setupCollaborativeEnv } from "./collaborative_helpers";
 
@@ -922,6 +922,22 @@ describe("Multi users synchronisation", () => {
     );
   });
 
+  test("deleting a cell content properly deletes the table and invalidates its computed style", () => {
+    const sheetId = alice.getters.getActiveSheetId();
+    setCellContent(alice, "A1", "hello");
+    createTable(alice, "A1:A2");
+    const A1 = toCellPosition(sheetId, "A1");
+    expect([alice, bob, charlie]).toHaveSynchronizedValue(
+      (user) => user.getters.getCellComputedStyle(A1),
+      { bold: true, fillColor: "#346B90", textColor: "#FFFFFF" }
+    );
+    deleteContent(alice, ["A1:A2"]);
+    expect([alice, bob, charlie]).toHaveSynchronizedValue(
+      (user) => user.getters.getCellComputedStyle(A1),
+      {}
+    );
+  });
+
   test.each(["COL", "ROW"] as const)("Can group headers concurrently", (dimension) => {
     const sheetId = alice.getters.getActiveSheetId();