tighten dependency

ztanner · ztanner · commit 82d76dbcbe8e · 2023-10-12T19:11:19.000-07:00
diff --git a/packages/next/src/client/components/navigation.ts b/packages/next/src/client/components/navigation.ts
@@ -172,13 +172,13 @@ export function useParams<T extends Params = Params>(): T {
 
   return useMemo(() => {
     // When it's under app router
-    if (globalLayoutRouter) {
+    if (globalLayoutRouter?.tree) {
       return getSelectedParams(globalLayoutRouter.tree) as T
     }
 
     // When it's under client side pages router
     return pathParams as T
-  }, [globalLayoutRouter, pathParams])
+  }, [globalLayoutRouter?.tree, pathParams])
 }
 
 // TODO-APP: handle parallel routes

Original file line number	Diff line number	Diff line change
`@@ -172,13 +172,13 @@ export function useParams<T extends Params = Params>(): T {`
`172`	`172`
`173`	`173`	`return useMemo(() => {`
`174`	`174`	`// When it's under app router`
`175`		`- if (globalLayoutRouter) {`
	`175`	`+ if (globalLayoutRouter?.tree) {`
`176`	`176`	`return getSelectedParams(globalLayoutRouter.tree) as T`
`177`	`177`	`}`
`178`	`178`
`179`	`179`	`// When it's under client side pages router`
`180`	`180`	`return pathParams as T`
`181`		`- }, [globalLayoutRouter, pathParams])`
	`181`	`+ }, [globalLayoutRouter?.tree, pathParams])`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`// TODO-APP: handle parallel routes`