thinktomorrow / chief

src/Pages/Page.php

Spacing +4 added lines, -4 removed lines

@@ -83,7 +83,7 @@ 
             return static::$managedModelKey;
         }
 
-        throw new \Exception('Missing required static property \'managedModelKey\' on ' . static::class. '.');
+        throw new \Exception('Missing required static property \'managedModelKey\' on '.static::class.'.');
     }
 
     /**
@@ -123,9 +123,9 @@ 
     public function flatReferenceLabel(): string
     {
         if ($this->exists) {
-            $status = ! $this->isPublished() ? ' [' . $this->statusAsPlainLabel().']' : null;
+            $status = !$this->isPublished() ? ' ['.$this->statusAsPlainLabel().']' : null;
 
-            return $this->title ? $this->title . $status : '';
+            return $this->title ? $this->title.$status : '';
         }
 
         return '';
@@ -145,7 +145,7 @@ 
         // What was the creator of the assetlibrary package thinking. It sure wasn't me... I promise...
         $assets = $this->getAllFiles($type, app()->getLocale())->map->getFileUrl();
 
-        if($assets->first() == null)
+        if ($assets->first() == null)
         {
             $assets = $this->getAllFiles($type)->map->getFileUrl();
         }

src/Fields/Types/MediaField.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -43,7 +43,7 @@ 
 
         $builder = $model->assets()->where('asset_pivots.type', $this->key());
 
-        if($locale) {
+        if ($locale) {
             $builder = $builder->where('asset_pivots.locale', $locale);
         }
 

src/Fields/Types/DocumentField.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -43,7 +43,7 @@ 
 
         $builder = $model->assets()->where('asset_pivots.type', $this->key());
 
-        if($locale) {
+        if ($locale) {
             $builder = $builder->where('asset_pivots.locale', $locale);
         }
         foreach ($builder->get() as $asset) {

src/Media/UploadMedia.php

Spacing +34 added lines, -34 removed lines

@@ -25,7 +25,7 @@ 
 
         // When no files are uploaded, we still would like to sort our assets duh
         if (empty($files_by_type)) {
-            foreach($files_order_by_type as $type => $fileIdsCollection){
+            foreach ($files_order_by_type as $type => $fileIdsCollection) {
                 $this->sortFiles($model, $type, $fileIdsCollection);
             }
 
@@ -34,7 +34,7 @@ 
 
         foreach ($files_by_type as $type => $files) {
 
-            foreach($files as $locale => $files)
+            foreach ($files as $locale => $files)
             {
                 $this->validateFileUploads($files);
 
@@ -54,7 +54,7 @@ 
         if (is_string($file)) {
             $image_name = json_decode($file)->output->name;
             $asset      = $this->addAsset(json_decode($file)->output->image, $type, $locale, $image_name, $model);
-        } else {
+        }else {
             $image_name = $file->getClientOriginalName();
             $asset      = $this->addAsset($file, $type, $locale, $image_name, $model);
         }
@@ -77,7 +77,7 @@ 
 
         if (is_string($file)) {
             $asset = AssetUploader::uploadFromBase64($file, $filename);
-        } else {
+        }else {
             $asset = AssetUploader::upload($file, $filename);
         }
 
@@ -114,7 +114,7 @@ 
     private function handleFiles(string $action, HasMedia $model, string $type = null, array $files, array $files_order = [], string $locale = null)
     {
         if (isset($files[$action]) && is_array($files[$action]) && !empty($files[$action])) {
-            if($action == 'delete')
+            if ($action == 'delete')
             {
                 foreach ($model->assets()->whereIn('id', $files[$action])->get() as $asset) {
                     $asset->delete();
@@ -127,10 +127,10 @@ 
                     continue;
                 }
 
-                if($action == 'new')
+                if ($action == 'new')
                 {
                     $this->addFile($model, $type, $files_order, $file, $locale);
-                }elseif($action == 'replace')
+                }elseif ($action == 'replace')
                 {
                     $asset = AssetUploader::uploadFromBase64(json_decode($file)->output->image, json_decode($file)->output->name);
                     $model->replaceAsset($id, $asset->id);
@@ -145,9 +145,9 @@ 
      */
     private function sluggifyFilename($filename): string
     {
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename  = substr($filename, 0, strrpos($filename, '.'));
-        $filename  = str_slug($filename) . '.' . $extension;
+        $filename  = str_slug($filename).'.'.$extension;
 
         return $filename;
     }
@@ -163,9 +163,9 @@ 
                 if ($file instanceof UploadedFile && !$file->isValid()) {
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
                         throw new FileTooBigException(
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
                         );
                     }
                 }
@@ -177,31 +177,31 @@ 
     {
         $actions = ['new', 'replace', 'delete'];
 
-        foreach($files_by_type as $type => $files)
+        foreach ($files_by_type as $type => $files)
         {
-            foreach($files as $locale => $_files){
-                if(!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+            foreach ($files as $locale => $_files) {
+                if (!in_array($locale, config('translatable.locales'))) {
+                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
 
-                if(!is_array($_files)) {
-                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a ' . gettype($_files) . ' is given.');
+                if (!is_array($_files)) {
+                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a '.gettype($_files).' is given.');
                 }
 
-                foreach($_files as $action => $file) {
-                    if(!in_array($action, $actions)) {
-                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead ' . $action . ' is given.');
+                foreach ($_files as $action => $file) {
+                    if (!in_array($action, $actions)) {
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
                     }
                 }
             }
         }
 
-        foreach($files_order_by_type as $type => $fileIdsCollection)
+        foreach ($files_order_by_type as $type => $fileIdsCollection)
         {
-            foreach($fileIdsCollection as $locale => $commaSeparatedFileIds){
+            foreach ($fileIdsCollection as $locale => $commaSeparatedFileIds) {
 
-                if(!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                if (!in_array($locale, config('translatable.locales'))) {
+                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
             }
         }
@@ -211,13 +211,13 @@ 
     {
         $defaultLocale = config('app.fallback_locale');
 
-        foreach($files_by_type as $type => $files)
+        foreach ($files_by_type as $type => $files)
         {
-            foreach($files as $locale => $_files){
-                if(!in_array($locale, config('translatable.locales'))) {
+            foreach ($files as $locale => $_files) {
+                if (!in_array($locale, config('translatable.locales'))) {
                     unset($files_by_type[$type][$locale]);
 
-                    if(!isset($files_by_type[$type][$defaultLocale])) {
+                    if (!isset($files_by_type[$type][$defaultLocale])) {
                         $files_by_type[$type][$defaultLocale] = [];
                     }
 
@@ -233,14 +233,14 @@ 
     {
         $defaultLocale = config('app.fallback_locale');
 
-        foreach($files_order_by_type as $type => $fileIdsCollection)
+        foreach ($files_order_by_type as $type => $fileIdsCollection)
         {
-            if(!is_array($fileIdsCollection)) {
+            if (!is_array($fileIdsCollection)) {
                 $fileIdsCollection = [$defaultLocale => $fileIdsCollection];
                 $files_order_by_type[$type] = $fileIdsCollection;
             }
 
-            foreach($fileIdsCollection as $locale => $commaSeparatedFileIds){
+            foreach ($fileIdsCollection as $locale => $commaSeparatedFileIds) {
                 $files_order_by_type[$type][$locale] = explode(',', $commaSeparatedFileIds);
             }
         }
@@ -252,7 +252,7 @@ 
     {
         $sortedFileIds = [];
 
-        foreach($fileIdsCollection as $locale => $fileIds) {
+        foreach ($fileIdsCollection as $locale => $fileIds) {
             $sortedFileIds = array_merge($sortedFileIds, $fileIds);
         }
 
@@ -263,7 +263,7 @@ 
     {
         $assets = $model->assets()->where('asset_pivots.type', $type)->get();
 
-        foreach($assets as $asset)
+        foreach ($assets as $asset)
         {
             $pivot = $asset->pivot;
             $pivot->order = array_search($asset->id, $sortedAssetIds);

Braces +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
                 if($action == 'new')
                 {
                     $this->addFile($model, $type, $files_order, $file, $locale);
-                }elseif($action == 'replace')
+                } elseif($action == 'replace')
                 {
                     $asset = AssetUploader::uploadFromBase64(json_decode($file)->output->image, json_decode($file)->output->name);
                     $model->replaceAsset($id, $asset->id);