fix(core): check if previous state is null for unmounted component (#5724)

cngonzalez · web-flow · commit 9ed5cca51ba8 · 2024-02-29T18:14:47.000Z
diff --git a/packages/sanity/src/core/form/inputs/CrossDatasetReferenceInput/useReferenceInfo.ts b/packages/sanity/src/core/form/inputs/CrossDatasetReferenceInput/useReferenceInfo.ts
@@ -78,7 +78,7 @@ export function useReferenceInfo(
   // workaround for a "bug" with useMemoObservable that doesn't
   // return the initial value upon resubscription
   const previousId = usePrevious(doc._id, doc._id)
-  if (previousId !== doc._id) {
+  if (doc._id && previousId !== doc._id) {
     return INITIAL_LOADING_STATE
   }
   return referenceInfo

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ export function useReferenceInfo(`
`78`	`78`	`// workaround for a "bug" with useMemoObservable that doesn't`
`79`	`79`	`// return the initial value upon resubscription`
`80`	`80`	`const previousId = usePrevious(doc._id, doc._id)`
`81`		`- if (previousId !== doc._id) {`
	`81`	`+ if (doc._id && previousId !== doc._id) {`
`82`	`82`	`return INITIAL_LOADING_STATE`
`83`	`83`	`}`
`84`	`84`	`return referenceInfo`