pat inject: Rebase URLs in pattern configuration attributes /3

Do not rebase pattern configuration values when they are empty.

Solves a problem where accidently empty options (e.g. pat-inject url)
are rebased and a subsequent pat-inject gets the rebased option
inherited which then invalidly masks the href attribute of the subsequent
pat-inject.

Author: thet

src/pat/inject/inject.js

@@ -931,7 +931,7 @@ const inject = {
                     let changed = false;
                     for (const opt of opts) {
                         const val = options[opt];
-                        if (typeof val === "undefined") {
+                        if (!val) {
                             continue;
                         }
                         changed = true;

src/pat/inject/inject.test.js

@@ -347,6 +347,29 @@ describe("pat-inject", function () {
 
             done();
         });
+
+        it("doesn't rebase invalid pattern configuration options", async (done) => {
+            await import("../calendar/calendar");
+
+            const res = pattern._rebaseHTML(
+                "https://example.com/test/",
+                '<div id="test" data-pat-calendar="url: ; event-sources: ../calendar2.json"/>'
+            );
+            console.log(res);
+
+            const el = document.createElement("div");
+            el.innerHTML = res;
+
+            const test_config = JSON.parse(
+                el.querySelector("#test").getAttribute("data-pat-calendar")
+            );
+            expect(test_config.url).toEqual("");
+            expect(test_config["event-sources"]).toEqual([
+                "https://example.com/test/../calendar2.json",
+            ]);
+
+            done();
+        });
     });
 
     describe("parseRawHtml", function () {