Skip to content

Commit 714b916

Browse files
nicolas-grekasnicolas-grekas
committed
[FrameworkBundle] Add & use Psr6CacheClearer
1 parent 4740c5c commit 714b916

File tree

20 files changed

+228
-52
lines changed

20 files changed

+228
-52
lines changed

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/CachePoolPass.php‎

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -27,36 +27,47 @@ class CachePoolPass implements CompilerPassInterface
2727
public function process(ContainerBuilder $container)
2828
{
2929
$attributes = array(
30-
'provider_service',
30+
'provider',
3131
'namespace',
3232
'default_lifetime',
33-
'directory',
3433
);
3534
foreach ($container->findTaggedServiceIds('cache.pool') as $id => $tags) {
3635
$adapter = $pool = $container->getDefinition($id);
37-
$tags[0] += array('namespace' => $this->getNamespace($id));
38-
36+
if ($pool->isAbstract()) {
37+
continue;
38+
}
39+
if (!isset($tags[0]['namespace'])) {
40+
$tags[0]['namespace'] = $this->getNamespace($id);
41+
}
3942
while ($adapter instanceof DefinitionDecorator) {
4043
$adapter = $container->findDefinition($adapter->getParent());
4144
if ($t = $adapter->getTag('cache.pool')) {
4245
$tags[0] += $t[0];
4346
}
4447
}
45-
if ($pool->isAbstract()) {
46-
continue;
48+
if (isset($tags[0]['clearer'])) {
49+
$clearer = $container->getDefinition($tags[0]['clearer']);
50+
} else {
51+
$clearer = null;
4752
}
48-
if (isset($tags[0]['provider_service']) && is_string($tags[0]['provider_service'])) {
49-
$tags[0]['provider_service'] = new Reference($tags[0]['provider_service']);
53+
unset($tags[0]['clearer']);
54+
55+
if (isset($tags[0]['provider']) && is_string($tags[0]['provider'])) {
56+
$tags[0]['provider'] = new Reference($tags[0]['provider']);
5057
}
5158
$i = 0;
5259
foreach ($attributes as $attr) {
5360
if (isset($tags[0][$attr])) {
5461
$pool->replaceArgument($i++, $tags[0][$attr]);
55-
unset($tags[0][$attr]);
5662
}
63+
unset($tags[0][$attr]);
5764
}
5865
if (!empty($tags[0])) {
59-
throw new \InvalidArgumentException(sprintf('Invalid "cache.pool" tag for service "%s": accepted attributes are "provider_service", "namespace", "default_lifetime" and "directory", found "%s".', $id, implode('", "', array_keys($tags[0]))));
66+
throw new \InvalidArgumentException(sprintf('Invalid "cache.pool" tag for service "%s": accepted attributes are "provider", "namespace" and "default_lifetime", found "%s".', $id, implode('", "', array_keys($tags[0]))));
67+
}
68+
69+
if (null !== $clearer) {
70+
$clearer->addMethodCall('addPool', array(new Reference($id)));
6071
}
6172
}
6273
}

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php‎

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -561,14 +561,19 @@ private function addCacheSection(ArrayNodeDefinition $rootNode)
561561
->useAttributeAsKey('name')
562562
->prototype('array')
563563
->children()
564-
->scalarNode('adapter_service')
565-
->info('The cache pool service to use as template definition.')
564+
->scalarNode('adapter')
565+
->info('The cache pool adapter service to use as template definition.')
566566
->defaultValue('cache.adapter.default')
567567
->end()
568568
->booleanNode('public')->defaultFalse()->end()
569-
->integerNode('default_lifetime')->defaultNull()->end()
570-
->scalarNode('provider_service')->defaultNull()->end()
571-
->scalarNode('directory')->defaultNull()->end()
569+
->integerNode('default_lifetime')->end()
570+
->scalarNode('provider')
571+
->info('The service name to use as provider when the specified adapter needs one.')
572+
->end()
573+
->scalarNode('namespace')
574+
->info('The namespace where cached items are stored. Auto-generated by default. Set to false to disable namespacing.')
575+
->end()
576+
->scalarNode('clearer')->defaultValue('cache.default_pools_clearer')->end()
572577
->end()
573578
->end()
574579
->end()

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1027,9 +1027,9 @@ private function registerCacheConfiguration(array $config, ContainerBuilder $con
10271027
}
10281028

10291029
foreach ($config['pools'] as $name => $poolConfig) {
1030-
$poolDefinition = new DefinitionDecorator($poolConfig['adapter_service']);
1030+
$poolDefinition = new DefinitionDecorator($poolConfig['adapter']);
10311031
$poolDefinition->setPublic($poolConfig['public']);
1032-
unset($poolConfig['adapter_service'], $poolConfig['public']);
1032+
unset($poolConfig['adapter'], $poolConfig['public']);
10331033

10341034
$poolDefinition->addTag('cache.pool', $poolConfig);
10351035
$container->setDefinition('cache.pool.'.$name, $poolDefinition);

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/cache_pools.xml‎

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,33 +6,39 @@
66

77
<services>
88

9+
<service id="cache.default_pools_clearer" class="Symfony\Component\HttpKernel\CacheClearer\Psr6CacheClearer" public="false">
10+
<tag name="kernel.cache_clearer" />
11+
</service>
12+
913
<service id="cache.adapter.default" alias="cache.adapter.filesystem" />
1014

1115
<service id="cache.adapter.apcu" class="Symfony\Component\Cache\Adapter\ApcuAdapter" abstract="true">
12-
<tag name="cache.pool" />
1316
<argument /> <!-- namespace -->
1417
<argument /> <!-- default lifetime -->
1518
</service>
1619

1720
<service id="cache.adapter.doctrine" class="Symfony\Component\Cache\Adapter\DoctrineAdapter" abstract="true">
18-
<tag name="cache.pool" />
1921
<argument /> <!-- Doctrine provider service -->
2022
<argument /> <!-- namespace -->
2123
<argument /> <!-- default lifetime -->
2224
</service>
2325

26+
<service id="cache.adapter.filesystem" class="Symfony\Component\Cache\Adapter\FilesystemAdapter" abstract="true">
27+
<argument /> <!-- namespace -->
28+
<argument /> <!-- default lifetime -->
29+
<argument>%kernel.cache_dir%/pools</argument>
30+
</service>
31+
2432
<service id="cache.adapter.psr6" class="Symfony\Component\Cache\Adapter\ProxyAdapter" abstract="true">
25-
<tag name="cache.pool" />
2633
<argument /> <!-- PSR-6 provider service -->
2734
<argument /> <!-- namespace -->
2835
<argument /> <!-- default lifetime -->
2936
</service>
3037

31-
<service id="cache.adapter.filesystem" class="Symfony\Component\Cache\Adapter\FilesystemAdapter" abstract="true">
32-
<tag name="cache.pool" />
38+
<service id="cache.adapter.redis" class="Symfony\Component\Cache\Adapter\RedisAdapter" abstract="true">
39+
<argument /> <!-- Redis connection object -->
3340
<argument /> <!-- namespace -->
3441
<argument /> <!-- default lifetime -->
35-
<argument>%kernel.cache_dir%</argument>
3642
</service>
3743

3844
</services>

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/schema/symfony-1.0.xsd‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,10 +212,11 @@
212212

213213
<xsd:complexType name="cache_pool">
214214
<xsd:attribute name="name" type="xsd:string" use="required" />
215-
<xsd:attribute name="adapter-service" type="xsd:string" />
215+
<xsd:attribute name="adapter" type="xsd:string" />
216216
<xsd:attribute name="public" type="xsd:boolean" />
217217
<xsd:attribute name="default-lifetime" type="xsd:integer" />
218-
<xsd:attribute name="provider-service" type="xsd:string" />
218+
<xsd:attribute name="provider" type="xsd:string" />
219219
<xsd:attribute name="directory" type="xsd:string" />
220+
<xsd:attribute name="clearer" type="xsd:string" />
220221
</xsd:complexType>
221222
</xsd:schema>

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/CachePoolPassTest.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public function testArgsAreReplaced()
4949
$container = new ContainerBuilder();
5050
$cachePool = new Definition();
5151
$cachePool->addTag('cache.pool', array(
52-
'provider_service' => 'foobar',
52+
'provider' => 'foobar',
5353
'default_lifetime' => 3,
5454
));
5555
$cachePool->addArgument(null);

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/php/cache.php‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,22 @@
44
'cache' => array(
55
'pools' => array(
66
'foo' => array(
7-
'adapter_service' => 'cache.adapter.apcu',
7+
'adapter' => 'cache.adapter.apcu',
88
'default_lifetime' => 30,
99
),
1010
'bar' => array(
11-
'adapter_service' => 'cache.adapter.doctrine',
11+
'adapter' => 'cache.adapter.doctrine',
1212
'default_lifetime' => 5,
13-
'provider_service' => 'app.doctrine_cache_provider',
13+
'provider' => 'app.doctrine_cache_provider',
1414
),
1515
'baz' => array(
16-
'adapter_service' => 'cache.adapter.filesystem',
16+
'adapter' => 'cache.adapter.filesystem',
1717
'default_lifetime' => 7,
18-
'directory' => 'app/cache/psr',
1918
),
2019
'foobar' => array(
21-
'adapter_service' => 'cache.adapter.psr6',
20+
'adapter' => 'cache.adapter.psr6',
2221
'default_lifetime' => 10,
23-
'provider_service' => 'app.cache_pool',
22+
'provider' => 'app.cache_pool',
2423
),
2524
'def' => array(
2625
'default_lifetime' => 11,

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/cache.xml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77

88
<framework:config>
99
<framework:cache>
10-
<framework:pool name="foo" adapter-service="cache.adapter.apcu" default-lifetime="30" />
11-
<framework:pool name="bar" adapter-service="cache.adapter.doctrine" default-lifetime="5" provider-service="app.doctrine_cache_provider" />
12-
<framework:pool name="baz" adapter-service="cache.adapter.filesystem" default-lifetime="7" directory="app/cache/psr" />
13-
<framework:pool name="foobar" adapter-service="cache.adapter.psr6" default-lifetime="10" provider-service="app.cache_pool" />
10+
<framework:pool name="foo" adapter="cache.adapter.apcu" default-lifetime="30" />
11+
<framework:pool name="bar" adapter="cache.adapter.doctrine" default-lifetime="5" provider="app.doctrine_cache_provider" />
12+
<framework:pool name="baz" adapter="cache.adapter.filesystem" default-lifetime="7" />
13+
<framework:pool name="foobar" adapter="cache.adapter.psr6" default-lifetime="10" provider="app.cache_pool" />
1414
<framework:pool name="def" default-lifetime="11" />
1515
</framework:cache>
1616
</framework:config>

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/yml/cache.yml‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,19 +2,18 @@ framework:
22
cache:
33
pools:
44
foo:
5-
adapter_service: cache.adapter.apcu
5+
adapter: cache.adapter.apcu
66
default_lifetime: 30
77
bar:
8-
adapter_service: cache.adapter.doctrine
8+
adapter: cache.adapter.doctrine
99
default_lifetime: 5
10-
provider_service: app.doctrine_cache_provider
10+
provider: app.doctrine_cache_provider
1111
baz:
12-
adapter_service: cache.adapter.filesystem
12+
adapter: cache.adapter.filesystem
1313
default_lifetime: 7
14-
directory: app/cache/psr
1514
foobar:
16-
adapter_service: cache.adapter.psr6
15+
adapter: cache.adapter.psr6
1716
default_lifetime: 10
18-
provider_service: app.cache_pool
17+
provider: app.cache_pool
1918
def:
2019
default_lifetime: 11

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -683,7 +683,6 @@ private function assertCachePoolServiceDefinitionIsCreated(ContainerBuilder $con
683683
break;
684684
}
685685

686-
$this->assertTrue($adapterDefinition->hasTag('cache.pool'), sprintf('Service definition "%s" is tagged with the "cache.pool" tag.', $adapterId));
687686
$this->assertTrue($adapterDefinition->isAbstract(), sprintf('Service definition "%s" is abstract.', $adapterId));
688687
}
689688
}

0 commit comments

Comments
 (0)