Merge pull request #30 from hosmelq/check-possible-undefined

Check possible undefined

Author: timacdonald

resources/js/wayfinder.ts

@@ -60,7 +60,9 @@ export const queryParams = (options?: {
             });
 
             for (const subKey in query[key]) {
-                params.set(`${key}[${subKey}]`, getValue(query[key][subKey]));
+                if (query[key][subKey] !== undefined) {
+                    params.set(`${key}[${subKey}]`, getValue(query[key][subKey]));
+                }
             }
         } else {
             params.set(key, getValue(query[key]));

tests/QueryParams.test.ts

@@ -168,3 +168,29 @@ it("can merge with the form method", () => {
         method: "get",
     });
 });
+
+it("handles nested objects with undefined keys", () => {
+    window.location.search = "?parent=og";
+
+    const query = (): {
+        good: string;
+    } => {
+        const obj = {
+            good: 'value',
+            bad: undefined,
+        }
+
+        return obj
+    }
+
+    expect(
+        index.form.head({
+            mergeQuery: {
+                parent: query(),
+            },
+        }),
+    ).toEqual({
+        action: "/posts?parent=og&_method=HEAD&parent%5Bgood%5D=value",
+        method: "get",
+    });
+});