chh/cache-service-provider

A Cache Service Provider for Silex, using the doctrine/cache package

cache-service-provider's Issues

Remove caches service?

Currently all individual caches are services in a separate Pimple container, which is set as the caches service of the application.

This makes it easy to pass the complete collection of defined caches to something, but at the same time makes it a bit more work to define or extend just a single cache at a time, because you have to extend the caches service first.

The question is: Is anyone using the caches service and would it be acceptable to remove this service and to setup caches as services of the application's container (e.g. cache.default)?

This change would also bring it more in line with other service providers, which create services on the application's container, e.g. Doctrine.

Move factories to be plain protected closures which return a cache

Currently the cache.factory and the cache.namespace services are protected closures, which return closures, which then return a cache instance. Change these services to be just protected closures, which return a cache instance to reduce complexity and bring it in line with conventions.

Before:

$app['caches'] = $app->extend('caches', function ($caches) use ($app) {
    $caches['foo'] = $app['cache.namespace']('foobar');
    return $caches;
});

After:

$app['caches'] = $app->extend('caches', function ($caches) use ($app) {
    $caches['foo'] = function () use ($app) {
        return $app['cache.namespace']('foobar');
    };
    return $caches;
});

Change namespace to be PSR-4 compliant

Tasks:

• Change the namespace to something where the CacheNamespace and the CacheServiceProvider class are in the same namespace (BC break)
• Use PSR-4 autoloader in composer.json

Add factory for cache namespace?

Fix Readme example about cache.factory

Update require to support Silex 1.1

Silex 1.1 stable released http://silex.sensiolabs.org/doc/changelog.html

Current requirement in this provider is 1.0.*@dev so doesn't let to update to it.