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

[Dependency Injection] Add alias for cache manager #1022

Merged
merged 1 commit into from
Dec 12, 2017
Merged

Conversation

garak
Copy link
Contributor

@garak garak commented Dec 8, 2017

Q A
Branch? 2.0
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets none
License MIT
Doc PR none

Without proposed alias, this bundle is not working on Symfony 4.0, and logging a deprecation notice on Symfony 3.4

Autowiring services based on the types they implement is deprecated since Symfony 3.3 and won't be supported in version 4.0. You should rename (or alias) the "liip_imagine.cache.manager" service to "Liip\ImagineBundle\Imagine\Cache\CacheManager" instead.

Without proposed alias, this bundle is not working on Symfony 4.0, and logging a deprecation notice on Symfony 3.4
>Autowiring services based on the types they implement is deprecated since Symfony 3.3 and won't be supported in version 4.0. You should rename (or alias) the "liip_imagine.cache.manager" service to "Liip\ImagineBundle\Imagine\Cache\CacheManager" instead.
@lsmith77
Copy link
Contributor

can you rebase please.

@garak
Copy link
Contributor Author

garak commented Dec 10, 2017

Rebase which branch?

@lsmith77
Copy link
Contributor

ah sorry .. I guess https://github.com/liip/LiipImagineBundle/pull/1023/files needs to be merged and then rebase against 2.0

@garak
Copy link
Contributor Author

garak commented Dec 12, 2017

What if you merge this PR and then rebase 1023?

@lsmith77 lsmith77 merged commit 28a9248 into liip:2.0 Dec 12, 2017
@garak garak deleted the patch-1 branch December 12, 2017 19:12
@robfrawley robfrawley changed the title Add needed alias [Dependency Injection] Add alias for cache manager Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants