[FIX] cell_menu_registry: link cell displays 'Edit Link' in context menu

Previously, if a link was inserted into a cell and right-clicked, the
context menu incorrectly displayed 'Insert Link' instead of 'Edit Link'.

This commit addresses the problem by verifying whether the cell is a
link cell and adjusts the menu item name accordingly.

Task ID: 3698437

closes #3661

X-original-commit: 7711581
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Signed-off-by: Dhrutik Patel (dhrp) <dhrp@odoo.com>

src/actions/menu_items_actions.ts

@@ -439,6 +439,14 @@ export const INSERT_LINK = (env: SpreadsheetChildEnv) => {
   env.model.dispatch("OPEN_CELL_POPOVER", { col, row, popoverType: "LinkEditor" });
 };
 
+export const INSERT_LINK_NAME = (env: SpreadsheetChildEnv) => {
+  const sheetId = env.model.getters.getActiveSheetId();
+  const { col, row } = env.model.getters.getActivePosition();
+  const cell = env.model.getters.getEvaluatedCell({ sheetId, col, row });
+
+  return cell && cell.link ? _lt("Edit link") : _lt("Insert link");
+};
+
 //------------------------------------------------------------------------------
 // Filters action
 //------------------------------------------------------------------------------

src/registries/menus/cell_menu_registry.ts

@@ -3,6 +3,7 @@ import { MenuItemRegistry } from "../menu_items_registry";
 
 import * as ACTION_EDIT from "../../actions/edit_actions";
 import * as ACTION_INSERT from "../../actions/insert_actions";
+import { INSERT_LINK_NAME } from "../../actions/menu_items_actions";
 
 //------------------------------------------------------------------------------
 // Context Menu Registry
@@ -89,7 +90,7 @@ cellMenuRegistry
   })
   .add("insert_link", {
     ...ACTION_INSERT.insertLink,
-    name: _lt("Insert link"),
+    name: INSERT_LINK_NAME,
     sequence: 150,
     separator: true,
   });

tests/components/link/link_display.test.ts

@@ -85,6 +85,20 @@ describe("link display component", () => {
     expect(fixture.querySelector(".o-link-tool")).toBeFalsy();
   });
 
+  test("right-click on a linked cell should show 'Edit Link' instead of 'Insert Link' in the context menu", async () => {
+    setCellContent(model, "A1", "HELLO");
+    await rightClickCell(model, "A1");
+    expect(
+      fixture.querySelector(".o-menu .o-menu-item[data-name='insert_link']")?.textContent
+    ).toBe("Insert link");
+
+    setCellContent(model, "A1", "[label](url.com)");
+    await rightClickCell(model, "A1");
+    expect(
+      fixture.querySelector(".o-menu .o-menu-item[data-name='insert_link']")?.textContent
+    ).toBe("Edit link");
+  });
+
   test("component is closed when cell is deleted", async () => {
     setCellContent(model, "A1", "[label](url.com)");
     await hoverCell(model, "A1", 400);