fix(pat-inject): Fix error when no title is found.

thet · thet · commit 37183618a9a8 · 2025-02-03T16:32:13.000+01:00
diff --git a/src/pat/inject/inject.js b/src/pat/inject/inject.js
@@ -504,7 +504,7 @@ const inject = {
         }
         history.pushState({ url: url }, "", url);
         // Also inject title element if we have one
-        if ($title.length) {
+        if ($title?.length) {
             const title_el = document.querySelector("title");
             if (title_el) {
                 this._inject(trigger, $title, title_el, {
diff --git a/src/pat/inject/inject.test.js b/src/pat/inject/inject.test.js
@@ -1739,6 +1739,84 @@ describe("pat-inject", function () {
                 expect(title).toBeTruthy();
                 expect(title.textContent.trim()).toBe("test"); // Old title
             });
+
+            it("9.4.3 - Does not break, if no title is found in source", async function () {
+                document.head.innerHTML = `
+                    <title>test</title>
+                `;
+                document.body.innerHTML = `
+                    <a class="pat-inject"
+                       href="test.html"
+                       data-pat-inject="
+                        source: body;
+                        target: body;
+                        history: record;
+                    ">link</a>
+                `;
+
+                answer(`
+                    <html>
+                        <body>
+                            OK
+                        </body>
+                    </html>
+                `);
+
+                const inject = document.querySelector(".pat-inject");
+
+                pattern.init($(inject));
+                await utils.timeout(1); // wait a tick for async to settle.
+
+                inject.click();
+
+                await utils.timeout(1); // wait a tick for async to settle.
+
+                expect(document.body.textContent.trim()).toBe("OK");
+
+                // Title in head  target is not modified.
+                const title = document.head.querySelector("title");
+                expect(title).toBeTruthy();
+                expect(title.textContent.trim()).toBe("test"); // Old title
+            });
+
+            it("9.4.4 - Does not break, if no title is found in target", async function () {
+                document.head.innerHTML = "";
+                document.body.innerHTML = `
+                    <a class="pat-inject"
+                       href="test.html"
+                       data-pat-inject="
+                        source: body;
+                        target: body;
+                        history: record;
+                    ">link</a>
+                `;
+
+                answer(`
+                    <html>
+                        <head>
+                            <title>hello</title>
+                        <body>
+                            OK
+                        </body>
+                    </html>
+                `);
+
+                const inject = document.querySelector(".pat-inject");
+
+                pattern.init($(inject));
+                await utils.timeout(1); // wait a tick for async to settle.
+
+                inject.click();
+
+                await utils.timeout(1); // wait a tick for async to settle.
+
+                expect(document.body.textContent.trim()).toBe("OK");
+
+                // There is no title to be updated in target.
+                const title = document.head.querySelector("title");
+                expect(title).toBeFalsy();
+            });
+
         });
 
         describe("9.5 - support multiple source element matches.", function () {

Original file line number	Diff line number	Diff line change
`@@ -504,7 +504,7 @@ const inject = {`
`504`	`504`	`}`
`505`	`505`	`history.pushState({ url: url }, "", url);`
`506`	`506`	`// Also inject title element if we have one`
`507`		`- if ($title.length) {`
	`507`	`+ if ($title?.length) {`
`508`	`508`	`const title_el = document.querySelector("title");`
`509`	`509`	`if (title_el) {`
`510`	`510`	`this._inject(trigger, $title, title_el, {`