Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 15, 2018. It is now read-only.

Add support for method autowiring #64

Merged
merged 2 commits into from
Jan 10, 2017
Merged

Add support for method autowiring #64

merged 2 commits into from
Jan 10, 2017

Conversation

dunglas
Copy link
Owner

@dunglas dunglas commented Dec 9, 2016 •
edited
Loading

@dunglas dunglas force-pushed the setter_autowiring branch from 55b626f to 4d6652c Compare January 9, 2017 20:24
@dunglas dunglas changed the title [WIP] Support setter autowiring Add support for method autowiring Jan 9, 2017
@dunglas dunglas force-pushed the setter_autowiring branch 3 times, most recently from a678b31 to 390271f Compare January 10, 2017 10:22
@dunglas dunglas merged commit c587ba5 into master Jan 10, 2017
@dunglas dunglas deleted the setter_autowiring branch January 10, 2017 12:35
GuilhemN
GuilhemN reviewed Jan 10, 2017
View reviewed changes
DependencyInjection/Configuration.php
$treeBuilder->root('dunglas_action')
->fixXmlConfig('directory', 'directories')
->children()
->arrayNode('methods')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about autowired_methods? it looks not very clear to me this way.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@GuilhemN GuilhemN GuilhemN left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dunglas @GuilhemN