Thanks to visit codestin.com
Credit goes to github.com

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/serializer
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: bd395bb
Choose a base ref
...
head repository: symfony/serializer
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 006fd51
Choose a head ref
  • 12 commits
  • 7 files changed
  • 6 contributors

Commits on Mar 10, 2026

  1. Add a test for self constructor promoted parameter

    - Introduce `DummyWithSelfConstructorPromotedParameter` test class to validate the normalization/denormalization.
    andersonamuller committed Mar 10, 2026
    Configuration menu
    Copy the full SHA
    6620ed4 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    2602f28 View commit details
    Browse the repository at this point in the history
  3. Fix merge

    nicolas-grekas committed Mar 10, 2026
    Configuration menu
    Copy the full SHA
    508be04 View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2026

  1. Configuration menu
    Copy the full SHA
    d155c56 View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2026

  1. [Serializer] Fix can*() prefix support in GetSetMethodNormalizer

    isGetMethod() recognizes can*() as a valid getter prefix, but
    isAllowedAttribute() and getAttributeValue() only support get*, is*
    and has*. This causes can*()-derived attributes to be silently dropped
    during normalization.
    sn3mdev committed Mar 26, 2026
    Configuration menu
    Copy the full SHA
    172f10a View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2026

  1. Configuration menu
    Copy the full SHA
    6df2763 View commit details
    Browse the repository at this point in the history
  2. Merge branch '6.4' into 7.4

    * 6.4:
      [Ldap] Make the Adapter resettable
      [Serializer] Remove needless line in changelog
      [MonologBridge] Fix ConsoleHandler losing output after nested command terminates
      [Cache] Fix tests
      [EventDispatcher] Fix memory leak in TraceableEventDispatcher for long-running processes
      [TwigBridge][Mime] Add missing tests
      [TwigBridge] Refactor image method to use DataPart content ID
      [Cache] Fix undefined property access
      [Console] Fix performance regression in OutputFormatter for ASCII content
      [Serializer] Fix can*() prefix support in GetSetMethodNormalizer
      [Cache] Fix Psr16Cache::getMultiple() returning ValueWrapper with TagAwareAdapter
    
    # Conflicts:
    #	src/Symfony/Bridge/Monolog/Handler/ConsoleHandler.php
    #	src/Symfony/Bridge/Monolog/Tests/Handler/ConsoleHandlerTest.php
    #	src/Symfony/Bridge/Twig/Mime/WrappedTemplatedEmail.php
    #	src/Symfony/Component/EventDispatcher/Debug/TraceableEventDispatcher.php
    #	src/Symfony/Component/Ldap/Tests/Adapter/ExtLdap/AdapterTest.php
    nicolas-grekas committed Mar 30, 2026
    Configuration menu
    Copy the full SHA
    1670285 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7c668b2 View commit details
    Browse the repository at this point in the history
  4. bug #63782 [Serializer] Fix mixed-typed constructor parameters overri…

    …ding getter-inferred type (pcescon)
    
    This PR was merged into the 7.4 branch.
    
    Discussion
    ----------
    
    [Serializer] Fix mixed-typed constructor parameters overriding getter-inferred type
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.4
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Issues        | Fix #63779
    | License       | MIT
    
    This PR fixes a regression introduced in #63401 (7.4.6) where `mixed`-typed constructor parameters incorrectly override the richer type inferred by the property type extractor.
    
    When a DTO has a constructor parameter typed `mixed` but a getter that returns a specific type (e.g., `?MyEntity`), the property type extractor correctly identifies the type from the getter. However, the condition introduced in #63401 was treating `mixed` as a meaningful type constraint and overriding the extractor's type with `Type::mixed()`.
    
    ```php
    class MyDTO
    {
        public function __construct(
            private mixed $entity = null,
        ) {}
    
        public function getEntity(): ?MyEntity
        {
            return $this->entity;
        }
    }
    ```
    
    With a custom denormalizer registered for `MyEntity`, `denormalize(['entity' => 42], MyDTO::class)` would **no longer invoke the custom denormalizer** in 7.4.6+. The raw integer `42` was passed to the constructor instead of a `MyEntity` instance.
    
    The fix treats `mixed` as carrying no useful type constraint (same as `null`) across all three code paths in `denormalizeParameter()`:
    
    - **`ReflectionTypeResolver` path**: exclude `TypeIdentifier::MIXED` in `isSatisfiedBy()`
    - **`ReflectionNamedType` fallback path**: skip override when `TypeIdentifier::MIXED`
    - **Legacy `getTypes()` BC path**: skip the whole override block when the parameter type is `mixed`
    
    Commits
    -------
    
    cbb9df00fdf [Serializer] Fix mixed-typed constructor parameters overriding getter-inferred type
    nicolas-grekas committed Mar 30, 2026
    Configuration menu
    Copy the full SHA
    714e9d2 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    9b8a7ba View commit details
    Browse the repository at this point in the history
  6. bug #63817 [Serializer] Fix denormalization of nested array with key …

    …types (mtarld)
    
    This PR was merged into the 6.4 branch.
    
    Discussion
    ----------
    
    [Serializer] Fix denormalization of nested array with key types
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 6.4
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Issues        | Fix #60036
    | License       | MIT
    
    The nested-array branch in `AbstractObjectNormalizer::validateAndDenormalize` did not propagate `key_type`/`value_type` into the context, unlike the non-nested branch. This caused the parent collection's `key_type` to leak into child properties.
    
    The fix propagates these context keys in `AbstractObjectNormalizer` and advances them to the next nesting level in `ArrayDenormalizer`.
    
    Commits
    -------
    
    15a12e0a6ea [Serializer] Fix denormalization of nested array with key types
    nicolas-grekas committed Mar 30, 2026
    Configuration menu
    Copy the full SHA
    90e4e01 View commit details
    Browse the repository at this point in the history
  7. Merge branch '6.4' into 7.4

    * 6.4:
      [DependencyInjection] Fix rejecting inline services in parameters section
      [Serializer] Fix denormalization of nested array with key types
      [VarDumper] Ensure that tests are resilient when the Xdebug file link format is defined
    nicolas-grekas committed Mar 30, 2026
    Configuration menu
    Copy the full SHA
    006fd51 View commit details
    Browse the repository at this point in the history
Loading