Fix useLocation to receive 0 instead of null when state is set to 0 (#…

…11495)

.changeset/smooth-sloths-exist.md

@@ -0,0 +1,6 @@
+---
+"react-router-dom": patch
+"react-router-dom-v5-compat": patch
+---
+
+Allow falsy `location.state` values passed to `<StaticRouter>`

contributors.yml

@@ -254,3 +254,4 @@
 - yracnet
 - yuleicul
 - zheng-chuang
+- jungwoo3490

packages/react-router-dom-v5-compat/lib/components.tsx

@@ -86,7 +86,7 @@ export function StaticRouter({
     pathname: locationProp.pathname || "/",
     search: locationProp.search || "",
     hash: locationProp.hash || "",
-    state: locationProp.state || null,
+    state: locationProp.state != null ? locationProp.state : null,
     key: locationProp.key || "default",
   };
 

packages/react-router-dom/__tests__/static-location-test.tsx

@@ -56,5 +56,31 @@ describe("A <StaticRouter>", () => {
         key: expect.any(String),
       });
     });
+
+    it("retains a non-null state when passed explicitly", () => {
+      let location!: ReturnType<typeof useLocation>;
+      function LocationChecker() {
+        location = useLocation();
+        return null;
+      }
+
+      ReactDOMServer.renderToStaticMarkup(
+        <StaticRouter
+          location={{ pathname: "/the/path", search: "?the=query", state: 0 }}
+        >
+          <Routes>
+            <Route path="/the/path" element={<LocationChecker />} />
+          </Routes>
+        </StaticRouter>
+      );
+
+      expect(location).toEqual({
+        pathname: "/the/path",
+        search: "?the=query",
+        hash: "",
+        state: 0,
+        key: expect.any(String),
+      });
+    });
   });
 });

packages/react-router-dom/server.tsx

@@ -66,7 +66,7 @@ export function StaticRouter({
     pathname: locationProp.pathname || "/",
     search: locationProp.search || "",
     hash: locationProp.hash || "",
-    state: locationProp.state || null,
+    state: locationProp.state != null ? locationProp.state : null,
     key: locationProp.key || "default",
   };