[Cache] [FrameworkBundle] Fix logging for TagAwareAdapter

Author: fancyweb

DependencyInjection/FrameworkExtension.php

@@ -1948,6 +1948,12 @@ private function registerCacheConfiguration(array $config, ContainerBuilder $con
                     ->setPublic($pool['public'])
                 ;
 
+                if (method_exists(TagAwareAdapter::class, 'setLogger')) {
+                    $container
+                        ->getDefinition($name)
+                        ->addMethodCall('setLogger', [new Reference('logger', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)]);
+                }
+
                 $pool['name'] = $name;
                 $pool['public'] = false;
                 $name = '.'.$name.'.inner';

Tests/DependencyInjection/Fixtures/php/cache.php

@@ -32,6 +32,11 @@
                     'redis://foo' => 'cache.adapter.redis',
                 ],
             ],
+            'cache.ccc' => [
+                'adapter' => 'cache.adapter.array',
+                'default_lifetime' => 410,
+                'tags' => true,
+            ],
         ],
     ],
 ]);

Tests/DependencyInjection/Fixtures/xml/cache.xml

@@ -17,6 +17,7 @@
                 <framework:adapter name="cache.adapter.filesystem" />
                 <framework:adapter name="cache.adapter.redis" provider="redis://foo" />
             </framework:pool>
+            <framework:pool name="cache.ccc" adapter="cache.adapter.array" default-lifetime="410" tags="true" />
         </framework:cache>
     </framework:config>
 </container>

Tests/DependencyInjection/Fixtures/yml/cache.yml

@@ -23,3 +23,7 @@ framework:
                     - cache.adapter.array
                     - cache.adapter.filesystem
                     - {name: cache.adapter.redis, provider: 'redis://foo'}
+            cache.ccc:
+                adapter: cache.adapter.array
+                default_lifetime: 410
+                tags: true

Tests/DependencyInjection/FrameworkExtensionTest.php

@@ -26,6 +26,7 @@
 use Symfony\Component\Cache\Adapter\FilesystemAdapter;
 use Symfony\Component\Cache\Adapter\ProxyAdapter;
 use Symfony\Component\Cache\Adapter\RedisAdapter;
+use Symfony\Component\Cache\Adapter\TagAwareAdapter;
 use Symfony\Component\Cache\DependencyInjection\CachePoolPass;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\DependencyInjection\ChildDefinition;
@@ -1504,6 +1505,17 @@ public function testCachePoolServices()
             12,
         ];
         $this->assertEquals($expected, $chain->getArguments());
+
+        // Test "tags: true" wrapping logic
+        $tagAwareDefinition = $container->getDefinition('cache.ccc');
+        $this->assertSame(TagAwareAdapter::class, $tagAwareDefinition->getClass());
+        $this->assertCachePoolServiceDefinitionIsCreated($container, (string) $tagAwareDefinition->getArgument(0), 'cache.adapter.array', 410);
+
+        if (method_exists(TagAwareAdapter::class, 'setLogger')) {
+            $this->assertEquals([
+                ['setLogger', [new Reference('logger', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)]],
+            ], $tagAwareDefinition->getMethodCalls());
+        }
     }
 
     public function testRemovesResourceCheckerConfigCacheFactoryArgumentOnlyIfNoDebug()
@@ -1813,6 +1825,9 @@ private function assertCachePoolServiceDefinitionIsCreated(ContainerBuilder $con
             case 'cache.adapter.redis':
                 $this->assertSame(RedisAdapter::class, $parentDefinition->getClass());
                 break;
+            case 'cache.adapter.array':
+                $this->assertSame(ArrayAdapter::class, $parentDefinition->getClass());
+                break;
             default:
                 $this->fail('Unresolved adapter: '.$adapter);
         }