fix: bug #1433

Author: js2me

.changeset/pretty-chicken-film.md

@@ -0,0 +1,5 @@
+---
+"swagger-typescript-api": patch
+---
+
+fixed bug linked with unused query param (bug #1433 , 1 point)

templates/default/procedure-call.ejs

@@ -21,10 +21,16 @@ const requestConfigParam = {
 
 const argToTmpl = ({ name, optional, type, defaultValue }) => `${name}${!defaultValue && optional ? '?' : ''}: ${type}${defaultValue ? ` = ${defaultValue}` : ''}`;
 
+const extractedRequestParamsName = pathParams.length
+    ? query
+        ? `{ ${_.join(pathParamsNames, ", ")}, ...${queryName} }`
+        : `{ ${_.join(pathParamsNames, ", ")} }`
+    : queryName;
+
 const rawWrapperArgs = config.extractRequestParams ?
     _.compact([
         requestParams && {
-          name: pathParams.length ? `{ ${_.join(pathParamsNames, ", ")}, ...${queryName} }` : queryName,
+          name: extractedRequestParamsName,
           optional: false,
           type: getInlineParseContent(requestParams),
         },

templates/modular/procedure-call.ejs

@@ -21,10 +21,16 @@ const requestConfigParam = {
 
 const argToTmpl = ({ name, optional, type, defaultValue }) => `${name}${!defaultValue && optional ? '?' : ''}: ${type}${defaultValue ? ` = ${defaultValue}` : ''}`;
 
+const extractedRequestParamsName = pathParams.length
+    ? query
+        ? `{ ${_.join(pathParamsNames, ", ")}, ...${queryName} }`
+        : `{ ${_.join(pathParamsNames, ", ")} }`
+    : queryName;
+
 const rawWrapperArgs = config.extractRequestParams ?
     _.compact([
         requestParams && {
-          name: pathParams.length ? `{ ${_.join(pathParamsNames, ", ")}, ...${queryName} }` : queryName,
+          name: extractedRequestParamsName,
           optional: false,
           type: getInlineParseContent(requestParams),
         },