minor #21677 [Serializer] Removed duplicate operation in camelcase denormalization (gadelat)

fabpot · fabpot · commit 50324777bf0d · 2017-02-19T15:30:59.000-08:00
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #21677). Discussion ---------- [Serializer] Removed duplicate operation in camelcase denormalization | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Commits ------- 32fcd91 [Serializer] Removed duplicate operation in camelcase denormalization
diff --git a/src/Symfony/Component/Serializer/NameConverter/CamelCaseToSnakeCaseNameConverter.php b/src/Symfony/Component/Serializer/NameConverter/CamelCaseToSnakeCaseNameConverter.php
@@ -76,7 +76,7 @@ public function denormalize($propertyName)
         }
 
         if (null === $this->attributes || in_array($camelCasedName, $this->attributes)) {
-            return $this->lowerCamelCase ? lcfirst($camelCasedName) : $camelCasedName;
+            return $camelCasedName;
         }
 
         return $propertyName;

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public function denormalize($propertyName)`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`if (null === $this->attributes \|\| in_array($camelCasedName, $this->attributes)) {`
`79`		`- return $this->lowerCamelCase ? lcfirst($camelCasedName) : $camelCasedName;`
	`79`	`+ return $camelCasedName;`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`return $propertyName;`