Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
minor #32305 [HttpKernel][DX] Improve the error message when not defi…
…ning the controller as a service but using construct parameters (Simperfit) This PR was merged into the 4.4 branch. Discussion ---------- [HttpKernel][DX] Improve the error message when not defining the controller as a service but using construct parameters | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | none <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | not needed <!-- required for new features --> <!-- Replace this notice by a short README for your feature/bugfix. This will help people understand your PR and can be used as a start for the documentation. Additionally (see https://symfony.com/roadmap): - Bug fixes must be submitted against the lowest maintained branch where they apply (lowest branches are regularly merged to upper ones so they get the fixes too). - Features and deprecations must be submitted against branch 4.4. - Legacy code removals go to the master branch. --> I'm working with newcomers to the symfony framework ATM and one of them has got stuck in this errors trying to understand why the constructor argument declared as a a service would not work and didn't check if the controller was declared as a service itself. I feel like this could be improve to be really specific since "such" is not that specific ;). Commits ------- c670d51 [HttpKernel][DX] Improve the error message when not defining the controller as a service but using contruct parameters
- Loading branch information