cs fixer

.github/workflows/static.yml

@@ -0,0 +1,48 @@
+name: Static analysis
+
+on:
+  push:
+    branches:
+      - '[0-9]+.x'
+      - '[0-9]+.[0-9]+'
+      - '[0-9]+.[0-9]+.x'
+  pull_request:
+
+jobs:
+  phpstan:
+    name: PHPStan
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.3'
+          extensions: "curl,dom,json,xml,dom"
+          coverage: none
+
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      # have to install phpstan ourselves here, the phpstan-ga fails at composer install with weird errors
+          #          composer require --no-update jackalope/jackalope-doctrine-dbal jackalope/jackalope-jackrabbit phpstan/phpstan
+      - name: Install phpstan
+        run: |
+
+          composer update
+
+      - name: PHPStan
+        run: vendor/bin/phpstan analyze --no-progress
+
+  php-cs-fixer:
+    name: PHP-CS-Fixer
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: PHP-CS-Fixer
+        uses: docker://oskarstark/php-cs-fixer-ga
+        with:
+          args: --dry-run --diff

.gitignore

@@ -2,4 +2,5 @@ phpunit.xml
 composer.lock
 vendor
 tests/Fixtures/App/var/
-.phpunit.result.cache 
+.phpunit.result.cache
+.php-cs-fixer.cache

.php-cs-fixer.dist.php

@@ -0,0 +1,19 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->in(__DIR__.'/src')
+    ->in(__DIR__.'/tests')
+    ->exclude('tests/Fixtures/App/var/')
+    ->name('*.php')
+;
+
+$config = new PhpCsFixer\Config();
+
+return $config
+    ->setRiskyAllowed(true)
+    ->setRules([
+        '@Symfony' => true,
+        'single_line_throw' => false,
+    ])
+    ->setFinder($finder)
+;

composer.json

@@ -52,6 +52,7 @@
     "conflict": {
         "doctrine/common": "<3.1.1",
         "doctrine/persistence": "<1.3.0",
+        "doctrine/phpcr-odm": "<2.0",
         "symfony/doctrine-bridge": "<6.4.0",
         "symfony/security-core": "<6.4.0"
     },

src/DependencyInjection/CmfRoutingExtension.php

@@ -27,9 +27,6 @@
  */
 final class CmfRoutingExtension extends Extension
 {
-    /**
-     * {@inheritdoc}
-     */
     public function load(array $configs, ContainerBuilder $container): void
     {
         $config = $this->processConfiguration(new Configuration(), $configs);

src/Doctrine/Orm/ContentRepository.php

@@ -37,8 +37,6 @@ protected function getModelAndId(string $identifier): array
     }
 
     /**
-     * {@inheritdoc}
-     *
      * @param string $id The ID contains both model name and id, separated by a colon
      */
     public function findById(mixed $id): ?object

src/Doctrine/Orm/RedirectRoute.php

@@ -52,9 +52,6 @@ public function setParameters(array $parameters): void
         $this->serialisedParameters = json_encode($parameters, \JSON_THROW_ON_ERROR);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getParameters(): array
     {
         if (!isset($this->serialisedParameters)) {
@@ -65,9 +62,6 @@ public function getParameters(): array
         return \is_array($params) ? $params : [];
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPath(): string
     {
         $pattern = parent::getPath();
@@ -78,9 +72,6 @@ public function getPath(): string
         return $pattern;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function isBooleanOption(string $name): bool
     {
         return 'add_trailing_slash' === $name || parent::isBooleanOption($name);

src/Doctrine/Orm/Route.php

@@ -58,9 +58,6 @@ public function getPosition(): int
         return $this->position;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getRouteKey(): string
     {
         return $this->getName();

src/Doctrine/Orm/RouteProvider.php

@@ -41,9 +41,6 @@ public function __construct(ManagerRegistry $managerRegistry, CandidatesInterfac
         $this->candidatesStrategy = $candidatesStrategy;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getRouteCollectionForRequest(Request $request): RouteCollection
     {
         $collection = new RouteCollection();
@@ -52,7 +49,7 @@ public function getRouteCollectionForRequest(Request $request): RouteCollection
         if (0 === \count($candidates)) {
             return $collection;
         }
-        $routes = $this->getRouteRepository()->findByStaticPrefix($candidates, ['position' => 'ASC']); /** @phpstan-ignore-line */
+        $routes = $this->getRouteRepository()->findByStaticPrefix($candidates, ['position' => 'ASC']); /* @phpstan-ignore-line */
         /** @var Route $route */
         foreach ($routes as $route) {
             $collection->add($route->getName(), $route);
@@ -61,9 +58,6 @@ public function getRouteCollectionForRequest(Request $request): RouteCollection
         return $collection;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getRouteByName($name): SymfonyRoute
     {
         if (!$this->candidatesStrategy->isCandidate($name)) {
@@ -78,9 +72,6 @@ public function getRouteByName($name): SymfonyRoute
         return $route;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getRoutesByNames($names = null): array
     {
         if (null === $names) {
@@ -90,7 +81,7 @@ public function getRoutesByNames($names = null): array
 
             try {
                 return $this->getRouteRepository()->findBy([], null, $this->routeCollectionLimit ?: null);
-            } catch (TableNotFoundException $e) {
+            } catch (TableNotFoundException) {
                 return [];
             }
         }
@@ -100,7 +91,7 @@ public function getRoutesByNames($names = null): array
             // TODO: if we do findByName with multivalue, we need to filter with isCandidate afterwards
             try {
                 $routes[] = $this->getRouteByName($name);
-            } catch (RouteNotFoundException $e) {
+            } catch (RouteNotFoundException) {
                 // not found
             }
         }

src/Doctrine/Phpcr/ContentRepository.php

@@ -26,17 +26,11 @@
  */
 class ContentRepository extends DoctrineProvider implements ContentRepositoryInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function findById($id): ?object
     {
         return $this->getObjectManager()->find(null, $id);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getContentId($content): ?string
     {
         if (!\is_object($content)) {

src/Doctrine/Phpcr/PrefixCandidates.php

@@ -47,7 +47,7 @@ final class PrefixCandidates extends Candidates
      *                                       is using
      * @param int                  $limit    Limit to candidates generated per prefix
      */
-    public function __construct(array $prefixes, array $locales = [], ManagerRegistry $doctrine = null, int $limit = 20)
+    public function __construct(array $prefixes, array $locales = [], ?ManagerRegistry $doctrine = null, int $limit = 20)
     {
         parent::__construct($locales, $limit);
         $this->setPrefixes($prefixes);
@@ -74,8 +74,6 @@ public function isCandidate($name): bool
     }
 
     /**
-     * {@inheritdoc}
-     *
      * @param QueryBuilder $queryBuilder
      */
     public function restrictQuery($queryBuilder): void
@@ -91,9 +89,6 @@ public function restrictQuery($queryBuilder): void
         }
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getCandidates(Request $request): array
     {
         $candidates = [];

src/Doctrine/Phpcr/RedirectRoute.php

@@ -69,36 +69,11 @@ public function setParentDocument($parent): static
         return $this;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getParentDocument(): ?object
     {
         return $this->parent;
     }
 
-    /**
-     * @deprecated For BC with the PHPCR-ODM 1.4 HierarchyInterface
-     * @see setParentDocument
-     */
-    public function setParent($parent)
-    {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
-
-        return $this->setParentDocument($parent);
-    }
-
-    /**
-     * @deprecated For BC with the PHPCR-ODM 1.4 HierarchyInterface
-     * @see getParentDocument
-     */
-    public function getParent()
-    {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
-
-        return $this->getParentDocument();
-    }
-
     /**
      * Rename a route by setting its new name.
      *
@@ -160,9 +135,6 @@ public function getPrefix(): string
         return $this->idPrefix;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function setPrefix(string $prefix): static
     {
         $this->idPrefix = $prefix;
@@ -206,9 +178,6 @@ public function generateStaticPrefix(string $id, string $idPrefix): string
         return $url;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPath(): string
     {
         $pattern = parent::getPath();
@@ -240,9 +209,6 @@ public function getRouteChildren(): array
         return $children;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function isBooleanOption(string $name): bool
     {
         return 'add_trailing_slash' === $name || parent::isBooleanOption($name);

src/Doctrine/Phpcr/Route.php

@@ -89,28 +89,6 @@ public function getParentDocument(): object
         return $this->parent;
     }
 
-    /**
-     * @deprecated For BC with the PHPCR-ODM 1.4 HierarchyInterface
-     * @see setParentDocument
-     */
-    public function setParent($parent)
-    {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
-
-        return $this->setParentDocument($parent);
-    }
-
-    /**
-     * @deprecated For BC with the PHPCR-ODM 1.4 HierarchyInterface
-     * @see getParentDocument
-     */
-    public function getParent()
-    {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
-
-        return $this->getParentDocument();
-    }
-
     /**
      * Rename a route by setting its new name.
      *
@@ -172,9 +150,6 @@ public function getPrefix(): ?string
         return $this->idPrefix;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function setPrefix(string $prefix): static
     {
         $this->idPrefix = $prefix;
@@ -264,9 +239,6 @@ public function getChildren(): Collection
         return $this->children;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function isBooleanOption(string $name): bool
     {
         return 'add_trailing_slash' === $name || parent::isBooleanOption($name);