Merge pull request #5 from php-cache/routing

Added a router listener

Author: cryptiklemur

src/CacheBundle.php

@@ -31,7 +31,6 @@ public function build(ContainerBuilder $container)
 
         $container->addCompilerPass(new Compiler\SessionSupportCompilerPass());
         $container->addCompilerPass(new Compiler\DoctrineSupportCompilerPass());
-        $container->addCompilerPass(new Compiler\RouterCompilerPass());
 
         if ($container->getParameter('kernel.debug')) {
             $container->addCompilerPass(new Compiler\DataCollectorCompilerPass());

src/DependencyInjection/CacheExtension.php

@@ -11,16 +11,17 @@
 
 namespace Cache\CacheBundle\DependencyInjection;
 
-use Cache\CacheBundle\Cache\LoggingCachePool;
-use Cache\CacheBundle\DataCollector\CacheDataCollector;
+use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
+use Symfony\Component\DependencyInjection\Loader;
+
 
 /**
- * Class CacheExtension
  *
  * @author Aaron Scherer <[email protected]>
+ * @author Tobias Nyholm <[email protected]>
  */
 class CacheExtension extends Extension
 {
@@ -34,17 +35,24 @@ public function load(array $configs, ContainerBuilder $container)
     {
         $config = $this->processConfiguration(new Configuration(), $configs);
 
-        if ($container->getParameter('kernel.debug')) {
-            $container->register('data_collector.cache', CacheDataCollector::class)
-                ->addTag('data_collector', ['template' => CacheDataCollector::TEMPLATE, 'id' => 'cache']);
-        }
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader->load('services.yml');
 
         foreach (['router', 'session', 'doctrine'] as $section) {
             if ($config[$section]['enabled']) {
                 $container->setParameter('cache.'.$section, $config[$section]);
             }
         }
 
+        if ($config['router']['enabled']) {
+            $container->getDefinition('cache.router_listener')->replaceArgument(0, new Reference($config['router']['service_id']));
+        } else {
+            $container->removeDefinition('cache.router_listener');
+        }
+
+        if (!$container->getParameter('kernel.debug')) {
+            $container->removeDefinition('data_collector.cache');
+        }
     }
 
     public function getAlias()

src/DependencyInjection/Compiler/RouterCompilerPass.php

@@ -142,10 +142,7 @@ private function addRouterSection()
                     ->defaultFalse()
                 ->end()
                 ->integerNode('ttl')
-                    ->defaultValue(86400)
-                ->end()
-                ->booleanNode('auto-register')
-                    ->defaultTrue()
+                    ->defaultValue(604800)
                 ->end()
                 ->scalarNode('service_id')
                     ->isRequired()

src/DependencyInjection/Configuration.php

@@ -0,0 +1,12 @@
+services:
+  data_collector.cache:
+    class: Cache\CacheBundle\DataCollector\CacheDataCollector
+    tags:
+      - { name: data_collectorr, template: 'CacheBundle:Collector:cache.html.twig', id: 'cache' }
+
+  cache.router_listener:
+    class: Cache\CacheBundle\Routing\RouterListener
+    arguments: [~, %cache.router.ttl%]
+    tags:
+      - { name: kernel.event_listener, event: kernel.request, method: onBeforeRouting, priority: 33 }
+      - { name: kernel.event_listener, event: kernel.request, method: onAfterRouting, priority: 31 }