Make sure you can change tags with a constructor argument

Author: Nyholm

src/Cache/DoctrineTaggingCachePool.php renamed to src/Cache/FixedTaggingCachePool.php

@@ -21,87 +21,94 @@
  *
  * @author Tobias Nyholm <[email protected]>
  */
-class DoctrineTaggingCachePool implements CacheItemPoolInterface
+class FixedTaggingCachePool implements CacheItemPoolInterface
 {
     /**
      * @type CacheItemPoolInterface|TaggablePoolInterface
      */
     private $cache;
 
+    /**
+     * @type array
+     */
+    private $tags;
+
     /**
      * @param TaggablePoolInterface $cache
+     * @param array                 $tags
      */
-    public function __construct(TaggablePoolInterface $cache)
+    public function __construct(TaggablePoolInterface $cache, array $tags)
     {
         $this->cache = $cache;
+        $this->tags  = $tags;
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function getItem($key)
     {
-        return $this->cache->getItem($key, ['doctrine']);
+        return $this->cache->getItem($key, $this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function getItems(array $keys = [])
     {
-        return $this->cache->getItems($keys, ['doctrine']);
+        return $this->cache->getItems($keys, $this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function hasItem($key)
     {
-        return $this->cache->hasItem($key, ['doctrine']);
+        return $this->cache->hasItem($key, $this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function clear()
     {
-        return $this->cache->clear(['doctrine']);
+        return $this->cache->clear($this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function deleteItem($key)
     {
-        return $this->cache->deleteItem($key, ['doctrine']);
+        return $this->cache->deleteItem($key, $this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function deleteItems(array $keys)
     {
-        return $this->cache->deleteItems($keys, ['doctrine']);
+        return $this->cache->deleteItems($keys, $this->tags);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function save(CacheItemInterface $item)
     {
         return $this->cache->save($item);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function saveDeferred(CacheItemInterface $item)
     {
         return $this->cache->saveDeferred($item);
     }
 
     /**
-     * @{@inheritdoc}
+     * {@inheritdoc}
      */
     public function commit()
     {

src/DependencyInjection/Compiler/DoctrineSupportCompilerPass.php

@@ -12,7 +12,7 @@
 namespace Cache\CacheBundle\DependencyInjection\Compiler;
 
 use Cache\Bridge\DoctrineCacheBridge;
-use Cache\CacheBundle\Cache\DoctrineTaggingCachePool;
+use Cache\CacheBundle\Cache\FixedTaggingCachePool;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\DependencyInjection\Reference;
 
@@ -106,8 +106,9 @@ public function getPoolReferenceForBridge($bridgeServiceId, $cacheData, $tagging
         }
 
         $taggingServiceId = $bridgeServiceId.'.tagging';
-        $taggingDef       = $this->container->register($taggingServiceId, DoctrineTaggingCachePool::class);
+        $taggingDef       = $this->container->register($taggingServiceId, FixedTaggingCachePool::class);
         $taggingDef->addArgument(new Reference($cacheData['service_id']))
+            ->addArgument(['doctrine'])
             ->setPublic(false);
 
         return $taggingServiceId;