-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[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
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
HasNamedArguments
attribute on__construct(...)
to fix symfony 7.4 deprecation