Merge pull request #66 from TomasVotruba/tv-fix-skip

[fix] fix bug when --skip-path also ignores usages in those paths

Author: TomasVotruba

composer.json

@@ -8,11 +8,11 @@
     ],
     "require": {
         "php": ">=8.2",
-        "illuminate/container": "^11.44",
+        "illuminate/container": "^12.0",
         "nette/utils": "^4.0",
         "nikic/php-parser": "^5.4",
         "symfony/console": "^6.4",
-        "symfony/finder": "^6.4",
+        "symfony/finder": "^7.2",
         "webmozart/assert": "^1.11"
     },
     "require-dev": {
@@ -65,4 +65,4 @@
             ]
         }
     }
-}
+}

src/Commands/CheckCommand.php

@@ -112,15 +112,19 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         /** @var string[] $fileExtensions */
         $fileExtensions = (array) $input->getOption('file-extension');
 
+        // we have to look for usage in every path
+        $allFilePaths = $this->phpFilesFinder->findPhpFiles($paths, $fileExtensions, []);
+
+        // but we only want to check the files that are not in the skipped paths
         $phpFilePaths = $this->phpFilesFinder->findPhpFiles($paths, $fileExtensions, $pathsToSkip);
 
         $progressBar = null;
         if (! $isJson) {
             $this->symfonyStyle->title('1. Finding used classes');
-            $progressBar = $this->symfonyStyle->createProgressBar(count($phpFilePaths));
+            $progressBar = $this->symfonyStyle->createProgressBar(count($allFilePaths));
         }
 
-        $usedNames = $this->resolveUsedClassNames($phpFilePaths, $progressBar);
+        $usedNames = $this->resolveUsedClassNames($allFilePaths, $progressBar);
 
         $this->symfonyStyle->newLine(2);