symfony/symfony — pull request #2977 by ericclemmons (2011-12-27 21:21:47+00:00)
Title: [Serializer] GetSetMethodNormalizer uses `lcfirst` instead of `strtolower`

Bug fix: yes
Feature addition: no
Backwards compatibility break: yes
Symfony2 tests pass: yes
Fixes the following tickets: #2863

As a courtesy to @lxg, this PR calculates method names using `lcfirst` instead of `strtolower`: `fooBar` was converted to `getFoobar`, but is now `getFooBar`.

However, according to the [PHP documentation](http://www.php.net/manual/en/functions.user-defined.php):

> **Note**: Function names are case-insensitive, though it is usually good form to call functions as they appear in their declaration.

In my opinion, this PR is simply cleanup, letting the normalized structure accurately reflect the underlying methods/properties. If anyone is using the normalizer now, their keys may change, causing a BC break.

If this PR is considered for merging, I recommend making it in `master` rather than `2.0`, because of potential BC issues.