Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

[ZF3] Remove StorageFactory and PatternFactory #175

Open
boesing opened this issue Sep 8, 2018 · 2 comments
Open

[ZF3] Remove StorageFactory and PatternFactory #175

boesing opened this issue Sep 8, 2018 · 2 comments
Milestone

Comments

@boesing
Copy link
Member

boesing commented Sep 8, 2018

Hey there,

I would like to drop those abstract factories in favor of using the related PluginManagers directly.
Since #93 introduces external packages, it will get more and more difficult to attach plugins/storage adapters to the PluginManagers, if not retrieved by the projects ContainerInterface.

Imho, each external package should provide their own factories for their plugins/storage adapters and just inject these to the related PluginManager (e.g. by using delegators? Some feedback would be great).

The original idea of @marc-mabe was to use the Module object to add aliases and factories to the PluginManager but as of zend-config-aggregator or in projects without zend-mvc, this wont work anymore.

I would appreciate to get some feedback on this.
@weierophinney

@Ocramius
Copy link
Member

The idea makes sense, but removal is a hard BC break - can you maybe send a PR with deprecations for the classes you are targeting?

@weierophinney
Copy link
Member

This repository has been closed and moved to laminas/laminas-cache; a new issue has been opened at laminas/laminas-cache#4.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants