Skip to content

Commit 7b72bd0

Browse files
committed
Merge pull request #16 from Nyholm/patch-31
Minor fixes
2 parents c4086fd + fec03a9 commit 7b72bd0

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

README.md

+1
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ cache:
115115
router:
116116
enabled: true
117117
service_id: cache.provider.acme_redis_cache
118+
ttl: 86400
118119
```
119120

120121
If you change any of your routes, you will need to clear the cache. If you use a cache implementation that supports

src/DependencyInjection/CacheExtension.php

+3-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,9 @@ public function load(array $configs, ContainerBuilder $container)
4343
}
4444

4545
if ($config['router']['enabled']) {
46-
$container->getDefinition('cache.router_listener')->replaceArgument(0, new Reference($config['router']['service_id']));
46+
$container->getDefinition('cache.router_listener')
47+
->replaceArgument(0, new Reference($config['router']['service_id']))
48+
->replaceArgument(1, $config['router']['ttl']);
4749
} else {
4850
$container->removeDefinition('cache.router_listener');
4951
}

src/Resources/config/services.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@ services:
22
data_collector.cache:
33
class: Cache\CacheBundle\DataCollector\CacheDataCollector
44
tags:
5-
- { name: data_collectorr, template: 'CacheBundle:Collector:cache.html.twig', id: 'cache' }
5+
- { name: data_collector, template: 'CacheBundle:Collector:cache.html.twig', id: 'cache' }
66

77
cache.router_listener:
88
class: Cache\CacheBundle\Routing\RouterListener
9-
arguments: [~, %cache.router.ttl%]
9+
arguments: [~, ~]
1010
tags:
1111
- { name: kernel.event_listener, event: kernel.request, method: onBeforeRouting, priority: 33 }
1212
- { name: kernel.event_listener, event: kernel.request, method: onAfterRouting, priority: 31 }

0 commit comments

Comments
 (0)