Merge pull request #395 from norkunas/batch

Fix some phpstan warnings

Author: norkunas

.github/workflows/ci.yml

@@ -52,7 +52,7 @@ jobs:
           composer-options: "--optimize-autoloader"
 
       - name: "Run PHP-CS-Fixer"
-        run: vendor/bin/php-cs-fixer fix -v --dry-run --allow-unsupported-php-version=yes --using-cache=no --format=checkstyle | cs2pr
+        run: vendor/bin/php-cs-fixer fix -v --dry-run --using-cache=no --format=checkstyle | cs2pr
 
   phpunit:
     name: PHPUnit (PHP ${{ matrix.php }}) (Symfony ${{ matrix.sf_version }})

.php-cs-fixer.php .php-cs-fixer.dist.php.php-cs-fixer.php renamed to .php-cs-fixer.dist.php

@@ -7,10 +7,11 @@
 ;
 
 return (new PhpCsFixer\Config())
+    ->setFinder($finder)
+    ->setUnsupportedPhpVersionAllowed(true)
     ->setRules([
         '@Symfony' => true,
         'no_superfluous_phpdoc_tags' => false,
         'phpdoc_to_comment' => ['ignored_tags' => ['var']], // phpstan errors pops up without this
     ])
-    ->setFinder($finder)
 ;

composer.json

@@ -25,8 +25,8 @@
   "require-dev": {
     "doctrine/doctrine-bundle": "^2.18 || ^3.0",
     "doctrine/orm": "^2.20 || ^3.0",
-    "fakerphp/faker": "^1.24",
-    "friendsofphp/php-cs-fixer": "^3.92",
+    "fakerphp/faker": "^1.24.1",
+    "friendsofphp/php-cs-fixer": "^3.92.4",
     "geocoder-php/algolia-places-provider": "^0.5",
     "geocoder-php/arcgis-online-provider": "^4.5",
     "geocoder-php/bing-maps-provider": "^4.4",
@@ -53,11 +53,11 @@
     "geocoder-php/pickpoint-provider": "^4.4",
     "geocoder-php/tomtom-provider": "^4.5",
     "geocoder-php/yandex-provider": "^4.6",
-    "nyholm/nsa": "^1.3",
-    "nyholm/psr7": "^1.8",
+    "nyholm/nsa": "^1.3.1",
+    "nyholm/psr7": "^1.8.2",
     "nyholm/symfony-bundle-test": "^3.1",
-    "phpstan/phpstan": "^2.1",
-    "psr/http-client": "^1.0",
+    "phpstan/phpstan": "^2.1.33",
+    "psr/http-client": "^1.0.3",
     "psr/simple-cache": "^1.0 || ^2.0 || ^3.0",
     "symfony/cache": "^6.4 || ^7.0 || ^8.0",
     "symfony/config": "^6.4 || ^7.0 || ^8.0",

src/DependencyInjection/Compiler/AddProvidersPass.php

@@ -20,7 +20,7 @@
 /**
  * @author William Durand <william.durand1@gmail.com>
  */
-class AddProvidersPass implements CompilerPassInterface
+final class AddProvidersPass implements CompilerPassInterface
 {
     /**
      * Get all providers based on their tag (`bazinga_geocoder.provider`) and

src/DependencyInjection/Compiler/FactoryValidatorPass.php

@@ -21,7 +21,7 @@
  *
  * @author Tobias Nyholm <tobias.nyholm@gmail.com>
  */
-class FactoryValidatorPass implements CompilerPassInterface
+final class FactoryValidatorPass implements CompilerPassInterface
 {
     /**
      * @var list<non-empty-string>

src/DependencyInjection/Compiler/ProfilerPass.php

@@ -22,7 +22,7 @@
  *
  * @author Tobias Nyholm <tobias.nyholm@gmail.com>
  */
-class ProfilerPass implements CompilerPassInterface
+final class ProfilerPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {

src/DependencyInjection/Configuration.php

@@ -12,6 +12,7 @@
 
 namespace Bazinga\GeocoderBundle\DependencyInjection;
 
+use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
@@ -42,7 +43,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                 ->addDefaultsIfNotSet()
                 ->treatFalseLike(['enabled' => false])
                 ->treatTrueLike(['enabled' => true])
-                ->treatNullLike(['enabled' => $this->debug])
+                ->treatNullLike(['enabled' => \class_exists(DoctrineBundle::class)])
                 ->info('Extend the debug profiler with information about requests.')
                 ->children()
                     ->booleanNode('enabled')
@@ -54,7 +55,7 @@ public function getConfigTreeBuilder(): TreeBuilder
             ->arrayNode('fake_ip')
                 ->beforeNormalization()
                 ->ifString()
-                    ->then(function ($value) {
+                    ->then(function (string $value): array {
                         return ['ip' => $value];
                     })
                 ->end()
@@ -132,7 +133,7 @@ private function createClientPluginNode(): ArrayNodeDefinition
         $pluginList
             // support having just a service id in the list
             ->beforeNormalization()
-                ->always(function ($plugin) {
+                ->always(function (array|string $plugin) {
                     if (is_string($plugin)) {
                         return [
                             'reference' => [

src/Plugin/ProfilingPlugin.php

@@ -43,12 +43,12 @@ public function handleQuery(Query $query, callable $next, callable $first): Prom
     {
         $startTime = microtime(true);
 
-        return $next($query)->then(function (Collection $result) use ($query, $startTime) {
+        return $next($query)->then(function (Collection $result) use ($query, $startTime): Collection {
             $duration = (microtime(true) - $startTime) * 1000;
             $this->logQuery($query, $duration, $result);
 
             return $result;
-        }, function (Exception $exception) use ($query, $startTime) {
+        }, function (Exception $exception) use ($query, $startTime): void {
             $duration = (microtime(true) - $startTime) * 1000;
             $this->logQuery($query, $duration, $exception);
 

tests/Command/GeocodeCommandTest.php

@@ -30,7 +30,7 @@
  */
 final class GeocodeCommandTest extends TestCase
 {
-    private static $address = '10 rue Gambetta, Paris, France';
+    private static string $address = '10 rue Gambetta, Paris, France';
 
     public function testExecute(): void
     {

tests/DependencyInjection/ConfigurationTest.php

@@ -25,6 +25,7 @@ final class ConfigurationTest extends TestCase
     public function testGetConfigTreeBuilder(): void
     {
         $config = Yaml::parseFile(__DIR__.'/Fixtures/config.yml');
+        self::assertIsArray($config);
 
         $configuration = new Configuration(true);
         $treeBuilder = $configuration->getConfigTreeBuilder();
@@ -41,6 +42,7 @@ public function testGetConfigTreeBuilder(): void
     public function testGetConfigTreeBuilderNoDebug(): void
     {
         $config = Yaml::parseFile(__DIR__.'/Fixtures/config.yml');
+        self::assertIsArray($config);
 
         $configuration = new Configuration(false);
         $treeBuilder = $configuration->getConfigTreeBuilder();