[FIX] web: webclient, action_adapters: change tab title whenever a push state occurs

From a list view with multiple records, go on a record of that list
Click on the pager next button

Before this commit, the browser tab's title was not updated to the second records.

After this commit, it is.

closes odoo/odoo#80346

X-original-commit: 3d27b739a3e8243d50e9764000727b31727dc2d2
Signed-off-by: Lucas Perais (lpe) <lpe@odoo.com>

addons/web/static/src/legacy/action_adapters.js

@@ -28,6 +28,7 @@ class ActionAdapter extends ComponentAdapter {
         // In Wowl, we want to have all states pushed during the same setTimeout.
         // This is protected in legacy (backward compatibility) but should not e supported in Wowl
         this.tempQuery = {};
+        this.waitTitle = true;
         let originalUpdateControlPanel;
         useEffect(
             () => {
@@ -38,6 +39,7 @@ class ActionAdapter extends ComponentAdapter {
                 if (!this.wowlEnv.inDialog) {
                     this.pushState(query);
                     this.title.setParts({ action: this.widget.getTitle() });
+                    this.waitTitle = false;
                 }
                 this.wowlEnv.bus.on("ACTION_MANAGER:UPDATE", this, () => {
                     this.env.bus.trigger("close_dialogs");
@@ -76,6 +78,12 @@ class ActionAdapter extends ComponentAdapter {
             Object.assign(this.tempQuery, query);
             return;
         }
+        if (!this.waitTitle && this.widget) {
+            const actionTitle = this.widget.getTitle();
+            if (actionTitle) {
+                this.title.setParts({ action: actionTitle });
+            }
+        }
         this.router.pushState(query);
     }
 

addons/web/static/tests/webclient/actions/window_action_tests.js

@@ -2403,4 +2403,18 @@ QUnit.module("ActionManager", (hooks) => {
 
         assert.verifySteps(["/web/dataset/search_read"]);
     });
+
+    QUnit.test("pushState also changes the title of the tab", async (assert) => {
+        assert.expect(3);
+
+        const webClient = await createWebClient({ serverData });
+        await doAction(webClient, 3); // list view
+        const titleService = webClient.env.services.title;
+        assert.strictEqual(titleService.current, "{\"zopenerp\":\"Odoo\",\"action\":\"Partners\"}");
+        await click(webClient.el.querySelector(".o_data_row"));
+        await legacyExtraNextTick();
+        assert.strictEqual(titleService.current, "{\"zopenerp\":\"Odoo\",\"action\":\"First record\"}");
+        await click(webClient.el.querySelector(".o_pager_next"));
+        assert.strictEqual(titleService.current, "{\"zopenerp\":\"Odoo\",\"action\":\"Second record\"}");
+    });
 });