-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[Validator] Add missing HasNamedArguments to some constraints #60840
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
AtLeastOneOf has the same issue? |
HasNamedArguments to fix symfony 7.4 deprecationHasNamedArguments to fix symfony 7.4 deprecation
Yes. I add it to this PR |
HasNamedArguments to fix symfony 7.4 deprecation6325699 to
8e77793
Compare
nicolas-grekas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(I rebased for 7.3)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do Composite, Compound, GroupSequence and Sequentially also need the same update?
Edit: And also the UserPassword constraint from the Security component.
8e77793 to
cce8eac
Compare
|
Thank you @jkgroupe. |
Add missing
HasNamedArgumentsattribute on__construct(...)to fix symfony 7.4 deprecation