#12 Apply phpstan level 10

Author: njoubert-cleverage

Diff for: phpstan.neon

@@ -1,5 +1,5 @@
 parameters:
-    level: 6
+    level: 10
     paths:
         - src
         - tests

Diff for: src/Task/FileFetchTask.php

@@ -51,8 +51,12 @@ public function initialize(ProcessState $state): void
         // Configure options
         parent::initialize($state);
 
-        $this->sourceFS = $this->storages->get($this->getOption($state, 'source_filesystem'));
-        $this->destinationFS = $this->storages->get($this->getOption($state, 'destination_filesystem'));
+        /** @var string $sourceFilesystemOption */
+        $sourceFilesystemOption = $this->getOption($state, 'source_filesystem');
+        $this->sourceFS = $this->storages->get($sourceFilesystemOption);
+        /** @var string $destinationFilesystemOption */
+        $destinationFilesystemOption = $this->getOption($state, 'destination_filesystem');
+        $this->destinationFS = $this->storages->get($destinationFilesystemOption);
     }
 
     /**
@@ -71,7 +75,9 @@ public function execute(ProcessState $state): void
             return;
         }
 
-        $this->doFileCopy($state, $file, $this->getOption($state, 'remove_source'));
+        /** @var bool $removeSourceOption */
+        $removeSourceOption = $this->getOption($state, 'remove_source');
+        $this->doFileCopy($state, $file, $removeSourceOption);
         $state->setOutput($file);
     }
 
@@ -94,16 +100,19 @@ public function next(ProcessState $state): bool
      */
     protected function findMatchingFiles(ProcessState $state): void
     {
+        /** @var ?string $filePattern */
         $filePattern = $this->getOption($state, 'file_pattern');
         if ($filePattern) {
             foreach ($this->sourceFS->listContents('/') as $file) {
-                if ('file' === $file['type']
-                    && preg_match($filePattern, (string) $file['path'])
-                    && !\in_array($file['path'], $this->matchingFiles, true)) {
-                    $this->matchingFiles[] = $file['path'];
+                if ('file' === $file->type()
+                    && preg_match($filePattern, $file->path())
+                    && !\in_array($file->path(), $this->matchingFiles, true)
+                ) {
+                    $this->matchingFiles[] = $file->path();
                 }
             }
         } else {
+            /** @var array<string>|string|null $input */
             $input = $state->getInput();
             if (!$input) {
                 throw new \UnexpectedValueException('No pattern neither input provided for the Task');

Diff for: src/Task/ListContentTask.php

@@ -53,10 +53,13 @@ protected function configureOptions(OptionsResolver $resolver): void
     public function execute(ProcessState $state): void
     {
         if (null === $this->fsContent || null === key($this->fsContent)) {
-            $filesystem = $this->storages->get($this->getOption($state, 'filesystem'));
-            $pattern = $this->getOption($state, 'file_pattern');
+            /** @var string $filesystemOption */
+            $filesystemOption = $this->getOption($state, 'filesystem');
+            $filesystem = $this->storages->get($filesystemOption);
+            /** @var ?string $patternOption */
+            $patternOption = $this->getOption($state, 'file_pattern');
 
-            $this->fsContent = $this->getFilteredFilesystemContents($filesystem, $pattern);
+            $this->fsContent = $this->getFilteredFilesystemContents($filesystem, $patternOption);
         }
 
         if (null === key($this->fsContent)) {
@@ -87,7 +90,7 @@ protected function getFilteredFilesystemContents(FilesystemOperator $filesystem,
     {
         $results = [];
         foreach ($filesystem->listContents('') as $item) {
-            if (null === $pattern || preg_match($pattern, (string) $item['path'])) {
+            if (null === $pattern || preg_match($pattern, $item->path())) {
                 $results[] = $item;
             }
         }

Diff for: src/Task/RemoveFileTask.php

@@ -41,7 +41,10 @@ protected function configureOptions(OptionsResolver $resolver): void
 
     public function execute(ProcessState $state): void
     {
-        $filesystem = $this->storages->get($this->getOption($state, 'filesystem'));
+        /** @var string $filesystemOption */
+        $filesystemOption = $this->getOption($state, 'filesystem');
+        $filesystem = $this->storages->get($filesystemOption);
+        /** @var string $filePath */
         $filePath = $state->getInput();
 
         try {