Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: symfony/symfony
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: e880e53
Choose a base ref
...
head repository: symfony/symfony
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: a923e6c
Choose a head ref
  • 4 commits
  • 4 files changed
  • 3 contributors

Commits on Dec 16, 2025

  1. [Serializer] Do not skip nested null values when denormalizing

    @MatTheCat @nicolas-grekas
    MatTheCat authored and nicolas-grekas committed Dec 16, 2025
    Configuration menu
    Copy the full SHA
    4b8580f View commit details
    Browse the repository at this point in the history

  2. bug #62773 [Serializer] Do not skip nested null values when denorma… 

    …lizing (MatTheCat)

This PR was squashed before being merged into the 6.4 branch.

Discussion
----------

[Serializer] Do not skip nested `null` values when denormalizing

| 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.

Commits
-------

4b8580f [Serializer] Do not skip nested `null` values when denormalizing
    @nicolas-grekas
    nicolas-grekas committed Dec 16, 2025
    Configuration menu
    Copy the full SHA
    acbd949 View commit details
    Browse the repository at this point in the history

  3. [PropertyInfo] Fix calling same-named method with required args inste… 

    …ad of reading public property
    @makraz @nicolas-grekas
    makraz authored and nicolas-grekas committed Dec 16, 2025
    Configuration menu
    Copy the full SHA
    02e3153 View commit details
    Browse the repository at this point in the history

  4. bug #62772 [PropertyInfo] Fix calling same-named method with required… 

    … args instead of reading public property (makraz)

This PR was submitted for the 7.3 branch but it was merged into the 6.4 branch instead.

Discussion
----------

[PropertyInfo] Fix calling same-named method with required args instead of reading public property

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #62733
| License       | MIT

- Fixes PropertyAccess choosing a same‑named method over a public property when the method requires parameters.
- Ensures jQuery‑style accessors (methods named exactly like the property) are only considered if they have zero required parameters.
- Prevents `ArgumentCountError` during serialization and aligns behavior with standard getter rules (`getX()/isX()/hasX()/canX()`), which already require zero required parameters.

Commits
-------

02e3153 [PropertyInfo] Fix calling same-named method with required args instead of reading public property
    @nicolas-grekas
    nicolas-grekas committed Dec 16, 2025
    Configuration menu
    Copy the full SHA
    a923e6c View commit details
    Browse the repository at this point in the history
Loading