Support for collections in query strings

Balakrishnan Ranganathan · Balakrishnan Ranganathan · commit d0848b135b42 · 2016-08-29T18:17:42.000+02:00
diff --git a/src/WebApiToTypeScript/WebApiToTypeScript.cs b/src/WebApiToTypeScript/WebApiToTypeScript.cs
@@ -188,16 +188,24 @@ private void CreateQueryStringBlock(TypeScriptBlock classBlock, WebApiAction act
 
                 if (argumentType.IsPrimitive || argumentType.IsEnum)
                 {
-                    block
-                        .AddStatement($"parameters.push(`{argumentName}=${{this.{argumentName}}}`);");
+                    if (argumentType.IsCollection)
+                    {
+                        block
+                            .AddStatement($"parameters.push(`{argumentName}=${{this.{argumentName}.join(',')}}`);");
+                    }
+                    else
+                    {
+                        block
+                            .AddStatement($"parameters.push(`{argumentName}=${{encodeURIComponent(this.{argumentName}.toString())}}`);");
+                    }
                 }
                 else
                 {
                     block
                         .AddStatement($"let {argumentName}Params = this.{argumentName}.getQueryParams();")
                         .AddAndUseBlock($"Object.keys({argumentName}Params).forEach((key) =>", isFunctionBlock: true, terminateWithSemicolon: true)
                         .AddAndUseBlock($"if ({argumentName}Params[key] != null)")
-                        .AddStatement($"parameters.push(`${{key}}=${{encodeURIComponent({argumentName}Params[key])}}`);");
+                        .AddStatement($"parameters.push(`${{key}}=${{encodeURIComponent({argumentName}Params[key].toString())}}`);");
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -188,16 +188,24 @@ private void CreateQueryStringBlock(TypeScriptBlock classBlock, WebApiAction act`
`188`	`188`
`189`	`189`	`if (argumentType.IsPrimitive \|\| argumentType.IsEnum)`
`190`	`190`	`{`
`191`		`- block`
`192`		- .AddStatement($"parameters.push(`{argumentName}=${{this.{argumentName}}}`);");
	`191`	`+ if (argumentType.IsCollection)`
	`192`	`+ {`
	`193`	`+ block`
	`194`	+ .AddStatement($"parameters.push(`{argumentName}=${{this.{argumentName}.join(',')}}`);");
	`195`	`+ }`
	`196`	`+ else`
	`197`	`+ {`
	`198`	`+ block`
	`199`	+ .AddStatement($"parameters.push(`{argumentName}=${{encodeURIComponent(this.{argumentName}.toString())}}`);");
	`200`	`+ }`
`193`	`201`	`}`
`194`	`202`	`else`
`195`	`203`	`{`
`196`	`204`	`block`
`197`	`205`	`.AddStatement($"let {argumentName}Params = this.{argumentName}.getQueryParams();")`
`198`	`206`	`.AddAndUseBlock($"Object.keys({argumentName}Params).forEach((key) =>", isFunctionBlock: true, terminateWithSemicolon: true)`
`199`	`207`	`.AddAndUseBlock($"if ({argumentName}Params[key] != null)")`
`200`		- .AddStatement($"parameters.push(`${{key}}=${{encodeURIComponent({argumentName}Params[key])}}`);");
	`208`	+ .AddStatement($"parameters.push(`${{key}}=${{encodeURIComponent({argumentName}Params[key].toString())}}`);");
`201`	`209`	`}`
`202`	`210`	`}`
`203`	`211`