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

[jungwoo3490]

Fixes #11448

I fixed location's state property value like this.

let location: Location = {
  pathname: locationProp.pathname || "/",
  search: locationProp.search || "",
  hash: locationProp.hash || "",
  state: locationProp.state ?? null, // fixed
  key: locationProp.key || "default",
};

Now, useLocation receive 0 instead of null when state is set to 0.

[changeset-bot]

🦋 Changeset detected

Latest commit: adaeedf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
react-router-dom	Patch
react-router-dom-v5-compat	Patch
react-router	Patch
react-router-native	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[emrerdem1]

Wouldn't it be better to use the nullish coalescing operator here?

 state: locationProp.state ?? null,

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing

[jungwoo3490]

Wouldn't it be better to use the nullish coalescing operator here?

 state: locationProp.state ?? null,

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing

Thanks!! I'll apply it :)

[remix-cla-bot]

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

[brophdawg11]

Could you add a unit test to packages/react-router-dom/__tests__/static-location-test.tsx?

[jungwoo3490]

Could you add a unit test to packages/react-router-dom/__tests__/static-location-test.tsx?

I added it!! @brophdawg11

[brophdawg11]

Thanks - now I think we just need a changeset and this should be good to merge. Just run pnpm changeset and it'll prompt you through the creation of one.

[jungwoo3490]

Thanks - now I think we just need a changeset and this should be good to merge. Just run pnpm changeset and it'll prompt you through the creation of one.

I created changeset and commited it. @brophdawg11

[github-actions]

🤖 Hello there,

We just published version 6.24.0-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 6.24.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!