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
use short config syntax for factories #7213
Conversation
|
||
.. versionadded:: 3.2 | ||
The ``factory_service:method_name`` syntax for the YAML configuration format | ||
was introduced in Symfony 3.2. |
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.
It looks like this was available since the introduction of the new factory syntax in Symfony 2.6: https://github.com/symfony/symfony/blob/2.7/src/Symfony/Component/DependencyInjection/Loader/YamlFileLoader.php#L195-L198
@@ -106,6 +106,23 @@ configure the service container to use the | |||
the configured class name may be used by compiler passes and therefore | |||
should be set to a sensible value. | |||
|
|||
.. note:: | |||
|
|||
The traditional configurator syntax in YAML files used an array to define |
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.
configuration syntax
Thanks @snoek09! 👍 |
|
||
.. versionadded:: 2.6 | ||
The ``factory_service:method_name`` syntax for the YAML configuration format | ||
was introduced in Symfony 2.6. |
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.
The versionadded
directive is not needed here.
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.
👍
👍 |
Thank you @snoek09. |
This fixes #7208.