chore: fix code style and static analysis

Author: SVillette

composer.json

@@ -42,7 +42,7 @@
         "friendsofphp/php-cs-fixer": "^3.16"
     },
     "scripts": {
-        "cs-fixer": "vendor/bin/php-cs-fixer fix --dry-run",
+        "cs-fixer": "vendor/bin/php-cs-fixer fix --dry-run --diff",
         "analyse": "vendor/bin/phpstan analyse",
         "test": "vendor/bin/phpunit"
     }

src/Command/BackupDatabasesCommand.php

@@ -23,6 +23,7 @@
 
 use function array_splice;
 use function getcwd;
+use function implode;
 use function iterator_to_array;
 use function sprintf;
 use function Symfony\Component\String\u;
@@ -112,12 +113,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 getcwd() :
                 throw new RuntimeException('Unable to get the current directory, check the user permissions')
             ;
-            
+
             $dumpOnlyData = $backup->getStrategy()->getOnlyData();
             $backupTables = $backup->getStrategy()->getTables();
-            
+
             $io->info(sprintf('The backup %s is in progress', $backupName));
-            
+
             foreach ($connection->getDatabases() as $database) {
                 if ($output->isVerbose()) {
                     $io->comment("Backup for $database database has started");
@@ -127,13 +128,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 $filePath = "$backupDirectory/$backupName-$database-$date.sql";
 
                 $process = Process::fromShellCommandline(
-                    \sprintf(
+                    sprintf(
                         '"${:MYSQL_DUMP}" -u "${:DB_USER}" -h "${:DB_HOST}" -P "${:DB_PORT}" %s "${:DB_NAME}" %s > "${:FILEPATH}"',
                         $dumpOnlyData ? '--no-create-info' : '',
-                        \implode(' ', $backupTables)
+                        implode(' ', $backupTables)
                     )
                 );
-                
+
                 $process->setPty(Process::isPtySupported());
                 $process->run(null, [
                     'MYSQL_DUMP' => $mysqldump,
@@ -144,7 +145,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                     'MYSQL_PWD' => $connection->getPassword(),
                     'FILEPATH' => $filePath,
                 ]);
-                
+
                 if (!$process->isSuccessful()) {
                     $message = '' !== $process->getErrorOutput() ? $process->getErrorOutput() : $process->getOutput();
 

src/Factory/Backup/BackupFactory.php

@@ -20,7 +20,7 @@ final class BackupFactory implements NamedFactoryInterface
      */
     public function __construct(
         private readonly ConnectionFactory $connectionFactory,
-        private readonly FactoryInterface $strategyFactory
+        private readonly FactoryInterface $strategyFactory,
     ) {
     }
 

src/Model/Backup/Backup.php

@@ -11,7 +11,7 @@ class Backup
     public function __construct(
         private readonly string $name,
         private readonly Connection $connection,
-        private readonly Strategy $strategy
+        private readonly Strategy $strategy,
     ) {
     }
 

src/Model/Backup/Strategy.php

@@ -7,20 +7,14 @@
 class Strategy
 {
     /**
-     * Default Constructor
-     * 
-     * @param int $maxFiles
-     * @param string $backupDirectory
-     * @param string $dateFormat
-     * @param bool $onlyData
-     * @param array<string> $tables
+     * @param list<string>|null $tables
      */
     public function __construct(
         private readonly ?int $maxFiles = null,
         private readonly ?string $backupDirectory = null,
         private readonly ?string $dateFormat = 'Y-m-d',
         private readonly ?bool $onlyData = false,
-        private readonly ?array $tables = null
+        private readonly ?array $tables = null,
     ) {
     }
 
@@ -38,13 +32,13 @@ public function getDateFormat(): ?string
     {
         return $this->dateFormat;
     }
-    
+
     public function getOnlyData(): ?bool
     {
         return $this->onlyData;
     }
-    
-    /** @return array<string> */
+
+    /** @return list<string>|null */
     public function getTables(): ?array
     {
         return $this->tables;

src/Model/Connection/ConnectionDriver.php

@@ -14,7 +14,7 @@ enum ConnectionDriver: string
     public function getConnectionClass(): string
     {
         return match ($this) {
-            self::MySQL => MySQLConnection::class
+            self::MySQL => MySQLConnection::class,
         };
     }
 }

src/Model/Connection/MySQLConnection.php

@@ -14,7 +14,7 @@ public function __construct(
         private readonly ?string $password = null,
         private readonly ?string $host = '127.0.0.1',
         private readonly ?int $port = 3306,
-        private readonly array $databases = []
+        private readonly array $databases = [],
     ) {
     }