fix(core): Do not serialize non-multi params into an array

Airblader · Airblader · commit bf6c3554a6cd · 2019-01-04T19:11:02.000+01:00
diff --git a/projects/ngqp/core/src/lib/query-param-group.directive.ts b/projects/ngqp/core/src/lib/query-param-group.directive.ts
@@ -158,10 +158,10 @@ export class QueryParamGroupDirective implements OnInit, OnDestroy {
         };
     }
 
-    private serialize<T = any>(queryParam: QueryParam<T | T[]>, value: T): string[] {
+    private serialize<T = any>(queryParam: QueryParam<T | T[]>, value: T): string | string[] {
         return hasArrayValue(queryParam, value)
             ? (value || <T[]>[]).map(queryParam.serialize)
-            : [queryParam.serialize(value)];
+            : queryParam.serialize(value);
     }
 
     private deserialize<T = any>(queryParam: QueryParam<T>, values: string | string[]): Unpack<T> | Unpack<T>[] {

Original file line number	Diff line number	Diff line change
`@@ -158,10 +158,10 @@ export class QueryParamGroupDirective implements OnInit, OnDestroy {`
`158`	`158`	`};`
`159`	`159`	`}`
`160`	`160`
`161`		`- private serialize<T = any>(queryParam: QueryParam<T \| T[]>, value: T): string[] {`
	`161`	`+ private serialize<T = any>(queryParam: QueryParam<T \| T[]>, value: T): string \| string[] {`
`162`	`162`	`return hasArrayValue(queryParam, value)`
`163`	`163`	`? (value \|\| <T[]>[]).map(queryParam.serialize)`
`164`		`- : [queryParam.serialize(value)];`
	`164`	`+ : queryParam.serialize(value);`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`private deserialize<T = any>(queryParam: QueryParam<T>, values: string \| string[]): Unpack<T> \| Unpack<T>[] {`