[BUGFIX] Avoid double autoloading of class aliases

CHANGELOG.md

@@ -20,6 +20,8 @@ Please also have a look at our
 
 ### Fixed
 
+- Avoid double autoloading of class aliases (#1552)
+
 ### Documentation
 
 ## 9.2.0: New features and deprecations

class-aliases/Rule/Rule.php

@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sabberworm\CSS\Rule;
+
+use Sabberworm\CSS\Property\Declaration;
+
+use function Safe\class_alias;
+
+if (!\class_exists(Rule::class) && !\interface_exists(Rule::class)) {
+    /**
+     * @deprecated in v9.2, will be removed in v10.0.  Use `Property\Declaration` instead, which is a direct
+     *             replacement.
+     */
+    class_alias(Declaration::class, Rule::class);
+}

class-aliases/RuleSet/RuleContainer.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sabberworm\CSS\RuleSet;
+
+use function Safe\class_alias;
+
+if (!\class_exists(RuleContainer::class) && !\interface_exists(RuleContainer::class)) {
+    /**
+     * @deprecated in v9.2, will be removed in v10.0.  Use `DeclarationList` instead, which is a direct replacement.
+     */
+    class_alias(DeclarationList::class, RuleContainer::class);
+}

composer.json

@@ -48,8 +48,8 @@
             "Sabberworm\\CSS\\": "src/"
         },
         "files": [
-            "src/Rule/Rule.php",
-            "src/RuleSet/RuleContainer.php"
+            "class-aliases/Rule/Rule.php",
+            "class-aliases/RuleSet/RuleContainer.php"
         ]
     },
     "autoload-dev": {
@@ -80,9 +80,9 @@
         "check:dynamic": [
             "@check:tests"
         ],
-        "check:php:codesniffer": "phpcs --standard=config/phpcs.xml bin config src tests",
-        "check:php:fixer": "\"./.phive/php-cs-fixer\" --config=config/php-cs-fixer.php fix --dry-run -v --show-progress=dots --diff bin config src tests",
-        "check:php:lint": "parallel-lint bin config src tests",
+        "check:php:codesniffer": "phpcs --standard=config/phpcs.xml bin class-aliases config src tests",
+        "check:php:fixer": "\"./.phive/php-cs-fixer\" --config=config/php-cs-fixer.php fix --dry-run -v --show-progress=dots --diff bin class-aliases config src tests",
+        "check:php:lint": "parallel-lint bin class-aliases config src tests",
         "check:php:rector": "rector process --no-progress-bar --dry-run --config=config/rector.php",
         "check:php:stan": "phpstan --no-progress --configuration=config/phpstan.neon",
         "check:static": [
@@ -109,8 +109,8 @@
             "@fix:php:codesniffer",
             "@fix:php:fixer"
         ],
-        "fix:php:codesniffer": "phpcbf --standard=config/phpcs.xml bin config src tests",
-        "fix:php:fixer": "\"./.phive/php-cs-fixer\" --config=config/php-cs-fixer.php fix bin config src tests",
+        "fix:php:codesniffer": "phpcbf --standard=config/phpcs.xml bin class-aliases config src tests",
+        "fix:php:fixer": "\"./.phive/php-cs-fixer\" --config=config/php-cs-fixer.php fix bin class-aliases config src tests",
         "fix:php:rector": "rector process --config=config/rector.php",
         "phpstan:baseline": "phpstan --configuration=config/phpstan.neon --generate-baseline=config/phpstan-baseline.neon --allow-empty-baseline",
         "phpstan:clearcache": "phpstan clear-result-cache"

config/phpstan.neon

@@ -8,6 +8,7 @@ parameters:
 
   paths:
     - %currentWorkingDirectory%/bin/
+    - %currentWorkingDirectory%/class-aliases/
     - %currentWorkingDirectory%/src/
     - %currentWorkingDirectory%/tests/
 

config/rector.php

@@ -11,6 +11,7 @@
     ->withPaths(
         [
             __DIR__ . '/../bin',
+            __DIR__ . '/../class-aliases',
             __DIR__ . '/../config',
             __DIR__ . '/../src',
             __DIR__ . '/../tests',