Fix the auto-detection of the Cache component to enable performance tracing (#501)

Author: guillaume-ro-fr

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Fix the CacheItem detection for tracing (#501)
+
 ## 4.1.1 (2021-05-10)
 
 - Fix the conditions to automatically enable the cache instrumentation when possible (#487)

src/DependencyInjection/Configuration.php

@@ -165,7 +165,7 @@ private function addDistributedTracingSection(ArrayNodeDefinition $rootNode): vo
                             ->{class_exists(TwigBundle::class) ? 'canBeDisabled' : 'canBeEnabled'}()
                         ->end()
                         ->arrayNode('cache')
-                            ->{interface_exists(CacheItem::class) ? 'canBeDisabled' : 'canBeEnabled'}()
+                            ->{class_exists(CacheItem::class) ? 'canBeDisabled' : 'canBeEnabled'}()
                         ->end()
                     ->end()
                 ->end()

src/DependencyInjection/SentryExtension.php

@@ -224,7 +224,7 @@ private function registerCacheTracingConfiguration(ContainerBuilder $container,
         $isConfigEnabled = $this->isConfigEnabled($container, $config)
             && $this->isConfigEnabled($container, $config['cache']);
 
-        if ($isConfigEnabled && !interface_exists(CacheItem::class)) {
+        if ($isConfigEnabled && !class_exists(CacheItem::class)) {
             throw new LogicException('Cache tracing support cannot be enabled because the symfony/cache Composer package is not installed.');
         }
 

tests/DependencyInjection/ConfigurationTest.php

@@ -48,7 +48,7 @@ public function testProcessConfigurationWithDefaultConfiguration(): void
                     'enabled' => class_exists(TwigBundle::class),
                 ],
                 'cache' => [
-                    'enabled' => interface_exists(CacheItem::class),
+                    'enabled' => class_exists(CacheItem::class),
                 ],
             ],
         ];