Problem/Motivation

Method "Symfony\Component\Validator\Validator\ValidatorInterface::validate()" will return "ConstraintViolationListInterface" as of its next major version. Doing the same in implementation "Drupal\Core\TypedData\Validation\RecursiveValidator" will be required when upgrading.

Method "Symfony\Component\Validator\Validator\ValidatorInterface::validateProperty()" will return "ConstraintViolationListInterface" as of its next major version. Doing the same in implementation "Drupal\Core\TypedData\Validation\RecursiveValidator" will be required when upgrading.

Method "Symfony\Component\Validator\Validator\ValidatorInterface::validatePropertyValue()" will return "ConstraintViolationListInterface" as of its next major version. Doing the same in implementation "Drupal\Core\TypedData\Validation\RecursiveValidator" will be required when upgrading.

Steps to reproduce

Proposed resolution

Add the "ConstraintViolationListInterface" type hint.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

daffie created an issue. See original summary.

daffie’s picture

Status: Active » Needs review
FileSize
2.12 KB

The fix.

daffie’s picture

Priority: Normal » Critical
Issue tags: +Drupal 10

I cannot find any instances in contrib were the class Drupal\Core\TypedData\Validation\RecursiveValidator is overridden or used in such a way that creates a BC break.

Part of the Symfony 6 in D10 initiative.

longwave’s picture

Status: Needs review » Reviewed & tested by the community

Looks OK to me.

larowlan’s picture

Re-roll, patch above didn't apply anymore

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 9d36d02 and pushed to 9.3.x. Thanks!

  • alexpott committed 9d36d02 on 9.3.x
    Issue #3231682 by daffie, larowlan: [Symfony 6] Add "...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.