Skip to content

[Serializer] fix upper camel case conversion (see #21399) #21400

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 16, 2017

Conversation

markusu49
Copy link
Contributor

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? ?
Deprecations? no
Tests pass? yes
Fixed tickets #21399
License MIT

Improve upper camel case support of CamelCaseToSnakeCaseConverter (ThisIsATest now converts to this_is_a_test instead of _this_is_a_test).

@chalasr
Copy link
Member

chalasr commented Jan 25, 2017

👍 build failure unrelated, 2.7 needs a merge into master

@dunglas
Copy link
Member

dunglas commented Jan 26, 2017

👍

@fabpot
Copy link
Member

fabpot commented Feb 16, 2017

Thank you @markusu49.

@fabpot fabpot merged commit 81e771c into symfony:2.7 Feb 16, 2017
fabpot added a commit that referenced this pull request Feb 16, 2017
…(markusu49)

This PR was merged into the 2.7 branch.

Discussion
----------

[Serializer] fix upper camel case conversion (see #21399)

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | ?
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #21399
| License       | MIT

Improve upper camel case support of CamelCaseToSnakeCaseConverter (`ThisIsATest` now converts to `this_is_a_test` instead of `_this_is_a_test`).

Commits
-------

81e771c [Serializer] fix upper camel case conversion (see #21399)
@fabpot fabpot mentioned this pull request Feb 17, 2017
This was referenced Mar 6, 2017
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.

6 participants