Set query parameter for array of primitive types

[dhruvCW]

Just a rebase of #886 onto the latest changes in master.

Fixes #883

[dhruvCW]

Failing tests seem to be because of a network hiccup 😅

[LeFnord]

thanks @dhruvCW … is it an addition to #927?
if yes, then I think some things can be refactored, especially stuff from the array_param_type method, or is it an rebase mistake?

[dhruvCW]

thanks @dhruvCW … is it an addition to #927? if yes, then I think some things can be refactored, especially stuff from the array_param_type method, or is it an rebase mistake?

it indeed builds on top of #927 specifically handling the case when an array parameter is incorrectly mapped as formData even when it's part of a GET request. This PR also fixes the problem wherein array parameters always default to formData (this was missed in #927)

[dhruvCW]

I think some things can be refactored, especially stuff from the array_param_type method, or is it an rebase mistake?

could you point them out please 😅 I just rebased the change so I haven't given the implementation itself much thought 😅 I refactored the existing param_type method to also handle array types. That removed the need for an array_param_type method.

[LeFnord]

thanks @dhruvCW

[dhruvCW]

thanks for merging the change ❤️ . Could we get a release soon ? This and the previous PR would help fix a long standing issue with the generated swagger docs.

[LeFnord]

released