bug #52683 [Serializer] Fix constructor deserialization path (mtarld)

nicolas-grekas · nicolas-grekas · commit ba6ddd69f4f5 · 2023-11-23T18:36:34.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [Serializer] Fix constructor deserialization path | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix #44925 | License | MIT This fix doesn't have to be upmerged because it already has been covered by symfony/symfony#46680. Commits ------- 272bc28 [Serializer] Fix constructor deserialization path
diff --git a/Normalizer/AbstractNormalizer.php b/Normalizer/AbstractNormalizer.php
@@ -489,6 +489,8 @@ protected function denormalizeParameter(\ReflectionClass $class, \ReflectionPara
      */
     protected function createChildContext(array $parentContext, string $attribute, ?string $format): array
     {
+        $parentContext['deserialization_path'] = ($parentContext['deserialization_path'] ?? false) ? $parentContext['deserialization_path'].'.'.$attribute : $attribute;
+
         if (isset($parentContext[self::ATTRIBUTES][$attribute])) {
             $parentContext[self::ATTRIBUTES] = $parentContext[self::ATTRIBUTES][$attribute];
         } else {

Original file line number	Diff line number	Diff line change
`@@ -489,6 +489,8 @@ protected function denormalizeParameter(\ReflectionClass $class, \ReflectionPara`
`489`	`489`	`*/`
`490`	`490`	`protected function createChildContext(array $parentContext, string $attribute, ?string $format): array`
`491`	`491`	`{`
	`492`	`+ $parentContext['deserialization_path'] = ($parentContext['deserialization_path'] ?? false) ? $parentContext['deserialization_path'].'.'.$attribute : $attribute;`
	`493`	`+`
`492`	`494`	`if (isset($parentContext[self::ATTRIBUTES][$attribute])) {`
`493`	`495`	`$parentContext[self::ATTRIBUTES] = $parentContext[self::ATTRIBUTES][$attribute];`
`494`	`496`	`} else {`