floor12 / yii2-module-files

src/logic/ImagePreviewer.php

Braces +27 added lines, -18 removed lines

@@ -31,8 +31,9 @@ 
         $this->width = $width;
         $this->webp = $webp;
 
-        if ($this->model->type != FileType::IMAGE)
-            throw new ErrorException('File is not an image.');
+        if ($this->model->type != FileType::IMAGE) {
+                    throw new ErrorException('File is not an image.');
+        }
     }
 
     /**
@@ -40,22 +41,26 @@ 
      */
     public function getUrl()
     {
-        if ($this->model->isSvg())
-            return $this->model->getRootPath();
+        if ($this->model->isSvg()) {
+                    return $this->model->getRootPath();
+        }
 
         $this->fileName = Yii::$app->getModule('files')->cacheFullPath . $this->model->makeNameWithSize($this->model->filename, $this->width, 0);
         $this->fileNameWebp = Yii::$app->getModule('files')->cacheFullPath . $this->model->makeNameWithSize($this->model->filename, $this->width, 0, true);
 
         $this->prepareFolders();
 
-        if (!file_exists($this->fileName) || filesize($this->fileName) == 0)
-            $this->createPreview();
+        if (!file_exists($this->fileName) || filesize($this->fileName) == 0) {
+                    $this->createPreview();
+        }
 
-        if ($this->webp && !file_exists($this->fileNameWebp))
-            $this->createPreviewWebp();
+        if ($this->webp && !file_exists($this->fileNameWebp)) {
+                    $this->createPreviewWebp();
+        }
 
-        if ($this->webp)
-            return $this->fileNameWebp;
+        if ($this->webp) {
+                    return $this->fileNameWebp;
+        }
 
         return $this->fileName;
     }
@@ -65,17 +70,21 @@ 
      */
     protected function prepareFolders()
     {
-        if (!file_exists(Yii::$app->getModule('files')->cacheFullPath))
-            mkdir(Yii::$app->getModule('files')->cacheFullPath);
+        if (!file_exists(Yii::$app->getModule('files')->cacheFullPath)) {
+                    mkdir(Yii::$app->getModule('files')->cacheFullPath);
+        }
 
         preg_match('/(.+\/\d{2})\/\d{2}\//', $this->fileName, $matches);
 
-        if (!file_exists($matches[1]))
-            @mkdir($matches[1]);
-        if (!file_exists($matches[0]))
-            @mkdir($matches[0]);
-        if (!file_exists($matches[0]))
-            throw new ErrorException("Unable to create cache folder: {$matches[0]}");
+        if (!file_exists($matches[1])) {
+                    @mkdir($matches[1]);
+        }
+        if (!file_exists($matches[0])) {
+                    @mkdir($matches[0]);
+        }
+        if (!file_exists($matches[0])) {
+                    throw new ErrorException("Unable to create cache folder: {$matches[0]}");
+        }
     }
 
     /**

src/logic/PathGenerator.php

Braces +15 added lines, -10 removed lines

@@ -20,14 +20,17 @@ 
     public function __construct($storagePath)
     {
 
-        if (!$storagePath)
-            throw new ErrorException('Storage path not set for path generator.');
+        if (!$storagePath) {
+                    throw new ErrorException('Storage path not set for path generator.');
+        }
 
-        if (!file_exists($storagePath))
-            mkdir($storagePath);
+        if (!file_exists($storagePath)) {
+                    mkdir($storagePath);
+        }
 
-        if (!file_exists($storagePath))
-            throw new ErrorException('Unable to create storage.');
+        if (!file_exists($storagePath)) {
+                    throw new ErrorException('Unable to create storage.');
+        }
 
         $folderName0 = rand(10, 99);
         $folderName1 = rand(10, 99);
@@ -38,10 +41,12 @@ 
         $fullPath0 = $storagePath . $path0;
         $fullPath1 = $storagePath . $path1;
 
-        if (!file_exists($fullPath0))
-            mkdir($fullPath0);
-        if (!file_exists($fullPath1))
-            mkdir($fullPath1);
+        if (!file_exists($fullPath0)) {
+                    mkdir($fullPath0);
+        }
+        if (!file_exists($fullPath1)) {
+                    mkdir($fullPath1);
+        }
 
         $this->path = $path1 . DIRECTORY_SEPARATOR . md5(rand(0, 1000) . time());
     }

src/controllers/DefaultController.php

Braces +36 added lines, -24 removed lines

@@ -73,8 +73,9 @@ 
      */
     private function checkFormToken()
     {
-        if (in_array($this->action->id, $this->actionsToCheck) && FileInputWidget::generateToken() != Yii::$app->request->post('_fileFormToken'))
-            throw new BadRequestHttpException('File-form token is wrong or missing.');
+        if (in_array($this->action->id, $this->actionsToCheck) && FileInputWidget::generateToken() != Yii::$app->request->post('_fileFormToken')) {
+                    throw new BadRequestHttpException('File-form token is wrong or missing.');
+        }
     }
 
     /**
@@ -87,12 +88,14 @@ 
 
         $zip = new  ZipArchive;
         $filename = Yii::getAlias("@webroot/assets/files.zip");
-        if (file_exists($filename))
-            @unlink($filename);
+        if (file_exists($filename)) {
+                    @unlink($filename);
+        }
         if (sizeof($files) && $zip->open($filename, ZipArchive::CREATE)) {
 
-            foreach ($files as $file)
-                $zip->addFile($file->rootPath, $file->title);
+            foreach ($files as $file) {
+                            $zip->addFile($file->rootPath, $file->title);
+            }
 
             $zip->close();
             header("Pragma: public");
@@ -164,8 +167,9 @@ 
 
         $view = Yii::$app->request->post('mode') == 'single' ? "_single" : "_file";
 
-        if ($ratio)
-            $this->getView()->registerJs("initCropper({$model->id}, '{$model->href}', {$ratio}, true);");
+        if ($ratio) {
+                    $this->getView()->registerJs("initCropper({$model->id}, '{$model->href}', {$ratio}, true);");
+        }
 
         return $this->renderAjax($view, [
             'model' => $model,
@@ -182,11 +186,13 @@ 
     {
         $model = File::findOne(['hash' => $hash]);
 
-        if (!$model)
-            throw new NotFoundHttpException("Запрашиваемый файл не найден");
+        if (!$model) {
+                    throw new NotFoundHttpException("Запрашиваемый файл не найден");
+        }
 
-        if (!file_exists($model->rootPath))
-            throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+        if (!file_exists($model->rootPath)) {
+                    throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+        }
 
         Yii::$app->response->headers->set('Last-Modified', date("c", $model->created));
         Yii::$app->response->headers->set('Cache-Control', 'public, max-age=' . (60 * 60 * 24 * 15));
@@ -216,14 +222,17 @@ 
     {
         $model = File::findOne(['hash' => $hash]);
 
-        if (!$model)
-            throw new NotFoundHttpException("Запрашиваемый файл не найден");
+        if (!$model) {
+                    throw new NotFoundHttpException("Запрашиваемый файл не найден");
+        }
 
-        if ($model->type != FileType::IMAGE)
-            throw new NotFoundHttpException("Запрашиваемый файл не является изображением");
+        if ($model->type != FileType::IMAGE) {
+                    throw new NotFoundHttpException("Запрашиваемый файл не является изображением");
+        }
 
-        if (!file_exists($model->rootPath))
-            throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+        if (!file_exists($model->rootPath)) {
+                    throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+        }
 
         Yii::$app->response->headers->set('Last-Modified', date("c", $model->created));
         Yii::$app->response->headers->set('Cache-Control', 'public, max-age=' . (60 * 60 * 24 * 15));
@@ -233,8 +242,9 @@ 
 
             $filename = Yii::createObject(ImagePreviewer::class, [$model, $width, $webp])->getUrl();
 
-            if (!file_exists($filename))
-                throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+            if (!file_exists($filename)) {
+                            throw new NotFoundHttpException('Запрашиваемый файл не найден на диске.');
+            }
 
             $response = Yii::$app->response;
             $response->format = Response::FORMAT_RAW;
@@ -275,8 +285,9 @@ 
     {
         $model = File::findOne(['hash' => $hash]);
 
-        if (!$model)
-            throw new NotFoundHttpException("Запрашиваемый файл не найден в базе.");
+        if (!$model) {
+                    throw new NotFoundHttpException("Запрашиваемый файл не найден в базе.");
+        }
 
         $response = Yii::$app->response;
         $response->format = Response::FORMAT_RAW;
@@ -285,8 +296,9 @@ 
         Yii::$app->response->headers->set('Last-Modified', date("c", $model->created));
         Yii::$app->response->headers->set('Cache-Control', 'public, max-age=' . (60 * 60 * 24 * 15));
 
-        if (!file_exists($model->rootPreviewPath))
-            throw new NotFoundHttpException('Preview not found.');
+        if (!file_exists($model->rootPreviewPath)) {
+                    throw new NotFoundHttpException('Preview not found.');
+        }
 
         $response->sendFile($model->rootPreviewPath, 'preview.jpg');
 

src/controllers/ConsoleController.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $commands[] = "find {$module->storageFullPath}  -regextype egrep -regex \".+/.{32}\..{3,4}\.jpg\" -exec rm -rf {} \;";
         $commands[] = "find {$module->cacheFullPath}  -regextype egrep -regex \".+/.{32}\..{3,4}\.jpg\" -exec rm -rf {} \;";
 
-        array_map(function ($command) {
+        array_map(function($command) {
             exec($command);
         }, $commands);
 

Braces +15 added lines, -9 removed lines

@@ -29,9 +29,11 @@ 
     {
         $time = strtotime('- 6 hours');
         $files = File::find()->where("`object_id`=0 AND `created`<'{$time}'")->all();
-        if ($files) foreach ($files as $file) {
+        if ($files) {
+            foreach ($files as $file) {
             $file->delete();
         }
+        }
     }
 
     /**
@@ -60,22 +62,26 @@ 
     {
         $ffmpeg = Yii::$app->getModule('files')->ffmpeg;
 
-        if (!file_exists($ffmpeg))
-            return $this->stdout("ffmpeg is not found: {$ffmpeg}" . PHP_EOL, Console::FG_RED);
+        if (!file_exists($ffmpeg)) {
+                    return $this->stdout("ffmpeg is not found: {$ffmpeg}" . PHP_EOL, Console::FG_RED);
+        }
 
-        if (!is_executable($ffmpeg))
-            return $this->stdout("ffmpeg is not executable: {$ffmpeg}" . PHP_EOL, Console::FG_RED);
+        if (!is_executable($ffmpeg)) {
+                    return $this->stdout("ffmpeg is not executable: {$ffmpeg}" . PHP_EOL, Console::FG_RED);
+        }
 
         $file = File::find()
             ->where(['type' => FileType::VIDEO, 'video_status' => VideoStatus::QUEUE])
             ->andWhere(['!=', 'object_id', 0])
             ->one();
 
-        if (!$file)
-            return $this->stdout("Convert queue is empty" . PHP_EOL, Console::FG_GREEN);
+        if (!$file) {
+                    return $this->stdout("Convert queue is empty" . PHP_EOL, Console::FG_GREEN);
+        }
 
-        if (!file_exists($file->rootPath))
-            return $this->stdout("Source file is not found: {$file->rootPath}" . PHP_EOL, Console::FG_RED);
+        if (!file_exists($file->rootPath)) {
+                    return $this->stdout("Source file is not found: {$file->rootPath}" . PHP_EOL, Console::FG_RED);
+        }
 
 
         $file->video_status = VideoStatus::CONVERTING;

src/views/default/_single.php

Braces +7 added lines, -3 removed lines

@@ -17,8 +17,9 @@ 
 use yii\helpers\Html;
 use yii\web\View;
 
-if (is_array($model))
+if (is_array($model)) {
     $model = $model[0];
+}
 
 $doc_contents = [
     'application/msword',
@@ -38,7 +39,8 @@ 
             <?= Html::hiddenInput((new ReflectionClass($model->class))->getShortName() . "[{$model->field}_ids][]", $model->id) ?>
         </div>
 
-    <?php else: ?>
+    <?php else {
+    : ?>
 
         <div data-title="<?= $model->title ?>"
              id="yii2-file-object-<?= $model->id ?>"
@@ -52,7 +54,9 @@ 
             <?php if ($model->type != FileType::IMAGE): ?>
                 <?= $model->icon ?>
                 <?= $model->title ?>
-            <?php endif; ?>
+            <?php endif;
+}
+?>
         </div>
     <?php endif; ?>