[IMP] Enable and adapt tour edit_link_popover

Author: msh-odoo

addons/html_editor/controllers/main.py

@@ -592,7 +592,12 @@ def link_preview_metadata_internal(self, preview_url):
             context = dict(request.env.context)
             words = preview_url.strip('/').split('/')
 
-            record_id = int(words.pop())
+            # If there is no recordID in url return empty dict.
+            try:
+                record_id = int(words.pop())
+            except ValueError:
+                return {}
+
             action_name = words.pop()
             if (action_name.startswith('m-') or '.' in action_name) and action_name in request.env and not request.env[action_name]._abstract:
                 # if path format is `odoo/<model>/<record_id>` so we use `action_name` as model name

addons/html_editor/tests/test_controller.py

@@ -192,8 +192,9 @@ def test_05_internal_link_preview(self):
             }),
             headers=self.headers
         )
+        response_not_record_json = response_not_record.json()
         self.assertEqual(200, response_not_record.status_code)
-        self.assertTrue('other_error_msg' in response_not_record.text)
+        self.assertEqual({}, response_not_record_json["result"])
 
         # Attempt to retrieve metadata for path format `odoo/<model>/<record_id>`
         response_model_record = self.url_open(

addons/website/static/src/js/tours/tour_utils.js

@@ -631,3 +631,30 @@ export function toggleMobilePreview(toggleOn) {
         },
     ];
 }
+
+/**
+ * Opens the link popup for the specified link element.
+ *
+ * @param {string} triggerSelector - Selector for the link element.
+ * @param {string} [linkName=""] - Name of the link.
+ * @returns {TourStep[]} The tour steps that opens the link popup.
+ */
+export function openLinkPopup(triggerSelector, linkName = "", childIndex = 0) {
+    return [
+        {
+            content: `Open '${linkName}' link popup`,
+            trigger: triggerSelector,
+            async run(actions) {
+                actions.click();
+                const el = this.anchor;
+                const sel = el.ownerDocument.getSelection();
+                sel.collapse(el.childNodes[childIndex], 1);
+                el.focus();
+            },
+        },
+        {
+            content: "Check if the link popover opened",
+            trigger: ".o-we-linkpopover",
+        },
+    ];
+}