Merge pull request #352 from OpenAPI-Qraft/fix/request-fn-micro-optim

radist2s · web-flow · commit c0504b23826a · 2025-06-21T20:43:08.000+04:00
refactor(react-client): remove the unnecessary duplicate `toString()` call on `getQueryString()`
diff --git a/.changeset/quick-friends-beam.md b/.changeset/quick-friends-beam.md
@@ -0,0 +1,2 @@
+---
+---
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)[]) {`