floor12 / yii2-module-backup

src/logic/BackupRestore.php

Braces +9 added lines, -6 removed lines

@@ -44,16 +44,19 @@
     {
         $this->_configs = Yii::$app->getModule('backup')->configs;
 
-        if (!is_array($this->_configs) || !sizeof($this->_configs))
-            throw new InvalidConfigException('Backup module need to be configured with `config array`');
+        if (!is_array($this->_configs) || !sizeof($this->_configs)) {
+                    throw new InvalidConfigException('Backup module need to be configured with `config array`');
+        }
 
         foreach ($this->_configs as $config) {
-            if (isset($config['id']) && $config['id'] == $model->config_id)
-                $this->_currentConfig = $config;
+            if (isset($config['id']) && $config['id'] == $model->config_id) {
+                            $this->_currentConfig = $config;
+            }
         }
 
-        if (!$this->_currentConfig)
-            throw new ErrorException("Config `{$model->config_id}` not found.");
+        if (!$this->_currentConfig) {
+                    throw new ErrorException("Config `{$model->config_id}` not found.");
+        }
 
         $this->_model = $model;
         $this->backupFilePath = Yii::getAlias(Yii::$app->getModule('backup')->backupFolder . DIRECTORY_SEPARATOR . $this->_model->filename);

tests/unit/BackupCreateTest.php

Braces +3 added lines, -2 removed lines

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

Indentation +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         $this->assertTrue(is_object($backup));
         $this->assertEquals(BackupType::FILES, $backup->type);
         $this->assertFileExists($backup->getFullPath());
-       // @unlink($backup->getFullPath());
+        // @unlink($backup->getFullPath());
     }
 
     public function testSaveAllOldBackups()

src/logic/BackupCreate.php

Braces +15 added lines, -10 removed lines

@@ -47,16 +47,19 @@ 
         $this->dumperClass = $dumperClass;
         $this->configs = Yii::$app->getModule('backup')->configs;
 
-        if (!is_array($this->configs) || !sizeof($this->configs))
-            throw new InvalidConfigExceptionAlias('Backup module need to be configured with `config array`');
+        if (!is_array($this->configs) || !sizeof($this->configs)) {
+                    throw new InvalidConfigExceptionAlias('Backup module need to be configured with `config array`');
+        }
 
         foreach ($this->configs as $config) {
-            if (isset($config['id']) && $config['id'] == $config_id)
-                $this->currentConfig = $config;
+            if (isset($config['id']) && $config['id'] == $config_id) {
+                            $this->currentConfig = $config;
+            }
         }
 
-        if (!$this->currentConfig)
-            throw new InvalidConfigExceptionAlias("Config `{$config_id}` not found.");
+        if (!$this->currentConfig) {
+                    throw new InvalidConfigExceptionAlias("Config `{$config_id}` not found.");
+        }
     }
 
     /**
@@ -91,17 +94,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->currentConfig['id']])
             ->orderBy('date DESC')
             ->offset($this->currentConfig['limit'] - 1)
             ->all();
 
-        if ($backups)
-            foreach ($backups as $backup)
+        if ($backups) {
+                    foreach ($backups as $backup)
                 $backup->delete();
+        }
     }
 
     /**