floor12 / yii2-module-backup

src/models/Backup.php

Braces +6 added lines, -4 removed lines

@@ -66,8 +66,9 @@ 
      */
     public function getFullPath()
     {
-        if (!$this->filename)
-            return;
+        if (!$this->filename) {
+                    return;
+        }
         return Yii::$app->getModule('backup')->backupRootPath . DIRECTORY_SEPARATOR . $this->filename;
     }
 
@@ -78,8 +79,9 @@ 
     public function updateFileSize()
     {
         $this->size = 0;
-        if (file_exists($this->getFullPath()))
-            $this->size = filesize($this->getFullPath());
+        if (file_exists($this->getFullPath())) {
+                    $this->size = filesize($this->getFullPath());
+        }
     }
 
     /**

src/views/admin/index.php

Spacing +3 added lines, -3 removed lines

@@ -72,12 +72,12 @@ 
         'filename',
         [
             'attribute' => 'status',
-            'content' => function (Backup $model) {
+            'content' => function(Backup $model) {
                 return BackupStatus::getLabel($model->status);
             }
         ],
         [
-            'content' => function (Backup $model) {
+            'content' => function(Backup $model) {
                 $html = ' ';
                 if (file_exists($model->getFullPath()))
                     $html .= Html::tag('span', IconHelper::CHECK, ['title' => 'Файл найден', 'style' => 'color:#38b704;']);
@@ -98,7 +98,7 @@ 
         'size:size',
         [
             'contentOptions' => ['class' => 'text-right'],
-            'content' => function (Backup $model) {
+            'content' => function(Backup $model) {
                 $html = Html::a(IconHelper::PLAY, null, [
                     'class' => 'btn btn-default btn-sm',
                     'title' => Yii::t('app.f12.backup', 'Restore'),

Braces +10 added lines, -8 removed lines

@@ -79,17 +79,19 @@
         [
             'content' => function (Backup $model) {
                 $html = ' ';
-                if (file_exists($model->getFullPath()))
-                    $html .= Html::tag('span', IconHelper::CHECK, ['title' => 'Файл найден', 'style' => 'color:#38b704;']);
-                else
-                    $html .= Html::tag('span', IconHelper::EXCLAMATION, ['title' => 'Файл не найден', 'style' => 'color:#eca70b;']);
+                if (file_exists($model->getFullPath())) {
+                                    $html .= Html::tag('span', IconHelper::CHECK, ['title' => 'Файл найден', 'style' => 'color:#38b704;']);
+                } else {
+                                    $html .= Html::tag('span', IconHelper::EXCLAMATION, ['title' => 'Файл не найден', 'style' => 'color:#eca70b;']);
+                }
 
                 $html .= ' ';
 
-                if (is_writable($model->getFullPath()))
-                    $html .= Html::tag('span', IconHelper::CHECK, ['title' => 'Права на запись найден', 'style' => 'color:#38b704;']);
-                else
-                    $html .= Html::tag('span', IconHelper::EXCLAMATION, ['title' => 'Нет прав на запись', 'style' => 'color:#eca70b;']);
+                if (is_writable($model->getFullPath())) {
+                                    $html .= Html::tag('span', IconHelper::CHECK, ['title' => 'Права на запись найден', 'style' => 'color:#38b704;']);
+                } else {
+                                    $html .= Html::tag('span', IconHelper::EXCLAMATION, ['title' => 'Нет прав на запись', 'style' => 'color:#eca70b;']);
+                }
 
 
                 return $html;

src/controllers/ApiController.php

Braces +9 added lines, -6 removed lines

@@ -101,8 +101,9 @@ 
      */
     public function actionBackup($config_id)
     {
-        if (!Yii::$app->getModule('backup')->checkConfig($config_id))
-            throw new NotFoundHttpException(Yii::t('app.f12.backup', 'Backup config is not found.'));
+        if (!Yii::$app->getModule('backup')->checkConfig($config_id)) {
+                    throw new NotFoundHttpException(Yii::t('app.f12.backup', 'Backup config is not found.'));
+        }
 
         Yii::createObject(BackupCreate::class, [$config_id])->run();
 
@@ -138,8 +139,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.'));
+        }
     }
 
     /**
@@ -150,8 +152,9 @@ 
     {
         $headers = Yii::$app->request->getHeaders();
         $authTokens = Yii::$app->getModule('backup')->authTokens;
-        if (!empty($headers[self::HEADER_NAME]) && in_array($headers[self::HEADER_NAME], $authTokens))
-            return true;
+        if (!empty($headers[self::HEADER_NAME]) && in_array($headers[self::HEADER_NAME], $authTokens)) {
+                    return true;
+        }
         throw new ForbiddenHttpException();
     }
 

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

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
                 $targetPath = Yii::getAlias($this->currentConfig['path']);
                 Yii::createObject(FolderBackupMaker::class, [$this->model->getFullPath(), $targetPath, $this->currentConfig['io']])->execute();
             }
-        } catch (MysqlDumpException|PostgresDumpException|FolderDumpException $exception) {
+        } catch (MysqlDumpException | PostgresDumpException | FolderDumpException $exception) {
             $this->setBackupFailed();
             throw new $exception;
         }

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