[DependencyInjection] [Autowiring] Allow undefined nullable parameters (6.3) #50023
Conversation
|
Grr... Tried renaming local patch to https://github.com/radar3301/symfony/tree/autowire-undefined-nullable-6.3 |
|
Thanks for the PRs. Usually we ask for the patch on the lowest affected branch, and we merge up periodically. When the merge generates conflicts, we fix the conflict and we sometimes ask the author for their help if it's non trivial. I'm therefor closing this PR but it'll definitely help when merging #50022, if accepted! |
|
Thanks for the feedback Nicolas. The Autowire Attribute was added in 6.1, and handling of the Autowire attribute was added directly to It's all a big mess, hence the 3 different patches I created. Hope this brief overview helps when then time comes! |
…iring undefined parameters for optional arguments (radar3301) This PR was merged into the 6.2 branch. Discussion ---------- [DependencyInjection] Fallback to default value when autowiring undefined parameters for optional arguments | Q | A | ------------- | --- | Branch? | 6.2 / 6.3 | Bug fix? | yes | New feature? | yes <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | Fix #50020 <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead --> | License | MIT | Related | PR #50023 Allows autowiring of undefined nullable parameters, similar to existing behavior for non-parameter values. See related issue for more detail. Commits ------- 2d9bc8c [DependencyInjection] Fallback to default value when autowiring undefined parameters for optional arguments
Allows autowiring of undefined nullable parameters, similar to existing behavior for non-parameter values. See related issue for more detail.