resolve paths after configuration processing

Author: wryk

src/AssetMapper/SassCssCompiler.php

@@ -12,7 +12,6 @@
 use Symfony\Component\AssetMapper\AssetMapperInterface;
 use Symfony\Component\AssetMapper\Compiler\AssetCompilerInterface;
 use Symfony\Component\AssetMapper\MappedAsset;
-use Symfony\Component\Filesystem\Path;
 use Symfonycasts\SassBundle\SassBuilder;
 
 class SassCssCompiler implements AssetCompilerInterface
@@ -28,9 +27,7 @@ public function __construct(
     public function supports(MappedAsset $asset): bool
     {
         foreach ($this->scssPaths as $path) {
-            $absolutePath = Path::isAbsolute($path) ? $path : Path::makeAbsolute($path, $this->projectDir);
-
-            if (realpath($asset->sourcePath) === realpath($absolutePath)) {
+            if (realpath($asset->sourcePath) === realpath($path)) {
                 return true;
             }
         }

src/DependencyInjection/SymfonycastsSassExtension.php

@@ -16,6 +16,7 @@
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Extension\Extension;
 use Symfony\Component\DependencyInjection\Loader;
+use Symfony\Component\Filesystem\Path;
 
 class SymfonycastsSassExtension extends Extension implements ConfigurationInterface
 {
@@ -27,6 +28,16 @@ public function load(array $configs, ContainerBuilder $container): void
         $configuration = $this->getConfiguration($configs, $container);
         $config = $this->processConfiguration($configuration, $configs);
 
+        // Ensure paths are absolute
+        $normalizeRootSassPath = function ($path) use ($container) {
+            return Path::isAbsolute($container->getParameterBag()->resolveValue($path))
+                ? $path
+                : '%kernel.project_dir%/'.$path
+            ;
+        };
+
+        $config['root_sass'] = array_map($normalizeRootSassPath, $config['root_sass']);
+
         // BC Layer with SassBundle < 0.4
         if (isset($config['embed_sourcemap'])) {
             $config['sass_options']['embed_source_map'] = $config['embed_sourcemap'];

tests/AssetMapper/SassCssCompilerTest.php

@@ -22,22 +22,13 @@ public function testSupports()
 
         $asset = new MappedAsset('assets/app.scss', __DIR__.'/../fixtures/assets/app.scss');
 
-        $compilerAbsolutePath = new SassCssCompiler(
+        $compiler = new SassCssCompiler(
             [__DIR__.'/../fixtures/assets/app.scss'],
             __DIR__.'/../fixtures/var/sass',
             __DIR__.'/../fixtures',
             $builder
         );
 
-        $this->assertTrue($compilerAbsolutePath->supports($asset), 'Supports absolute paths');
-
-        $compilerRelativePath = new SassCssCompiler(
-            ['assets/app.scss'],
-            __DIR__.'/../fixtures/var/sass',
-            __DIR__.'/../fixtures',
-            $builder
-        );
-
-        $this->assertTrue($compilerRelativePath->supports($asset), 'Supportes relative paths');
+        $this->assertTrue($compiler->supports($asset));
     }
 }