cs fixer

.github/workflows/static.yml

@@ -0,0 +1,47 @@
+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
+      - name: Install phpstan
+        run: |
+#          composer require --no-update jackalope/jackalope-doctrine-dbal jackalope/jackalope-jackrabbit phpstan/phpstan
+          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)
+;

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) {

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,9 +69,6 @@ public function setParentDocument($parent): static
         return $this;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getParentDocument(): ?object
     {
         return $this->parent;
@@ -160,9 +157,6 @@ public function getPrefix(): string
         return $this->idPrefix;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function setPrefix(string $prefix): static
     {
         $this->idPrefix = $prefix;
@@ -206,9 +200,6 @@ public function generateStaticPrefix(string $id, string $idPrefix): string
         return $url;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPath(): string
     {
         $pattern = parent::getPath();
@@ -240,9 +231,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

@@ -172,9 +172,6 @@ public function getPrefix(): ?string
         return $this->idPrefix;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function setPrefix(string $prefix): static
     {
         $this->idPrefix = $prefix;
@@ -264,9 +261,6 @@ public function getChildren(): Collection
         return $this->children;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function isBooleanOption(string $name): bool
     {
         return 'add_trailing_slash' === $name || parent::isBooleanOption($name);

src/Doctrine/Phpcr/RouteProvider.php

@@ -45,7 +45,7 @@ public function __construct(
         ManagerRegistry $managerRegistry,
         CandidatesInterface $candidatesStrategy,
         ?string $className = null,
-        LoggerInterface $logger = null
+        ?LoggerInterface $logger = null
     ) {
         parent::__construct($managerRegistry, $className);
         $this->candidatesStrategy = $candidatesStrategy;
@@ -102,8 +102,6 @@ public function getRouteCollectionForRequest(Request $request): RouteCollection
     }
 
     /**
-     * {@inheritdoc}
-     *
      * @param string $name The absolute path or uuid of the Route document
      */
     public function getRouteByName(string $name): SymfonyRoute

src/Form/Type/RouteTypeType.php

@@ -19,9 +19,6 @@ class RouteTypeType extends AbstractType
 {
     protected array $routeTypes = [];
 
-    /**
-     * {@inheritdoc}
-     */
     public function configureOptions(OptionsResolver $resolver): void
     {
         $choices = [];
@@ -43,17 +40,11 @@ public function addRouteType(string $type): void
         $this->routeTypes[$type] = $type;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getParent(): string
     {
         return ChoiceType::class;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getBlockPrefix(): string
     {
         return 'cmf_routing_route_type';