Skip to content
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

[All] Autowiring #850

Merged
merged 3 commits into from
Jun 6, 2019
Merged

[All] Autowiring #850

merged 3 commits into from
Jun 6, 2019

Conversation

dpfaffenbauer
Copy link
Member

@dpfaffenbauer dpfaffenbauer commented Mar 4, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no

make more use of auto-wiring in symfony 3.4. This will automatically apply tags to classes implementing the needed interface for all kinds of Rule Conditions/Actions, Price Calculators, Index/Filters, Contextes. This also allows auto-wiring of coreshop generated services like repositories and manager by using the name of it. Like productRepositoryorstoreRepository`.

If a tag is not supplied, but the compiler-pass expects a type, the type is getting auto-generated by the class-name.

@dpfaffenbauer dpfaffenbauer added this to the 2.0.5 milestone Mar 4, 2019
@dpfaffenbauer dpfaffenbauer self-assigned this Mar 4, 2019
@dpfaffenbauer dpfaffenbauer requested a review from a team March 4, 2019 08:19
@dpfaffenbauer dpfaffenbauer force-pushed the autowiring branch 3 times, most recently from 178ad6a to b3556a4 Compare March 12, 2019 17:11
@dpfaffenbauer dpfaffenbauer modified the milestones: 2.0.5, 2.0.6 Mar 17, 2019
@dpfaffenbauer dpfaffenbauer modified the milestones: 2.0.6, 2.0.7 Apr 8, 2019
@dpfaffenbauer dpfaffenbauer modified the milestones: 2.0.7, 2.1.0 May 6, 2019
@dpfaffenbauer dpfaffenbauer changed the base branch from 2.0 to master May 6, 2019 13:38
@dpfaffenbauer
Copy link
Member Author

rebased to master -> will be released with 2.1

…cally apply tags to classes implementing the needed interface for all kinds of Rule Conditions/Actions, Price Calculators, Index/Filters, Contextes. This also allows auto-wiring of coreshop generated services like repositories and manager by using the name of it. Like productRepository` or `storeRepository`
@dpfaffenbauer dpfaffenbauer merged commit 1f22a63 into coreshop:master Jun 6, 2019
@dpfaffenbauer dpfaffenbauer deleted the autowiring branch June 6, 2019 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant