Skip to content

[FrameworkBundle] Fix registering ExpressionValidator #49889

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

Merged
merged 1 commit into from
Apr 1, 2023

Conversation

nicolas-grekas
Copy link
Member

Q A
Branch? 6.2
Bug fix? yes
New feature? no
Deprecations? no
Tickets #49886
License MIT
Doc PR -

Introduced in #49850, my bad.

@NicoHaase
Copy link
Contributor

@nicolas-grekas would it be possible to write a test that covers this problem?

@nicolas-grekas
Copy link
Member Author

nicolas-grekas commented Apr 4, 2023

It'd be great to have a test case that ensures that what is returned by validatedBy is properly found as an alias when registering the service. Up for a PR?

@NicoHaase
Copy link
Contributor

It'd be great to have a test case that ensures that what is returned by validatedBy is properly found as an alias when registering the service. Up for a PR?

To be honest: If I had a good idea about such a test case, I would have provided one 🙃

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

Successfully merging this pull request may close these issues.

3 participants