Skip to content

Conversation

@eerison
Copy link
Contributor

@eerison eerison commented Dec 2, 2025

Q A
Branch? the branch of the current release for fixes
Bug fix? no
New feature? no
BC breaks? maybe yes/no?
Deprecations? -
Fixed tickets none
License MIT
Doc PR none

error message symfony 7.4/8.0 changed the Symfony\Component\Validator\ConstraintValidatorInterface:validate

PHP Fatal error:  Declaration of
Symfony\Cmf\Bundle\RoutingBundle\Validator\Constraints\RouteDefaultsTwigValidator::validate($defaults, Symfony\Component\Validator\Constraint $constraint)
 must be compatible with
Symfony\Component\Validator\ConstraintValidatorInterface::validate(mixed $value, Symfony\Component\Validator\Constraint $constraint): void in /home/erison/project
s/SonataPageBundle/vendor/symfony-cmf/routing-bundle/src/Validator/Constraints/RouteDefaultsTwigValidator.php on line 34

@eerison eerison marked this pull request as ready for review December 2, 2025 13:25
@dbu
Copy link
Member

dbu commented Dec 2, 2025

can you please rebase on 3.x? i fixed the build in the 3.x branch.

PHP Fatal error:  Declaration of
Symfony\Cmf\Bundle\RoutingBundle\Validator\Constraints\RouteDefaultsTwigValidator::validate($defaults,
Symfony\Component\Validator\Constraint $constraint)
 must be compatible with
Symfony\Component\Validator\ConstraintValidatorInterface::validate(mixed
$value, Symfony\Component\Validator\Constraint $constraint): void in
/home/erison/project
s/SonataPageBundle/vendor/symfony-cmf/routing-bundle/src/Validator/Constraints/RouteDefaultsTwigValidator.php
on line 34
@eerison eerison force-pushed the validator-interface branch from eb465e6 to 81f952e Compare December 2, 2025 17:15
@eerison
Copy link
Contributor Author

eerison commented Dec 2, 2025

I just run phpstan locally, and it looks fine now :)

Copy link
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks a lot!

@dbu dbu merged commit cfc6d19 into symfony-cmf:3.x Dec 3, 2025
8 checks passed
@dbu
Copy link
Member

dbu commented Dec 3, 2025

i tried to also provide symfony 8 support in #500, but that is more involved. afaik we also need to rewrite the service configuration from xml to php (there is a tool to at least automate it some). if you feel up to it, i am glad for contributions.

meanwhile, i tagged https://github.com/symfony-cmf/routing-bundle/releases/tag/3.1.2 with symfony 7.4 no deprecations support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants