Open rieschl opened 1 month ago
I think either the dependency should be added to the
require
section and notrequire-dev
or the factory should use the newStorageAdapterFactoryInterface
(?) instead of the deprecated one.
The problem is the soft dependency on laminas-cache. If laminas-cache is not needed then the deprecated factory is also not needed. This will be solved with the next major version because the soft dependencies are crap. And caching in general should not be integrated in the translator or in a paginator.
Bug Report
Summary
We cleaned up our composer dependencies and removed the
laminas/laminas-cache-storage-deprecated-factory
because there weren't any usages in our code. After that, the factory of the Translator exploded because it tries to instantiate the cache for the translator with theStorageFactory
from said package.Current behavior
The application explodes because
\Laminas\Cache\StorageFactory
cannot be found.How to reproduce
Add
to the config and try to pull
\Laminas\I18n\Translator\Translator
from the container or build it withTranslator::factory($options)
where$options
is the above array inside thetranslator
key.Expected behavior
No boom π I think either the dependency should be added to the
require
section and notrequire-dev
or the factory should use the newStorageAdapterFactoryInterface
(?) instead of the deprecated one.