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

CommentFileSizeAuthor
#5 3231682.patch2.12 KBlarowlan
#2 3231682-2.patch2.12 KBdaffie

Comments

daffie created an issue. See original summary.

daffie’s picture

Status: Active » Needs review
StatusFileSize
new2.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

StatusFileSize
new2.12 KB

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.