thinktomorrow / chief

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();

src/Urls/ProvidesUrl/BaseUrlSegment.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      */
     public static function prepend(ProvidesUrl $model, string $slug, $locale): string
     {
-        $slugWithBaseSegment = $model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $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/Management/Assistants/UrlAssistant.php

Spacing +6 added lines, -6 removed lines

@@ -25,10 +25,10 @@ 
 
     public function manager(Manager $manager)
     {
-        $this->manager  = $manager;
+        $this->manager = $manager;
 
-        if (! $manager->model() instanceof ProvidesUrl) {
-            throw new \Exception('UrlAssistant requires the model interfaced by ' . ProvidesUrl::class . '.');
+        if (!$manager->model() instanceof ProvidesUrl) {
+            throw new \Exception('UrlAssistant requires the model interfaced by '.ProvidesUrl::class.'.');
         }
 
         $this->model = $manager->model();
@@ -63,7 +63,7 @@ 
                         'url-slugs.*' => 'taalspecifieke link',
                     ])
                 ->view('chief::back._fields.url-slugs')
-                ->viewData(['fields' => UrlSlugFields::fromModel($this->model) ]),
+                ->viewData(['fields' => UrlSlugFields::fromModel($this->model)]),
         ]);
     }
 
@@ -73,9 +73,9 @@ 
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($this->model)->reject(function ($record) {
+        UrlRecord::getByModel($this->model)->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
     }

src/Common/helpers.php

Spacing +10 added lines, -10 removed lines

@@ -74,7 +74,7 @@ 
         $empty = true;
 
         foreach ($values as $value) {
-            if (! $value || !trim($value)) {
+            if (!$value || !trim($value)) {
                 continue;
             }
             $empty = false;
@@ -84,7 +84,7 @@ 
     }
 }
 
-if (! function_exists('contract')) {
+if (!function_exists('contract')) {
     function contract($instance, $contract)
     {
         return $instance instanceof $contract;
@@ -113,7 +113,7 @@ 
             $text = cleanupHTML($text, $clean);
         }
         $teaser = substr($text, 0, $max);
-        return strlen($text) <= $max ? $teaser : $teaser . $ending;
+        return strlen($text) <= $max ? $teaser : $teaser.$ending;
     }
 }
 
@@ -171,7 +171,7 @@ 
          * htmlPurifier converts characters to their encode equivalents. This is something
          * that we need to reverse after the htmlPurifier cleanup.
          */
-        $value  = str_replace('&amp;', '&', $value);
+        $value = str_replace('&amp;', '&', $value);
 
         return $value;
     }
@@ -239,16 +239,16 @@ 
             'scheme', 'host', 'port', 'path', 'query', 'fragment'
         ], null), $parsed_url, $overrides);
 
-        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'] . '://' : null;
-        $port = $parsed_url['port'] ? ':' . $parsed_url['port'] : null;
-        $fragment = $parsed_url['fragment'] ? '#' . $parsed_url['fragment'] : null;
+        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'].'://' : null;
+        $port = $parsed_url['port'] ? ':'.$parsed_url['port'] : null;
+        $fragment = $parsed_url['fragment'] ? '#'.$parsed_url['fragment'] : null;
 
-        $baseurl = $scheme . $parsed_url['host'] . $port . $parsed_url['path'];
+        $baseurl = $scheme.$parsed_url['host'].$port.$parsed_url['path'];
         $current_query = [];
 
         $_query = explode('&', $parsed_url['query']);
 
-        array_map(function ($v) use (&$current_query) {
+        array_map(function($v) use (&$current_query) {
             if (!$v) {
                 return;
             }
@@ -266,7 +266,7 @@ 
 
         $query = urldecode(http_build_query(array_merge($current_query, $query_params)));
 
-        return $baseurl . '?' . $query . $fragment;
+        return $baseurl.'?'.$query.$fragment;
     }
 }
 

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);
     }