thinktomorrow / chief

src/Media/UploadMedia.php

Spacing +7 added lines, -7 removed lines

@@ -59,7 +59,7 @@ 
         if (is_string($file)) {
             $image_name = json_decode($file)->output->name;
             $asset      = $this->addAsset(json_decode($file)->output->image, $type, null, $image_name, $model);
-        } else {
+        }else {
             $image_name = $file->getClientOriginalName();
             $asset      = $this->addAsset($file, $type, null, $image_name, $model);
         }
@@ -82,7 +82,7 @@ 
 
         if (is_string($file)) {
             $asset = AssetUploader::uploadFromBase64($file, $filename);
-        } else {
+        }else {
             $asset = AssetUploader::upload($file, $filename);
         }
 
@@ -126,9 +126,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;
     }
@@ -144,9 +144,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'
                         );
                     }
                 }

src/PageBuilder/PresentSections.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
             $this->addModelToCollection($i, $child);
         }
 
-        return $this->sets->values()->map(function (ViewableContract $child) {
+        return $this->sets->values()->map(function(ViewableContract $child) {
             return ($this->withSnippets && method_exists($child, 'withSnippets'))
                 ? $child->withSnippets()->setViewParent($this->parent)->renderView()
                 : $child->setViewParent($this->parent)->renderView();
@@ -95,7 +95,7 @@ 
     {
         // Only published pages you fool!
         // TODO: check for assistant instead of method existence
-        if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
+        if (method_exists($model, 'isPublished') && !$model->isPublished()) {
             return;
         }
 

src/Sets/Set.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
 
         // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module.
-        return $this->map(function ($item) {
+        return $this->map(function($item) {
             return ($this->viewParent && $item instanceof ViewableContract)
                 ? $item->setViewParent($this->viewParent)->renderView()
                 : ($item->content ?? '');
@@ -79,7 +79,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage);
 
         return (new \Illuminate\Pagination\Paginator($items, $perPage, $currentPage))->setPath($path);
     }
@@ -96,7 +96,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage);
 
         return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->count(), $perPage, $currentPage))->setPath($path);
     }

src/Urls/UrlRecord.php

Spacing +2 added lines, -2 removed lines

@@ -112,12 +112,12 @@
             $builder->where('locale', $locale);
         }
 
-        if (! $includeRedirects) {
+        if (!$includeRedirects) {
             $builder->whereNull('redirect_id');
         }
 
         if ($ignoredModel) {
-            $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
+            $builder->whereNotIn('id', function($query) use ($ignoredModel) {
                 $query->select('id')
                       ->from('chief_urls')
                       ->where('model_type', '=', $ignoredModel->getMorphClass())

src/Urls/MemoizedUrlRecord.php

Spacing +2 added lines, -2 removed lines

@@ -8,14 +8,14 @@
 {
     public static function findByModel(Model $model, string $locale = null): UrlRecord
     {
-        return chiefMemoize('url-records-find-by-model', function ($model, $locale = null) {
+        return chiefMemoize('url-records-find-by-model', function($model, $locale = null) {
             return parent::findByModel($model, $locale);
         }, [$model, $locale]);
     }
 
     public static function getByModel(Model $model)
     {
-        return chiefMemoize('url-records-get-by-model', function ($model) {
+        return chiefMemoize('url-records-get-by-model', function($model) {
             return parent::getByModel($model);
         }, [$model]);
     }

src/Urls/SaveUrlSlugs.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     private function saveRecord(string $locale, ?string $slug)
     {
         // Existing ones for this locale?
-        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function ($record) use ($locale) {
+        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 !$record->isRedirect()
@@ -55,14 +55,14 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
         } elseif ($nonRedirectsWithSameLocale->isEmpty()) {
             $this->createRecord($locale, $slug);
-        } else {
+        }else {
             // Only replace the existing records that differ from the current passed slugs
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) use ($slug) {
                 if ($existingRecord->slug != $slug) {
                     $existingRecord->replaceAndRedirect(['slug' => $slug]);
                 }
@@ -87,12 +87,12 @@ 
      */
     private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
     {
-        $existingRecords->filter(function ($record) use ($locale) {
+        $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 $record->isRedirect()
             );
-        })->each(function ($existingRecord) use ($slug) {
+        })->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug == $slug) {
                 $existingRecord->delete();
             }
@@ -106,7 +106,7 @@ 
      */
     private function prependBaseUrlSegment(string $slug, $locale): string
     {
-        $slugWithBaseSegment = $this->model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $this->model->baseUrlSegment($locale).'/'.$slug;
         $slugWithBaseSegment = trim($slugWithBaseSegment, '/');
 
         // If slug with base segment is empty string, it means that the passed slug was probably a "/" character.

src/Urls/UrlSlugFields.php

Spacing +5 added lines, -5 removed lines

@@ -42,10 +42,10 @@ 
         $fields = new static([]);
 
         foreach ($locales as $locale) {
-            $fields['url-slugs.' . $locale] = UrlSlugField::make('url-slugs.' . $locale)
+            $fields['url-slugs.'.$locale] = UrlSlugField::make('url-slugs.'.$locale)
                                                 ->setBaseUrlSegment($model->baseUrlSegment($locale))
-                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)) .'/')
-                                                ->name('url-slugs[' . $locale . ']')
+                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)).'/')
+                                                ->name('url-slugs['.$locale.']')
                                                 ->label($locale);
         }
 
@@ -58,7 +58,7 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
@@ -70,7 +70,7 @@ 
             $fields['url-slugs.'.$record->locale]
                 ->setUrlRecord($record)
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) .'/');
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
         }
     }
 }

src/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -39,11 +39,11 @@ 
                 return static::createRedirect($model->url($locale));
             }
 
-            if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
+            if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
-                if (! PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request ['. $slug .'] but it is not published.');
+                if (!PreviewMode::fromRequest()->check()) {
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -54,7 +54,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request ['. $slug .'] for locale ['.$locale.'].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)

src/Fields/Types/Field.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;
 
@@ -142,7 +142,7 @@ 
         }
 
         if (!in_array($name, ['label', 'key', 'description', 'column', 'name', 'prepend', 'append'])) {
-            throw new \InvalidArgumentException('Cannot set value by ['. $name .'].');
+            throw new \InvalidArgumentException('Cannot set value by ['.$name.'].');
         }
 
         $this->values[$name] = $arguments[0];