fix: use parameter placeholders in configuration

Author: simPod

src/DependencyInjection/Configuration.php

@@ -24,24 +24,8 @@ final class Configuration implements ConfigurationInterface
     public const SectionConsole = 'console';
     public const SectionController = 'controller';
 
-    /** @var string */
-    private $alias;
-
-    /** @var string */
-    private $kernelProjectDir;
-
-    /** @var string */
-    private $kernelLogsDir;
-
-    /** @var string */
-    private $kernelCacheDir;
-
-    public function __construct(string $alias, string $kernelProjectDir, string $kernelLogsDir, string $kernelCacheDir)
+    public function __construct(private readonly string $alias)
     {
-        $this->alias = $alias;
-        $this->kernelProjectDir = $kernelProjectDir;
-        $this->kernelLogsDir = $kernelLogsDir;
-        $this->kernelCacheDir = $kernelCacheDir;
     }
 
     /** @return TreeBuilder<'array'> */
@@ -81,7 +65,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                                     TracyLogger::class,
                                 ),
                             )
-                            ->defaultValue($this->kernelLogsDir)
+                            ->defaultValue('%kernel.logs_dir%')
                             ->end()
                         ->scalarNode(self::ParameterConsoleBrowser)
                             ->info(
@@ -106,8 +90,8 @@ public function getConfigTreeBuilder(): TreeBuilder
                             ->prototype('scalar')
                                 ->end()
                             ->defaultValue([
-                                $this->kernelProjectDir . '/bootstrap.php.cache',
-                                $this->kernelCacheDir,
+                                '%kernel.project_dir%/bootstrap.php.cache',
+                                '%kernel.cache_dir%',
                             ])
                             ->end()
                         ->end()

src/DependencyInjection/TracyBlueScreenExtension.php

@@ -80,19 +80,7 @@ public function loadInternal(array $mergedConfig, ContainerBuilder $container):
     /** @param mixed[] $config */
     public function getConfiguration(array $config, ContainerBuilder $container): Configuration
     {
-        $kernelProjectDir = $container->getParameter('kernel.project_dir');
-        $kernelLogsDir = $container->getParameter('kernel.logs_dir');
-        $kernelCacheDir = $container->getParameter('kernel.cache_dir');
-        assert(is_string($kernelProjectDir));
-        assert(is_string($kernelLogsDir));
-        assert(is_string($kernelCacheDir));
-
-        return new Configuration(
-            $this->getAlias(),
-            $kernelProjectDir,
-            $kernelLogsDir,
-            $kernelCacheDir,
-        );
+        return new Configuration($this->getAlias());
     }
 
     private function isEnabled(bool|null $configOption, string $environment, bool $debug): bool

tests/DependencyInjection/TracyBlueScreenExtensionConsoleTest.php

@@ -15,9 +15,6 @@ public function setUp(): void
     {
         parent::setUp();
 
-        $this->setParameter('kernel.project_dir', __DIR__);
-        $this->setParameter('kernel.logs_dir', __DIR__ . '/tests-logs-dir');
-        $this->setParameter('kernel.cache_dir', __DIR__ . '/tests-cache-dir');
         $this->setParameter('kernel.environment', 'dev');
         $this->setParameter('kernel.debug', true);
     }
@@ -85,7 +82,7 @@ public function testDefaultLogsDirIsKernelLogsDir(): void
 
         $this->assertContainerBuilderHasParameter(
             TracyBlueScreenExtension::ContainerParameterConsoleLogDirectory,
-            __DIR__ . '/tests-logs-dir',
+            '%kernel.logs_dir%',
         );
     }
 

tests/DependencyInjection/TracyBlueScreenExtensionControllerTest.php

@@ -15,9 +15,6 @@ public function setUp(): void
     {
         parent::setUp();
 
-        $this->setParameter('kernel.project_dir', __DIR__);
-        $this->setParameter('kernel.logs_dir', __DIR__);
-        $this->setParameter('kernel.cache_dir', __DIR__ . '/tests-cache-dir');
         $this->setParameter('kernel.environment', 'dev');
         $this->setParameter('kernel.debug', true);
     }

tests/DependencyInjection/TracyBlueScreenExtensionTest.php

@@ -102,7 +102,7 @@ public function testCollapseCacheDirsByDefault(): void
         $collapsePaths = $this->container->getParameter('cdn77.tracy_blue_screen.blue_screen.collapse_paths');
 
         $this->assertArrayContainsStringPart('/bootstrap.php.cache', $collapsePaths);
-        $this->assertArrayContainsStringPart('/tests-cache-dir', $collapsePaths);
+        $this->assertArrayContainsStringPart('%kernel.cache_dir%', $collapsePaths);
     }
 
     public function testSetCollapseDirs(): void