8000 Document the new short syntax for factory declarations · Issue #7442 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content
8000

Document the new short syntax for factory declarations #7442

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.

Sign up for GitHub

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

Jump to bottom
Closed
javiereguiluz opened this issue Feb 1, 2017 · 1 comment
Closed

Document the new short syntax for factory declarations #7442

javiereguiluz opened this issue Feb 1, 2017 · 1 comment
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Milestone
3.3

Comments

@javiereguiluz
Copy link
Member
javiereguiluz commented Feb 1, 2017 •
edited
Loading

See symfony/symfony#20943

Before:

<service id="app.newsletter_manager" class="AppBundle\Email\NewsletterManager">
    <factory class="AppBundle\Email\NewsletterManager" method="create" />
</service>

After:

<service id="app.newsletter_manager" class="AppBundle\Email\NewsletterManager">
    <factory method="create" />
</service>
@javiereguiluz javiereguiluz added this to the 3.3 milestone Feb 1, 2017
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Feb 6, 2018
@javiereguiluz javiereguiluz mentioned this issue Feb 6, 2018
Merged
javiereguiluz added a commit that referenced this issue Feb 7, 2018
@javiereguiluz
minor #9214 Mentioned the short syntax for XML config of service fact…
4485c9c 
…ories (javiereguiluz)

This PR was merged into the 3.4 branch.

Discussion
----------

Mentioned the short syntax for XML config of service factories

This fixes #7442.

Commits
-------

eafe44a Mentioned the short syntax for XML config of service factories
@javiereguiluz
Copy link
Member Author

Fixed by #9214.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
3.3
Development

No branches or pull requests
1 participant
@javiereguiluz
0