#185 - Run quality tools

Author: xaviermarchegay

src/CleverAgeProcessBundle.php

@@ -34,6 +34,7 @@ public function build(ContainerBuilder $container): void
         $container->addCompilerPass(new CheckSerializerCompilerPass());
     }
 
+    #[\Override]
     public function getPath(): string
     {
         return \dirname(__DIR__);

src/Configuration/ProcessConfiguration.php

@@ -112,14 +112,7 @@ public function getDependencyGroups(): array
         if (null === $this->dependencyGroups) {
             $this->dependencyGroups = [];
             foreach ($this->getTaskConfigurations() as $taskConfiguration) {
-                $isInBranch = false;
-                foreach ($this->dependencyGroups as $branch) {
-                    if (\in_array($taskConfiguration->getCode(), $branch, true)) {
-                        $isInBranch = true;
-                        break;
-                    }
-                }
-
+                $isInBranch = array_any($this->dependencyGroups, static fn ($branch) => \in_array($taskConfiguration->getCode(), $branch, true));
                 if (!$isInBranch) {
                     $dependencies = $this->buildDependencies($taskConfiguration);
                     $dependencies = $this->sortDependencies($dependencies);

src/Configuration/TaskConfiguration.php

@@ -112,9 +112,7 @@ public function getOutputs(): array
         return $this->outputs;
     }
 
-    /**
-     * @deprecated Use getErrorOutputs method instead
-     */
+    #[\Deprecated(message: 'Use getErrorOutputs method instead')]
     public function getErrors(): array
     {
         @trigger_error('Deprecated method, use getErrorOutputs instead', \E_USER_DEPRECATED);

src/Filesystem/CsvFile.php

@@ -56,11 +56,13 @@ public function __construct(
     /**
      * Will return a resource if the file was created using a resource.
      */
+    #[\Override]
     public function getFilePath(): string
     {
         return $this->filePath;
     }
 
+    #[\Override]
     protected function getResourceName(): string
     {
         return "CSV file '{$this->filePath}'";

src/Logger/TaskProcessor.php

@@ -17,6 +17,7 @@
 
 class TaskProcessor extends AbstractProcessor
 {
+    #[\Override]
     public function __invoke(LogRecord $record): LogRecord
     {
         $record = parent::__invoke($record);

src/Logger/TransformerProcessor.php

@@ -17,6 +17,7 @@
 
 class TransformerProcessor extends AbstractProcessor
 {
+    #[\Override]
     public function __invoke(LogRecord $record): LogRecord
     {
         $record = parent::__invoke($record);

src/Task/File/Csv/AbstractCsvTask.php

@@ -24,6 +24,7 @@
  */
 abstract class AbstractCsvTask extends AbstractCsvResourceTask
 {
+    #[\Override]
     protected function initFile(ProcessState $state): void
     {
         if ($this->csv instanceof CsvResource) {
@@ -41,6 +42,7 @@ protected function initFile(ProcessState $state): void
         );
     }
 
+    #[\Override]
     protected function configureOptions(OptionsResolver $resolver): void
     {
         parent::configureOptions($resolver);

src/Task/File/Csv/CsvReaderTask.php

@@ -82,6 +82,7 @@ protected function getHeaders(ProcessState $state, array $options): ?array
         return $options['headers'];
     }
 
+    #[\Override]
     protected function configureOptions(OptionsResolver $resolver): void
     {
         parent::configureOptions($resolver);

src/Task/File/Csv/CsvSplitterTask.php

@@ -23,6 +23,7 @@
  */
 class CsvSplitterTask extends InputCsvReaderTask
 {
+    #[\Override]
     public function execute(ProcessState $state): void
     {
         $options = $this->getOptions($state);
@@ -49,6 +50,7 @@ public function execute(ProcessState $state): void
      * return true if the task has a next element
      * return false if the task has terminated it's iteration.
      */
+    #[\Override]
     public function next(ProcessState $state): bool
     {
         if (!$this->csv instanceof CsvResource) {
@@ -64,6 +66,7 @@ public function next(ProcessState $state): bool
         return !$endOfFile;
     }
 
+    #[\Override]
     public function finalize(ProcessState $state): void
     {
         if ($this->csv instanceof CsvResource) {
@@ -100,6 +103,7 @@ protected function splitCsv(CsvResource $csv, int $maxLines): string
         return $tmpFilePath;
     }
 
+    #[\Override]
     protected function configureOptions(OptionsResolver $resolver): void
     {
         parent::configureOptions($resolver);

src/Task/File/Csv/CsvWriterTask.php

@@ -43,6 +43,7 @@ public function proceed(ProcessState $state): void
         $state->setOutput($this->csv->getFilePath());
     }
 
+    #[\Override]
     protected function configureOptions(OptionsResolver $resolver): void
     {
         parent::configureOptions($resolver);