minor #52992 [Serializer] Make sure Serializer::denormalize() shows what exception it throws (Nyholm)

fabpot · fabpot · commit 1ae24df3dd62 · 2023-12-13T08:20:46.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [Serializer] Make sure Serializer::denormalize() shows what exception it throws | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | no | License | MIT This was missing from #42502. We have no contract that explains that this exceptions should be thrown. With this addition, we technically make this a "sometimes" feature of the Serializer class. Commits ------- bca846b Make sure Serializer::denormalize have show what exception it throws
diff --git a/src/Symfony/Component/Serializer/Serializer.php b/src/Symfony/Component/Serializer/Serializer.php
@@ -197,6 +197,7 @@ public function normalize($data, string $format = null, array $context = [])
      * {@inheritdoc}
      *
      * @throws NotNormalizableValueException
+     * @throws PartialDenormalizationException Occurs when one or more properties of $type fails to denormalize
      */
     public function denormalize($data, string $type, string $format = null, array $context = [])
     {

Original file line number	Diff line number	Diff line change
`@@ -197,6 +197,7 @@ public function normalize($data, string $format = null, array $context = [])`
`197`	`197`	`* {@inheritdoc}`
`198`	`198`	`*`
`199`	`199`	`* @throws NotNormalizableValueException`
	`200`	`+ * @throws PartialDenormalizationException Occurs when one or more properties of $type fails to denormalize`
`200`	`201`	`*/`
`201`	`202`	`public function denormalize($data, string $type, string $format = null, array $context = [])`
`202`	`203`	`{`