thinktomorrow / chief

src/Media/UploadMedia.php

Spacing +11 added lines, -11 removed lines

@@ -111,7 +111,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);
         }
@@ -119,7 +119,7 @@ 
         // New files are passed with their filename (instead of their id)
         // For new files we will replace the filename with the id.
         if (false !== ($key = array_search($image_name, $files_order))) {
-            $files_order[$key] = (string) $asset->id;
+            $files_order[$key] = (string)$asset->id;
         }
     }
 
@@ -134,7 +134,7 @@ 
 
         if (is_string($file)) {
             $asset = AssetUploader::uploadFromBase64($file, $filename);
-        } else {
+        }else {
             $asset = AssetUploader::upload($file, $filename);
         }
 
@@ -151,9 +151,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;
     }
@@ -169,9 +169,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'
                         );
                     }
                 }
@@ -185,16 +185,16 @@ 
         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.');
+                    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.');
+                    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.');
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
                     }
                 }
             }

src/HealthMonitor/Checks/HomepageAccessibleCheck.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\HealthMonitor\Checks;
 
@@ -16,7 +16,7 @@ 
 
     private function get_http_response_code(string $url)
     {
-        if ($url =='') {
+        if ($url == '') {
             return false;
         }
 
@@ -34,7 +34,7 @@ 
 
     public function message(): string
     {
-        return 'Het lijkt erop dat de homepagina niet meer bereikbaar is. <a href="'. route('chief.back.settings.edit') .'" class="text-secondary-800 underline hover:text-white">Kies een nieuwe</a>.';
+        return 'Het lijkt erop dat de homepagina niet meer bereikbaar is. <a href="'.route('chief.back.settings.edit').'" class="text-secondary-800 underline hover:text-white">Kies een nieuwe</a>.';
     }
 
     public function notifiers(): array

src/HealthMonitor/Checks/HomepageSetCheck.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\HealthMonitor\Checks;
 
@@ -18,7 +18,7 @@ 
 
     public function message(): string
     {
-        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="'. route('chief.back.settings.edit') .'" class="text-secondary-800 underline hover:text-white">Settings</a>';
+        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="'.route('chief.back.settings.edit').'" class="text-secondary-800 underline hover:text-white">Settings</a>';
     }
 
     public function notifiers(): array

src/HealthMonitor/Monitor.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\HealthMonitor;
 
@@ -22,7 +22,7 @@ 
         foreach ($this->checks as $check) {
             $checkInstance = app($check);
             
-            if (! $checkInstance instanceof HealthCheck) {
+            if (!$checkInstance instanceof HealthCheck) {
                 throw new InvalidClassException('Checks must implement Healthcheck interface.');
             }
 
@@ -33,7 +33,7 @@ 
                     app($notifier)->onFailure($checkInstance);
                 }
                 return;
-            } else {
+            }else {
                 foreach ($notifiers as $notifier) {
                     app($notifier)->onSuccess($checkInstance);
                 }

src/Management/ManagesPagebuilder.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 
         $availableChildren = AvailableChildren::forParent($model);
 
-        $modules = $availableChildren->onlyModules()->reject(function ($module) use ($model) {
+        $modules = $availableChildren->onlyModules()->reject(function($module) use ($model) {
             return $module->page_id != null && $module->page_id != $model->id;
         });
 
@@ -58,7 +58,7 @@ 
         $available_sets = FlatReferencePresenter::toGroupedSelectValues($availableChildren->onlySets())->toArray();
 
         // Current sections
-        $sections = $model->children()->map(function ($section, $index) {
+        $sections = $model->children()->map(function($section, $index) {
             if ($section instanceof TranslatableContract) {
                 $section->injectTranslationForForm();
             }
@@ -92,7 +92,7 @@ 
      */
     private function findEditUrl($model): ?string
     {
-        if (! $model instanceof ManagedModel) {
+        if (!$model instanceof ManagedModel) {
             return null;
         }
 

src/Modules/Application/DeleteModule.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
             $module->update([
-                'slug' => $module->slug . $this->appendDeleteMarker(),
+                'slug' => $module->slug.$this->appendDeleteMarker(),
             ]);
 
             $module->delete();
@@ -35,6 +35,6 @@ 
 
     private function appendDeleteMarker(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

src/Fields/Types/FieldType.php

Spacing +12 added lines, -12 removed lines

@@ -6,19 +6,19 @@ 
 
 class FieldType
 {
-    const INPUT = 'input';   // oneliner text (input)
-    const TEXT = 'text';    // Plain text (textarea)
+    const INPUT = 'input'; // oneliner text (input)
+    const TEXT = 'text'; // Plain text (textarea)
     const NUMBER = 'number'; // number
     const RANGE = 'range'; // range slider
-    const DATE = 'date';    // Timestamp input
-    const PHONENUMBER = 'phonenumber';    // Timestamp input
-    const HTML = 'html';    // Html text (wysiwyg)
-    const SELECT = 'select';  // Select options
-    const MEDIA = 'media';  // media file (slim uploader)
-    const DOCUMENT = 'document';  // documents
-    const RADIO = 'radio';  // radio select
-    const CHECKBOX = 'checkbox';  // checkbox select
-    const PAGEBUILDER = 'pagebuilder';  // the most special field there is...
+    const DATE = 'date'; // Timestamp input
+    const PHONENUMBER = 'phonenumber'; // Timestamp input
+    const HTML = 'html'; // Html text (wysiwyg)
+    const SELECT = 'select'; // Select options
+    const MEDIA = 'media'; // media file (slim uploader)
+    const DOCUMENT = 'document'; // documents
+    const RADIO = 'radio'; // radio select
+    const CHECKBOX = 'checkbox'; // checkbox select
+    const PAGEBUILDER = 'pagebuilder'; // the most special field there is...
 
     /** @var string */
     private $type;
@@ -37,7 +37,7 @@ 
 
     public static function fromString(string $type): self
     {
-        $class = 'Thinktomorrow\Chief\Fields\Types\\' . ucfirst($type . 'Field');
+        $class = 'Thinktomorrow\Chief\Fields\Types\\'.ucfirst($type.'Field');
 
         return new $class(new static($type));
     }

src/Fields/Types/CheckboxField.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 

src/Urls/Application/SaveUrlSlugs.php

Spacing +7 added lines, -7 removed lines

@@ -57,7 +57,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()
@@ -66,7 +66,7 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
 
@@ -82,7 +82,7 @@ 
         }
 
         // 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]);
             }
@@ -127,12 +127,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();
             }
@@ -147,11 +147,11 @@ 
 
         // The old homepage url should be removed since this is no longer in effect.
         // In case of any redirect to this old homepage, the last used redirect is now back in effect.
-        $existingRecords->reject(function ($existingRecord) {
+        $existingRecords->reject(function($existingRecord) {
             return (
                 $existingRecord->model_type == $this->model->getMorphClass() &&
                 $existingRecord->model_id == $this->model->id);
-        })->each(function ($existingRecord) {
+        })->each(function($existingRecord) {
 
             // TODO: if there is a redirect to this page, we'll take this one as the new url
             $existingRecord->delete();