PHP 8.2 fix for dynamic property $exclude (#653)

* PHP 8.2 fix for dynamic property $exclude

* Add phpdoc value type for iterable type array

* formatting

---------

Co-authored-by: Danny Herpol <hdcore@lachjekrom.com>
Co-authored-by: Chris Gmyr <cmgmyr@gmail.com>

src/Domain/InsightLoader/SniffLoader.php

@@ -28,10 +28,18 @@ public function load(string $insightClass, string $dir, array $config, Collector
         /** @var SniffContract $sniff */
         $sniff = new $insightClass();
 
+        $excludeConfig = [];
+
+        if (isset($config['exclude'])) {
+            /** @var array<string> $excludeConfig */
+            $excludeConfig = $config['exclude'];
+            unset($config['exclude']);
+        }
+
         foreach ($config as $property => $value) {
             $sniff->{$property} = $value;
         }
 
-        return new SniffDecorator($sniff, $dir);
+        return new SniffDecorator($sniff, $dir, $excludeConfig);
     }
 }

src/Domain/Insights/SniffDecorator.php

@@ -34,13 +34,18 @@ final class SniffDecorator implements Sniff, Insight, DetailsCarrier, Fixable
      */
     private array $excludedFiles;
 
-    public function __construct(Sniff $sniff, string $dir)
+    /**
+     * SniffDecorator constructor.
+     *
+     * @param array<string> $exclude
+     */
+    public function __construct(Sniff $sniff, string $dir, array $exclude)
     {
         $this->sniff = $sniff;
         $this->excludedFiles = [];
 
-        if ($this->getIgnoredFilesPath() !== []) {
-            $this->excludedFiles = Files::find($dir, $this->getIgnoredFilesPath());
+        if ($exclude !== []) {
+            $this->excludedFiles = Files::find($dir, $exclude);
         }
     }
 
@@ -131,14 +136,4 @@ private function skipFilesFromIgnoreFiles(InsightFile $file): bool
             $this->excludedFiles
         );
     }
-
-    /**
-     * Contains the setting for all files which the sniff should ignore.
-     *
-     * @return array<int, string>
-     */
-    private function getIgnoredFilesPath(): array
-    {
-        return $this->sniff->exclude ?? [];
-    }
 }