[FIX] composer: keep edition mode inactive on cursor selection change

Before this commit:
- Changing the composer cursor selection triggered a switch to edition mode,
  even when the composer was inactive.
- This caused unwanted activation of editing while simply moving the selection.

After this commit:
- Selection changes no longer modify the edition mode when the composer is
  inactive, ensuring the edition state remains stable.

closes #7548

Task: 5354541
X-original-commit: 790c2c5
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>

Author: rmbh-odoo

src/components/composer/composer/abstract_composer_store.ts

@@ -131,7 +131,7 @@ export abstract class AbstractComposerStore extends SpreadsheetStore {
     }
     this.selectionStart = start;
     this.selectionEnd = end;
-    this.editionMode = "editing";
+    this.stopComposerRangeSelection();
     this.computeFormulaCursorContext();
     this.computeParenthesisRelatedToCursor();
   }

tests/composer/composer_integration_component.test.ts

@@ -33,6 +33,7 @@ import {
   getElComputedStyle,
   gridMouseEvent,
   keyDown,
+  keyUp,
   rightClickCell,
   selectColumnByClicking,
   simulateClick,
@@ -361,6 +362,15 @@ describe("Composer interactions", () => {
     expect(composerStore.editionMode).toBe("inactive");
   });
 
+  test("should switch topbar composer from editing to inactive when pressing Escape on cell A1 containing '=A2'", async () => {
+    setCellContent(model, "A1", "=A2");
+    await click(fixture, ".o-spreadsheet-topbar .o-composer");
+    expect(composerStore.editionMode).toBe("editing");
+    keyDown({ key: "Escape" });
+    keyUp({ key: "Escape" });
+    expect(composerStore.editionMode).toBe("inactive");
+  });
+
   test("ArrowKeys will move to neighbour cell, if not in contentFocus mode (left/right)", async () => {
     let composerEl: Element;
     composerEl = await startComposition("a");

tests/composer/composer_store.test.ts

@@ -124,6 +124,12 @@ describe("edition", () => {
     expect(composerStore.editionMode).toBe("inactive");
   });
 
+  test("should keep edition mode inactive when selection changes while composer is inactive", () => {
+    expect(composerStore.editionMode).toBe("inactive");
+    composerStore.changeComposerCursorSelection(0, 0);
+    expect(composerStore.editionMode).toBe("inactive");
+  });
+
   test("should switch to editing mode when composer cursor selection changes", () => {
     composerStore.startEdition("=sum(");
     expect(composerStore.editionMode).toBe("selecting");