floor12 / yii2-module-backup

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()

src/logic/BackupRestore.php

Braces +9 added lines, -6 removed lines

@@ -53,8 +53,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');
+        }
     }
 
     /**
@@ -63,8 +64,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];
     }
 
@@ -76,8 +78,9 @@ 
      */
     public function run()
     {
-        if (empty($this->currentConfig['io']))
-            $this->currentConfig['io'] = IOPriority::IDLE;
+        if (empty($this->currentConfig['io'])) {
+                    $this->currentConfig['io'] = IOPriority::IDLE;
+        }
 
         if ($this->currentConfig['type'] == BackupType::DB) {
             $this->connection = Yii::$app->{$this->currentConfig['connection']};

src/logic/FolderBackupMaker.php

Braces +12 added lines, -8 removed lines

@@ -35,11 +35,13 @@ 
      */
     public function __construct(string $backupFilePath, string $targetFolder, $io = IOPriority::IDLE)
     {
-        if (file_exists($backupFilePath))
-            throw new Exception("Backup file exists.");
+        if (file_exists($backupFilePath)) {
+                    throw new Exception("Backup file exists.");
+        }
 
-        if (!file_exists($targetFolder))
-            throw new Exception("Target folder not exists.");
+        if (!file_exists($targetFolder)) {
+                    throw new Exception("Target folder not exists.");
+        }
 
         $this->module = Yii::$app->getModule('backup');
         $this->backupFilePath = $backupFilePath;
@@ -58,11 +60,13 @@ 
         $zipPath = $this->module->binaries['zip'];
         $command = "cd {$this->targetFolder} && {$ionicePath} -c{$this->io} {$zipPath} -r -0 {$this->backupFilePath} * > /dev/null";
         exec($command);
-        if (!file_exists($this->backupFilePath))
-            throw new FolderDumpException();
+        if (!file_exists($this->backupFilePath)) {
+                    throw new FolderDumpException();
+        }
 
-        if ($this->module->chmod)
-            chmod($this->backupFilePath, $this->module->chmod);
+        if ($this->module->chmod) {
+                    chmod($this->backupFilePath, $this->module->chmod);
+        }
 
         return true;
     }

src/logic/BackupImporter.php

Braces +9 added lines, -6 removed lines

@@ -34,14 +34,17 @@
     {
         $this->module = Yii::$app->getModule('backup');
 
-        if (!file_exists($absoluteFilePath))
-            throw new FileNotFoundException();
+        if (!file_exists($absoluteFilePath)) {
+                    throw new FileNotFoundException();
+        }
 
-        if (!is_array($this->module->configs) || empty($this->module->configs))
-            throw new ModuleNotConfiguredException();
+        if (!is_array($this->module->configs) || empty($this->module->configs)) {
+                    throw new ModuleNotConfiguredException();
+        }
 
-        if (!is_array($this->module->configs[$config_id]))
-            throw new ConfigurationNotFoundException();
+        if (!is_array($this->module->configs[$config_id])) {
+                    throw new ConfigurationNotFoundException();
+        }
 
         $this->currentConfig = $this->module->configs[$config_id];
         $this->model = new Backup();

src/controllers/ConsoleController.php

Braces +14 added lines, -10 removed lines

@@ -52,8 +52,9 @@ 
     public function actionRestore(string $backup_id)
     {
         $model = Backup::findOne((int)$backup_id);
-        if (!$model)
-            throw new ErrorException('Backup not found.');
+        if (!$model) {
+                    throw new ErrorException('Backup not found.');
+        }
 
         Yii::createObject(BackupRestore::class, [$model])->run();
         $this->stdout('Backup restored.' . PHP_EOL, Console::FG_GREEN);
@@ -69,14 +70,16 @@ 
     {
         $models = Backup::find()->orderBy('id DESC')->all();
 
-        if (empty($models))
-            return $this->stderr('Backups not found.' . PHP_EOL, Console::FG_YELLOW);
+        if (empty($models)) {
+                    return $this->stderr('Backups not found.' . PHP_EOL, Console::FG_YELLOW);
+        }
 
-        foreach ($models as $model)
-            $this->stdout("{$model->id}: " . Yii::$app->formatter->asDatetime($model->date) . "\t\t{$model->config_id}\t\t" .
+        foreach ($models as $model) {
+                    $this->stdout("{$model->id}: " . Yii::$app->formatter->asDatetime($model->date) . "\t\t{$model->config_id}\t\t" .
                 BackupType::$list[$model->type] .
                 PHP_EOL,
                 $model->status ? Console::FG_GREEN : Console::FG_RED);
+        }
     }
 
     /**
@@ -89,9 +92,10 @@ 
     public function actionImport(string $config_id, string $absoluteFilePath)
     {
         $importer = new BackupImporter($config_id, $absoluteFilePath);
-        if ($importer->import())
-            $this->stdout("Backup imported: {$absoluteFilePath}\n", Console::FG_GREEN);
-        else
-            $this->stdout("Something went wrong.\n", Console::FG_RED);
+        if ($importer->import()) {
+                    $this->stdout("Backup imported: {$absoluteFilePath}\n", Console::FG_GREEN);
+        } else {
+                    $this->stdout("Something went wrong.\n", Console::FG_RED);
+        }
     }
 }

src/controllers/AdminController.php

Braces +3 added lines, -2 removed lines

@@ -148,8 +148,9 @@
     protected function getBackup(int $id)
     {
         $this->model = Backup::findOne($id);
-        if (!$this->model)
-            throw new NotFoundHttpException(Yii::t('app.f12.backup', 'Backup is not found.'));
+        if (!$this->model) {
+                    throw new NotFoundHttpException(Yii::t('app.f12.backup', 'Backup is not found.'));
+        }
     }
 
 }