The SCSS compiler mistakes some strings as namespaced modules

See https://www.woltlab.com/community/thread/314508/

Author: dtdesign

wcfsetup/install/files/lib/system/style/StyleCompiler.class.php

@@ -3,6 +3,7 @@
 namespace wcf\system\style;
 
 use ScssPhp\ScssPhp\Compiler;
+use ScssPhp\ScssPhp\Exception\SassException;
 use ScssPhp\ScssPhp\OutputStyle;
 use ScssPhp\ScssPhp\ValueConverter;
 use wcf\data\application\Application;
@@ -705,20 +706,18 @@ private function prepareFile(string $filename): string
     private function compileStylesheet(string $scss, array $variables): string
     {
         $compiler = $this->makeCompiler();
-        $compiler->replaceVariables(\array_map(static function ($value) {
+        $compiler->replaceVariables(\array_map(static function ($value) use (&$i) {
             if ($value === "" || \is_int($value)) {
                 return ValueConverter::fromPhp($value);
             }
 
-            // The SCSS v2 compiler trips over string values when trying to feed
-            // them into `parseValue()`. Explicitly treating them as strings
-            // when they are not numeric fixes this while preserving the ability
-            // to work with numbers passed as strings.
-            if (\is_string($value) && !\is_numeric($value)) {
-                return ValueConverter::fromPhp($value);
+            try {
+                return ValueConverter::parseValue($value);
+            } catch (SassException $e) {
+                if (\str_contains($e->getMessage(), 'Sass modules are not implemented yet')) {
+                    return ValueConverter::fromPhp($value);
+                }
             }
-
-            return ValueConverter::parseValue($value);
         }, $variables));
 
         try {