floor12 / yii2-module-backup

tests/unit/BackupCreateTest.php

Braces +3 added lines, -2 removed lines

@@ -101,8 +101,9 @@
     {
         $config_id = 'backup_test_folder';
         $creator = new BackupCreate($config_id);
-        for ($i = 0; $i < 5; $i++)
-            $creator->run();
+        for ($i = 0; $i < 5; $i++) {
+                    $creator->run();
+        }
     }
 
 

src/Module.php

Braces +9 added lines, -6 removed lines

@@ -81,14 +81,16 @@ 
         $this->backupRootPath = Yii::getAlias($this->backupFolder);
 
         try {
-            if (!file_exists($this->backupRootPath))
-                mkdir($this->backupRootPath);
+            if (!file_exists($this->backupRootPath)) {
+                            mkdir($this->backupRootPath);
+            }
         } catch (ErrorException $e) {
             throw new ErrorException("Backup folder not exists. Its impossible to create it because of permission error.");
         }
 
-        if (!is_writable($this->backupRootPath))
-            throw new ErrorException("Backup folder is not writable.");
+        if (!is_writable($this->backupRootPath)) {
+                    throw new ErrorException("Backup folder is not writable.");
+        }
 
         $this->checkSqliteDb();
         $this->registerTranslations();
@@ -129,9 +131,10 @@ 
      */
     public function checkConfig(string $config_id)
     {
-        foreach (Yii::$app->getModule('backup')->configs as $config)
-            if ($config['id'] == $config_id)
+        foreach (Yii::$app->getModule('backup')->configs as $config) {
+                    if ($config['id'] == $config_id)
                 return true;
+        }
         return false;
     }
 

src/logic/DatabaseBackuper.php

Braces +3 added lines, -2 removed lines

@@ -47,8 +47,9 @@
      */
     public function backup()
     {
-        if (file_exists($this->backupFilePath))
-            throw new Exception("Backup file exists.");
+        if (file_exists($this->backupFilePath)) {
+                    throw new Exception("Backup file exists.");
+        }
         $this->dbProcessor->backup();
     }
 

src/logic/FolderBackupRestorer.php

Braces +6 added lines, -4 removed lines

@@ -25,11 +25,13 @@
      */
     public function __construct(string $backupFilePath, string $targetFolder, $io = IOPriority::IDLE)
     {
-        if (!file_exists($backupFilePath))
-            throw new Exception("Backup file don`t exist..");
+        if (!file_exists($backupFilePath)) {
+                    throw new Exception("Backup file don`t exist..");
+        }
 
-        if (!file_exists($targetFolder))
-            throw new Exception("Target folder don`t exist.");
+        if (!file_exists($targetFolder)) {
+                    throw new Exception("Target folder don`t exist.");
+        }
 
         $this->backupFilePath = $backupFilePath;
         $this->targetFolder = $targetFolder;

src/logic/BackupCreate.php

Braces +18 added lines, -12 removed lines

@@ -61,8 +61,9 @@ 
     protected function loadConfigs()
     {
         $this->configs = Yii::$app->getModule('backup')->configs;
-        if (!is_array($this->configs) || !sizeof($this->configs))
-            throw new ModuleNotConfiguredException('Backup configs is empty');
+        if (!is_array($this->configs) || !sizeof($this->configs)) {
+                    throw new ModuleNotConfiguredException('Backup configs is empty');
+        }
     }
 
     /**
@@ -71,8 +72,9 @@ 
      */
     protected function setUpActiveConfig()
     {
-        if (!is_array($this->configs[$this->config_id]))
-            throw new ConfigurationNotFoundException("Configuration `{$this->config_id}` not found.");
+        if (!is_array($this->configs[$this->config_id])) {
+                    throw new ConfigurationNotFoundException("Configuration `{$this->config_id}` not found.");
+        }
         $this->currentConfig = $this->configs[$this->config_id];
     }
 
@@ -87,8 +89,9 @@ 
         $this->deleteOldFiles();
         $this->createBackupItem();
 
-        if (empty($this->currentConfig['io']))
-            $this->currentConfig['io'] = IOPriority::IDLE;
+        if (empty($this->currentConfig['io'])) {
+                    $this->currentConfig['io'] = IOPriority::IDLE;
+        }
         try {
             if ($this->currentConfig['type'] == BackupType::DB) {
                 $connection = Yii::$app->{$this->currentConfig['connection']};
@@ -114,17 +117,19 @@ 
      */
     protected function deleteOldFiles()
     {
-        if (!isset($this->currentConfig['limit']) || empty($this->currentConfig['limit']))
-            return false;
+        if (!isset($this->currentConfig['limit']) || empty($this->currentConfig['limit'])) {
+                    return false;
+        }
         $backups = Backup::find()
             ->where(['config_id' => $this->config_id])
             ->orderBy('date DESC')
             ->offset($this->currentConfig['limit'] - 1)
             ->all();
 
-        if ($backups)
-            foreach ($backups as $backup)
+        if ($backups) {
+                    foreach ($backups as $backup)
                 $backup->delete();
+        }
     }
 
     /**
@@ -152,8 +157,9 @@ 
             $extension = Backup::EXT_TGZ;
             /** @var Connection $connection */
             $connection = Yii::$app->{$this->currentConfig['connection']};
-            if ($connection->driverName == 'pgsql')
-                $extension = Backup::EXT_DUMP;
+            if ($connection->driverName == 'pgsql') {
+                            $extension = Backup::EXT_DUMP;
+            }
         }
         $date = date("Y-m-d_H-i-s");
         $rand = substr(md5(rand(0, 9999)), 0, 3);

src/logic/DatabaseBackupRestorer.php

Braces +3 added lines, -2 removed lines

@@ -32,8 +32,9 @@
      */
     public function __construct(string $backupFilePath, Connection $connection)
     {
-        if (file_exists($backupFilePath))
-            throw new Exception("Backup file exists.");
+        if (file_exists($backupFilePath)) {
+                    throw new Exception("Backup file exists.");
+        }
 
         $currentProcessorClassname = $this->dbProcessors[$connection->driverName];
         $this->dbProcessor = new $currentProcessorClassname($backupFilePath, $connection);

src/logic/processors/MysqlProcessor.php

Braces +6 added lines, -4 removed lines

@@ -25,8 +25,9 @@ 
         $this->checkBinary($ionicePath);
         $command = "{$ionicePath} -c{$this->io} {$mysqldumpPath} -h {$this->host} -P {$this->port} -u {$this->username}  -p{$this->password}  {$this->database} | {$gzipPath} -9 -c > {$this->backupFilePath}";
         (exec($command));
-        if (!file_exists($this->backupFilePath) || filesize($this->backupFilePath) < 100)
-            throw new MysqlDumpException();
+        if (!file_exists($this->backupFilePath) || filesize($this->backupFilePath) < 100) {
+                    throw new MysqlDumpException();
+        }
     }
 
     public function restore()
@@ -37,7 +38,8 @@ 
         $this->checkBinary($mysqlPath);
         $command = "{$zcatPath} {$this->backupFilePath} | {$mysqlPath} -h {$this->host} -P {$this->port} -u {$this->username} -p{$this->password} {$this->database}";
         exec($command, $return);
-        if (!empty($return))
-            throw new PostgresDumpException();
+        if (!empty($return)) {
+                    throw new PostgresDumpException();
+        }
     }
 }

src/logic/processors/DbProcessor.php

Braces +9 added lines, -6 removed lines

@@ -58,13 +58,15 @@ 
         $dsn = substr($dsn, strpos($dsn, ':') + 1);
         $dsnParts = explode(';', $dsn);
 
-        if (empty($dsnParts))
-            throw new DsnParseException();
+        if (empty($dsnParts)) {
+                    throw new DsnParseException();
+        }
 
         foreach ($dsnParts as $part) {
             $row = explode('=', $part);
-            if (isset($row[0]) && isset($row[1]))
-                $this->parsedDsn[$row[0]] = $row[1];
+            if (isset($row[0]) && isset($row[1])) {
+                            $this->parsedDsn[$row[0]] = $row[1];
+            }
         }
 
         $this->host = $this->parsedDsn['host'] ?: null;
@@ -74,8 +76,9 @@ 
 
     protected function checkBinary(string $binary)
     {
-        if (!file_exists($binary) || !is_executable($binary))
-            throw new BinaryNotFoundException($binary);
+        if (!file_exists($binary) || !is_executable($binary)) {
+                    throw new BinaryNotFoundException($binary);
+        }
     }
 
     public function backup()

src/logic/processors/PostgresProcessor.php

Braces +6 added lines, -4 removed lines

@@ -15,8 +15,9 @@ 
         $ionicePath = $this->module->binaries['ionice'];
         $command = "PGPASSWORD='{$this->password}' {$ionicePath} -c{$this->io} {$binaryPath} -h {$this->host} -p {$this->port} -U {$this->username} -Fc -Z1 {$this->database} -f {$this->backupFilePath}";
         exec($command, $return);
-        if (!empty($return))
-            throw new PostgresDumpException();
+        if (!empty($return)) {
+                    throw new PostgresDumpException();
+        }
     }
 
     public function restore()
@@ -24,8 +25,9 @@ 
         $binaryPath = $this->module->binaries['pg_restore'];
         $command = "PGPASSWORD='{$this->password}' {$binaryPath} -c -Fc -j 4 -h {$this->host} -p {$this->port} -U {$this->username}  -d {$this->database} {$this->backupFilePath}";
         exec($command, $return);
-        if (!empty($return))
-            throw new PostgresDumpException();
+        if (!empty($return)) {
+                    throw new PostgresDumpException();
+        }
     }
 
     public function init()