api-platform / core / 25727713442

27%

master: 37%

Build Type

push

github

Committed by web-flow

Commit Message

fix(serializer): translate PropertyAccess type mismatches to NotNormalizableValueException (#7967)

* fix(serializer): translate PropertyAccess type mismatches to NotNormalizableValueException

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Tickets       | symfony/symfony#64159
| License       | MIT
| Doc PR        | ∅

AbstractItemNormalizer::setAttributeValue() now catches PropertyAccess\InvalidArgumentException
and rethrows as NotNormalizableValueException, mirroring the contract ObjectNormalizer implements
per symfony/symfony#64067. Without this, null on a non-nullable typed property bubbled up as
HTTP 500 instead of a 4xx denormalization error.

* chore(serializer): bump symfony/serializer minimum to require setAttributeValue contract

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Tickets       | symfony/symfony#64067
| License       | MIT
| Doc PR        | ∅

The COLLECT_DENORMALIZATION_ERRORS aggregation in AbstractObjectNormalizer was reworked in
symfony/symfony#64067 (released in 6.4.37 / 7.4.9 / 8.0.9). Now that AbstractItemNormalizer
implements that contract, older Symfony parents would no longer aggregate our translated
exceptions and would respond with 400 instead of 422.

Coverage Stats

2 of 2 new or added lines in 1 file covered. (100.0%)

498 existing lines in 44 files now uncovered.

18470 of 63935 relevant lines covered (28.89%)

78.99 hits per line