Skip to content

[Form][FrameworkBundle][HttpKernel] param validator.translation_domain param not available #58833

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

Closed
kbond opened this issue Nov 12, 2024 · 1 comment · Fixed by #58834
Closed
Labels
Bug Form FrameworkBundle HttpKernel ❄️ Feature Freeze Important Pull Requests to finish before the next Symfony "feature freeze" Status: Needs Review

Comments

@kbond
Copy link
Member

kbond commented Nov 12, 2024

Symfony version(s) affected

7.2.0

Description

If symfony/form is not available, you get the following exception:

Symfony\Component\DependencyInjection\Exception\RuntimeException: You have requested a non-existent parameter "validator.translation_domain".

I believe #58187 caused this bug.

How to reproduce

Install a Symfony 7.2 app w/o symfony/validator or symfony/form.

Possible Solution

No response

Additional Context

No response

@xabbuh
Copy link
Member

xabbuh commented Nov 12, 2024

Can you confirm that #58834 fixes this?

nicolas-grekas added a commit that referenced this issue Nov 13, 2024
…rameter is always set (xabbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

[FrameworkBundle] ensure `validator.translation_domain` parameter is always set

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #58833
| License       | MIT

Commits
-------

67d0854 ensure that the validator.translation_domain parameter is always set
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Form FrameworkBundle HttpKernel ❄️ Feature Freeze Important Pull Requests to finish before the next Symfony "feature freeze" Status: Needs Review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants