Skip to content

Conversation

stcarrez
Copy link
Contributor

@stcarrez stcarrez commented Jan 27, 2018

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\.
  • Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

The Ada client body template was missing to take into account the hasFormParams when doing the call to the swagger Ada runtime and it forgets passing the request instance that contained the form parameters. Hence, an empty body was submitted.

The pull request fixes the template and updates the Ada client petstore samples.

To fix #7511

@wing328 wing328 added this to the v2.4.0 milestone Jan 28, 2018
@wing328 wing328 merged commit 4bd8fc6 into swagger-api:master Jan 28, 2018
@wing328 wing328 changed the title Fix 7511: [Ada] Client call is not correct with multiple parameters and application/x-www-urlencoded [Ada] Client call is not correct with multiple parameters and application/x-www-urlencoded Jan 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Ada] Client call is not correct with multiple parameters and application/x-www-urlencoded
2 participants