Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
[Validator] Removal code-block php-annotation#17468
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
[Validator] Removal code-block php-annotation#17468
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Thank you Maxime. |
| @@ -38,54 +38,27 @@ use it as an annotation/attribute in other classes. | |||
| The ``#[HasNamedArguments]`` attribute was introduced in Symfony 6.1. | |||
| You can use ``#[HasNamedArguments]`` or ``getRequiredOptions()`` to make some constraint options required: | |||
| You can use ``#[HasNamedArguments]`` or ``getRequiredOptions()`` to make some constraint options required:: | |||
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.
not sure if we really should mention this method any longer if we do not have an example for it anymore
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 agree with you that this method is no longer used. Thanks
I have created a PR#17476
…the example (mdoutreluingne)This PR was merged into the 6.1 branch.Discussion----------[Validator] `getRequiredOptions()` is no longer used in the exampleFollowing#17468I forgot to delete the method `getRequiredOptions()` as it is no longer used in the example and I have also removed the ``@Annotation`` because it was also no longer used in the example.Commits-------858cabc [Validator] Remove annotation
Following#17223