Added services.yml

Author: Nyholm

src/DependencyInjection/CacheExtension.php

@@ -11,16 +11,17 @@
 
 namespace Cache\CacheBundle\DependencyInjection;
 
-use Cache\CacheBundle\DataCollector\CacheDataCollector;
-use Cache\CacheBundle\Routing\RouterListener;
+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,10 +35,8 @@ 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']) {
@@ -46,13 +45,14 @@ public function load(array $configs, ContainerBuilder $container)
         }
 
         if ($config['router']['enabled']) {
-            $container->register('cache.router_listener', RouterListener::class)
-                ->addArgument(new Reference($config['router']['service_id']))
-                ->addArgument($config['router']['ttl'])
-                ->addTag('kernel.event_listener', ['event'=>'kernel.request', 'method'=>'onBeforeRouting', 'priority'=>33])
-                ->addTag('kernel.event_listener', ['event'=>'kernel.request', 'method'=>'onAfterRouting', 'priority'=>31]);
+            $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/Resources/config/services.yml

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