Skip to content

Conversation

@wing328
Copy link
Contributor

@wing328 wing328 commented Feb 17, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch 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)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Use original path name for path variable naming to resolve #4257

@wing328 wing328 merged commit 835c7c0 into swagger-api:master Mar 3, 2017
@wing328 wing328 deleted the silex_fix_path_name branch March 3, 2017 08:49
spr3nk3ls pushed a commit to spr3nk3ls/swagger-codegen that referenced this pull request Mar 28, 2017
…-api#4817)

* use original path name in php silex

* fix path variable naming by camelizing the name
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.

[PHP][Silex] Generated parameters don't use same naming conventions as path variables

1 participant