Skip to content

Conversation

@MatTheCat
Copy link
Contributor

@MatTheCat MatTheCat commented Dec 15, 2025

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues N/A
License MIT

Since the nested values accessor was built without THROW_ON_INVALID_INDEX there was no way to distinguish between null and missing values. As a result null values were considered missing.

@MatTheCat MatTheCat requested a review from dunglas as a code owner December 15, 2025 11:54
@carsonbot carsonbot added this to the 6.4 milestone Dec 15, 2025
@MatTheCat MatTheCat force-pushed the denormalize_nested_null_value branch from b2e8f9d to a6dbe49 Compare December 15, 2025 13:42
@MatTheCat MatTheCat marked this pull request as draft December 15, 2025 13:55
@MatTheCat MatTheCat force-pushed the denormalize_nested_null_value branch from a6dbe49 to 58a9593 Compare December 15, 2025 13:59
@MatTheCat MatTheCat marked this pull request as ready for review December 15, 2025 14:02
@MatTheCat MatTheCat force-pushed the denormalize_nested_null_value branch from 58a9593 to 877392b Compare December 15, 2025 14:11
@nicolas-grekas nicolas-grekas force-pushed the denormalize_nested_null_value branch from 87f6212 to 4b8580f Compare December 16, 2025 10:31
@nicolas-grekas
Copy link
Member

Thank you @MatTheCat.

@nicolas-grekas nicolas-grekas merged commit acbd949 into symfony:6.4 Dec 16, 2025
11 of 12 checks passed
@MatTheCat MatTheCat deleted the denormalize_nested_null_value branch December 16, 2025 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants