LiipImagineBundle icon indicating copy to clipboard operation
LiipImagineBundle copied to clipboard

Add aliases for manager services for improved autowiring support

Open mbabker opened this issue 1 year ago • 1 comments

Q A
Branch? 2.x
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets N/A
License MIT
Doc N/A

The liip_imagine.cache.manager, liip_imagine.data.manager, and liip_imagine.filter.manager services cannot be autowired at present, requiring folks to either manually create aliases in their app's configuration or to use one of the newer dependency injection attributes to specify the right service in their classes. This PR adds the appropriate service aliases to support autowiring and remove that little bit of extra overhead needed to use these three services.

mbabker avatar Jan 03 '24 14:01 mbabker

Coverage Status

coverage: 81.998%. remained the same when pulling 63d9dbab513018cbadef639ea67c13721d88eb35 on mbabker:autowiring-aliases into 1d42ed1edb41a64514a287c7ea6c4d895b0220dc on liip:2.x.

coveralls avatar Jan 03 '24 14:01 coveralls

can you please also update the documentation of the corresponding services?

Sorry, I forgot all about this ask with the way things have gone for me over the last several weeks. I'll try to throw a PR together for the docs soon.

mbabker avatar Feb 26 '24 14:02 mbabker

no worries. i also forgot and just noticed now. i adjusted the doc in 13be8f6810407a34241c1efe6ffcbded4f3efc39

i want to cleanup rather radically for 3.x, i think no further doc refinement for the 2.x is needed now.

and thanks for the contribution.

dbu avatar Feb 26 '24 14:02 dbu