-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[Validator] Remove exclusion of Constraints/Validator/ExpressionLanguageSyntaxValidator #61166
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
[Validator] Remove exclusion of Constraints/Validator/ExpressionLanguageSyntaxValidator #61166
Conversation
Can you please explain a bit more what happens without this change? What error do you get? |
I'm also wondering about how you experienced this @corradogrimoldi |
@nicolas-grekas I'm using framework-bundle 7.3 and running composer require symfony/validator I got the 7.3 version and there ExpressionLanguageSyntaxValidator is not present.
the exception is catched and due to the fact that $ignoreErrors is set to true, nothing happens to the app |
So there's no bug, this is normal behavior and everything works, isn't it? |
…ageSyntaxValidator
2fdfa85
to
5e61506
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.
For 8.0 as explained in #61166 (comment)
Thank you @corradogrimoldi. |
Correct,
my fault, I was debugging with break on all exceptions and didn't realize that the code get rid of this exception until your comment.
Tnx
…________________________________
From: Nicolas Grekas ***@***.***>
Sent: Monday, July 21, 2025 9:40 AM
To: symfony/symfony ***@***.***>
Cc: Grimoldi, Corrado ***@***.***>; Mention ***@***.***>
Subject: Re: [symfony/symfony] [Validator] Remove exclusion of Constraints/Validator/ExpressionLanguageSyntaxValidator (PR #61166)
This mail has been sent from an external source. Do not reply to it, or open any links/attachments unless you are sure of the sender's identity.
[https://avatars.githubusercontent.com/u/243674?s=20&v=4]nicolas-grekas left a comment (symfony/symfony#61166)<#61166 (comment)>
So there's no bug, this is normal behavior and everything works, isn't it?
—
Reply to this email directly, view it on GitHub<#61166 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BDJWH3MUXDUHNU65ESB3HPL3JSKQHAVCNFSM6AAAAACB557K4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTAOJVGU4DONZWHA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.
|
🛠️ Validation Component removed the ExpressionLanguageSyntaxValidator so it is no longer loaded in the service container.
The method used to exclude the service stands on the fact that the class file exists, this cause an exception