bug #53081 [Serializer] Keep stack trace for enum value denormalizer error (kylekatarnls)

nicolas-grekas · nicolas-grekas · commit 37708cb97248 · 2023-12-18T15:15:26.000+01:00
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- [Serializer] Keep stack trace for enum value denormalizer error | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | License | MIT Simple `previous: $exception` added not to loose track of the error origin. Commits ------- de3f711 [Serializer] Keep stack trace for enum value denormalizer error
diff --git a/src/Symfony/Component/Serializer/Normalizer/BackedEnumNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/BackedEnumNormalizer.php
@@ -65,7 +65,7 @@ public function denormalize($data, string $type, string $format = null, array $c
             return $type::from($data);
         } catch (\ValueError $e) {
             if (isset($context['has_constructor'])) {
-                throw new InvalidArgumentException('The data must belong to a backed enumeration of type '.$type);
+                throw new InvalidArgumentException('The data must belong to a backed enumeration of type '.$type, 0, $e);
             }
 
             throw NotNormalizableValueException::createForUnexpectedDataType('The data must belong to a backed enumeration of type '.$type, $data, [$type], $context['deserialization_path'] ?? null, true, 0, $e);

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public function denormalize($data, string $type, string $format = null, array $c`
`65`	`65`	`return $type::from($data);`
`66`	`66`	`} catch (\ValueError $e) {`
`67`	`67`	`if (isset($context['has_constructor'])) {`
`68`		`- throw new InvalidArgumentException('The data must belong to a backed enumeration of type '.$type);`
	`68`	`+ throw new InvalidArgumentException('The data must belong to a backed enumeration of type '.$type, 0, $e);`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`throw NotNormalizableValueException::createForUnexpectedDataType('The data must belong to a backed enumeration of type '.$type, $data, [$type], $context['deserialization_path'] ?? null, true, 0, $e);`