fix: handle properly null and undefined props values

Co-authored-by: Julien <[email protected]>

src/inertia.ts

@@ -103,7 +103,11 @@ export class Inertia {
      */
     if (!isPartial) {
       newProps = Object.fromEntries(
-        Object.entries(props).filter(([_, value]) => !(value as any)[ignoreFirstLoadSymbol])
+        Object.entries(props).filter(([_, value]) => {
+          if (value && (value as any)[ignoreFirstLoadSymbol]) return false
+
+          return true
+        })
       )
     }
 

tests/inertia.spec.ts

@@ -212,6 +212,24 @@ test.group('Inertia', () => {
     assert.deepEqual(result.mergeProps, ['baz', 'bar'])
   })
 
+  test('properly handle null and undefined values props on first visit', async ({ assert }) => {
+    setupViewMacroMock()
+
+    const inertia = await new InertiaFactory().create()
+
+    const result: any = await inertia.render('Auth/Login', {
+      user: undefined,
+      password: null,
+      message: 'hello',
+    })
+
+    assert.deepEqual(result.props.page.props, {
+      message: 'hello',
+      password: null,
+      user: undefined,
+    })
+  })
+
   test("don't return lazy props on first visit", async ({ assert }) => {
     setupViewMacroMock()