Fix PHPStan errors

Author: loevgaard

phpstan.neon

@@ -22,7 +22,7 @@ parameters:
         objectManagerLoader: tests/PHPStan/object_manager.php
 
     reportUnmatchedIgnoredErrors: false
-    treatPhpDocTypesAsCertain: false
+    treatPhpDocTypesAsCertain: true
 
     ignoreErrors:
         -
@@ -31,6 +31,7 @@ parameters:
                 - doctrine.associationType
                 - doctrine.columnType
                 - missingType.generics
+                - missingType.parameter
         -
             identifier: class.notFound
             message: '#League\\Flysystem\\#'

src/Controller/Action/Shop/ShowFeedAction.php

@@ -68,9 +68,8 @@ public function __invoke(string $code): StreamedResponse
             }
         }
 
-        /** @var resource|false $stream */
         $stream = $filesystem->readStream((string) $feedPath);
-        if (false === $stream) {
+        if (!\is_resource($stream)) {
             throw new RuntimeException(sprintf('An error occurred trying to read the feed file %s', $feedPath));
         }
 

src/DataProvider/DataProvider.php

@@ -23,6 +23,9 @@ class DataProvider implements DataProviderInterface
     /** @var CollectionBatcherInterface[] */
     private array $batchers = [];
 
+    /**
+     * @param class-string $class
+     */
     public function __construct(
         private readonly BatcherFactoryInterface $batcherFactory,
         private readonly QueryRebuilderInterface $queryRebuilder,
@@ -53,6 +56,7 @@ public function getBatchCount(ChannelInterface $channel, LocaleInterface $locale
 
     public function getItems(BatchInterface $batch): iterable
     {
+        /** @phpstan-ignore return.type */
         return $this->queryRebuilder->rebuild($batch)->getResult();
     }
 

src/DataProvider/DataProviderInterface.php

@@ -14,6 +14,8 @@ interface DataProviderInterface
 {
     /**
      * This will be the root class of the data provided by this data provider
+     *
+     * @return class-string
      */
     public function getClass(): string;
 

src/DependencyInjection/Compiler/RegisterFilesystemPass.php

@@ -33,6 +33,8 @@ public function process(ContainerBuilder $container): void
 
         foreach (self::PARAMETERS as $parameter) {
             $parameterValue = $container->getParameter($parameter);
+            Assert::string($parameterValue);
+
             if (!$container->hasDefinition($parameterValue)) {
                 throw new InvalidArgumentException(sprintf('No service definition exists with id "%s"', $parameterValue));
             }

src/DependencyInjection/Compiler/ValidateDataProvidersPass.php

@@ -23,6 +23,7 @@ public function process(ContainerBuilder $container): void
                 throw new InvalidArgumentException(sprintf('The service %s needs the code attribute. Something like this: <tag name="setono_sylius_feed.data_provider" code="insert code here"/>', $id));
             }
 
+            /** @var array{code?: string} $attributes */
             foreach ($tagged as $attributes) {
                 if (!isset($attributes['code'])) {
                     throw new InvalidArgumentException(sprintf('The service %s needs the code attribute. Something like this: <tag name="setono_sylius_feed.data_provider" code="insert code here"/>', $id));

src/DependencyInjection/SetonoSyliusFeedExtension.php

@@ -18,6 +18,13 @@ final class SetonoSyliusFeedExtension extends AbstractResourceExtension implemen
 {
     public function load(array $configs, ContainerBuilder $container): void
     {
+        /**
+         * @var array{
+         *     driver: string,
+         *     storage: array{feed: string, feed_tmp: string},
+         *     resources: array
+         * } $config
+         */
         $config = $this->processConfiguration($this->getConfiguration([], $container), $configs);
         $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
 

src/Doctrine/ORM/FeedRepository.php

@@ -13,12 +13,16 @@ class FeedRepository extends EntityRepository implements FeedRepositoryInterface
 {
     public function findOneByCode(string $code): ?FeedInterface
     {
-        return $this->createQueryBuilder('o')
+        $result = $this->createQueryBuilder('o')
             ->where('o.code = :code')
             ->setParameter('code', $code)
             ->getQuery()
             ->getOneOrNullResult()
         ;
+
+        Assert::nullOrIsInstanceOf($result, FeedInterface::class);
+
+        return $result;
     }
 
     public function findEnabled(): array
@@ -29,6 +33,7 @@ public function findEnabled(): array
             ->getResult()
         ;
 
+        Assert::isArray($res);
         Assert::allIsInstanceOf($res, FeedInterface::class);
 
         return $res;

src/Doctrine/ORM/ViolationRepository.php

@@ -19,8 +19,6 @@ public function findCountsGroupedBySeverity($feed = null): array
             $feed = (int) $feed->getId();
         }
 
-        Assert::nullOrInteger($feed);
-
         $qb = $this->createQueryBuilder('o')
             ->select('NEW Setono\SyliusFeedPlugin\DTO\SeverityCount(o.severity, count(o))')
             ->groupBy('o.severity')
@@ -32,6 +30,7 @@ public function findCountsGroupedBySeverity($feed = null): array
         }
 
         $res = $qb->getQuery()->getResult();
+        Assert::isArray($res);
         Assert::allIsInstanceOf($res, SeverityCount::class);
 
         return $res;

src/EventListener/Filter/AbstractFilterListener.php

@@ -15,6 +15,9 @@ public function __construct(private readonly string $class)
     {
     }
 
+    /**
+     * @param list<class-string> $additionalInstanceChecks
+     */
     protected function isEligible(QueryBuilderEvent $event, array $additionalInstanceChecks = []): bool
     {
         $class = $event->getDataProvider()->getClass();
@@ -39,7 +42,7 @@ protected function getAlias(QueryBuilder $queryBuilder): string
             throw new InvalidArgumentException('This filter only works with one root alias');
         }
 
-        return reset($aliases);
+        return $aliases[0];
     }
 
     protected function getClassMetadata(QueryBuilderEvent $event): ClassMetadata