Problem/Motivation

Method "Symfony\Component\Validator\ConstraintValidatorFactoryInterface::getInstance()" will return "ConstraintValidatorInterface" as of its next major version. Doing the same in implementation "Drupal\Core\Validation\ConstraintValidatorFactory" will be required when upgrading.

Steps to reproduce

Proposed resolution

Add the "ConstraintValidatorInterface" return type hint.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3233466-2.patch1.13 KBdaffie
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
1.13 KB

I could not find any instances where the method was overridden in contrib. See: http://grep.xnddx.ru/search?text=ConstraintValidatorFactory&filename=.

larowlan’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 0ffa3f0 and pushed to 9.3.x. Thanks!

As per @catch There's a CR here: https://www.drupal.org/node/3236232

  • alexpott committed 0ffa3f0 on 9.3.x
    Issue #3233466 by daffie: [Symfony 6] Add "ConstraintValidatorInterface...

Status: Fixed » Closed (fixed)

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