Merge branch 'main' into bump-to-v3.18.1

Author: CollinBeczak

package-lock.json

@@ -72,7 +72,7 @@
     "leaflet-lasso": "^2.2.13",
     "leaflet-vectoricon": "https://github.com/CollinBeczak/Leaflet.VectorIcon",
     "leaflet.markercluster": "^1.5.3",
-    "lodash": "4.17.23",
+    "lodash": "4.18.1",
     "mapillary-js": "^4.1.2",
     "normalize.css": "^7.0.0",
     "normalizr": "^3.6.0",

package.json

@@ -62,7 +62,7 @@ export const replacePropertyTags = (
     }
 
     // Conditionally url encode replacement value
-    const replacement = _get(properties, tagName, "");
+    const replacement = _get(properties, tagName, "") ?? "";
     return firstChar + (urlEncodeReplacement ? encodeURIComponent(replacement) : replacement);
   });
 };

src/hooks/UsePropertyReplacement/UsePropertyReplacement.js

@@ -94,6 +94,12 @@ describe("usePropertyReplacement additional tests", () => {
     expect(result.current).toBe(content);
   });
 
+  it("replaces null property values with empty string", () => {
+    const content = "alt_name={{alt_name}}";
+    const { result } = renderHook(() => usePropertyReplacement(content, { alt_name: null }));
+    expect(result.current).toBe("alt_name=");
+  });
+
   it("handles content with special characters in property values", () => {
     const content = "Special: {{special}}";
     const { result } = renderHook(() => usePropertyReplacement(content, properties));