refactor(react-client): remove the unnecessary duplicate toString() call on getQueryString().

radist2s · radist2s · commit 8d911b9b27a7 · 2025-06-21T20:38:51.000+04:00
diff --git a/packages/react-client/src/lib/requestFn.ts b/packages/react-client/src/lib/requestFn.ts
@@ -133,7 +133,9 @@ function getQueryString(params: Record<string, any>): string {
 
   Object.entries(params).forEach(([k, v]) => walk(k, v));
 
-  return search.toString() ? `?${search.toString()}` : '';
+  const searchString = search.toString();
+
+  return searchString ? `?${searchString}` : '';
 }
 
 export function mergeHeaders(...allHeaders: (HeadersOptions | undefined)[]) {

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,9 @@ function getQueryString(params: Record<string, any>): string {`
`133`	`133`
`134`	`134`	`Object.entries(params).forEach(([k, v]) => walk(k, v));`
`135`	`135`
`136`		- return search.toString() ? `?${search.toString()}` : '';
	`136`	`+ const searchString = search.toString();`
	`137`	`+`
	`138`	+ return searchString ? `?${searchString}` : '';
`137`	`139`	`}`
`138`	`140`
`139`	`141`	`export function mergeHeaders(...allHeaders: (HeadersOptions \| undefined)[]) {`