thinktomorrow / chief

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/Pages/Page.php

Spacing +5 added lines, -5 removed lines

@@ -84,7 +84,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.'.');
     }
 
     /**
@@ -124,9 +124,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 '';
@@ -136,8 +136,8 @@ 
     {
         $classKey = get_class($this);
         if (property_exists($this, 'labelSingular')) {
-            $labelSingular =  $this->labelSingular;
-        } else {
+            $labelSingular = $this->labelSingular;
+        }else {
             $labelSingular = Str::singular($classKey);
         }
 

src/Fields/Types/Field.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -71,7 +71,7 @@ 
 
     public function optional(): bool
     {
-        return ! $this->required();
+        return !$this->required();
     }
 
     public function name(string $name = null)
@@ -146,7 +146,7 @@ 
 
     private function defaultValueResolver(): callable
     {
-        return function (Model $model, $locale) {
+        return function(Model $model, $locale) {
             if ($this->isTranslatable() && $locale) {
                 return $model->getTranslationFor($this->column(), $locale);
             }
@@ -214,7 +214,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];

src/Management/AbstractManager.php

Spacing +6 added lines, -6 removed lines

@@ -84,7 +84,7 @@ 
             return $this->indexPagination($builder);
         }
         
-        return $builder->get()->map(function ($model) {
+        return $builder->get()->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -112,7 +112,7 @@ 
     {
         $paginator = $builder->paginate($this->pageCount);
 
-        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function ($model) {
+        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function($model) {
             return (new static($this->registration))->manage($model);
         });
 
@@ -186,7 +186,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -209,7 +209,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }
 
@@ -291,9 +291,9 @@ 
             static::$bootedTraitMethods[$baseMethod] = [];
         
             foreach (class_uses_recursive($class) as $trait) {
-                $method = class_basename($trait) . ucfirst($baseMethod);
+                $method = class_basename($trait).ucfirst($baseMethod);
                 
-                if (method_exists($class, $method) && ! in_array($method, static::$bootedTraitMethods[$baseMethod])) {
+                if (method_exists($class, $method) && !in_array($method, static::$bootedTraitMethods[$baseMethod])) {
                     static::$bootedTraitMethods[$baseMethod][] = lcfirst($method);
                 }
             }

src/Sets/Set.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,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 ?? '');
@@ -84,7 +84,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 ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }
@@ -101,7 +101,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->paginateSetting('total', $this->count()), $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }

src/Sets/SetReference.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     {
         // Constraints
         if (!isset($values['action'])) {
-            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: ' . print_r($values, true));
+            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: '.print_r($values, true));
         }
 
         return new static(
@@ -54,14 +54,14 @@ 
     {
         $sets = config('thinktomorrow.chief.sets', []);
 
-        return collect($sets)->map(function ($set, $key) {
+        return collect($sets)->map(function($set, $key) {
             return SetReference::fromArray($key, $set);
         });
     }
 
     public static function find($key): ?SetReference
     {
-        return static::all()->filter(function ($ref) use ($key) {
+        return static::all()->filter(function($ref) use ($key) {
             return $ref->key() == $key;
         })->first();
     }
@@ -78,13 +78,13 @@ 
 
         $this->validateAction($class, $method);
 
-        $result = call_user_func_array([app($class),$method], $this->parameters($class, $method, $parent));
+        $result = call_user_func_array([app($class), $method], $this->parameters($class, $method, $parent));
 
-        if (! $result instanceof Set && $result instanceof Collection) {
+        if (!$result instanceof Set && $result instanceof Collection) {
             return new Set($result->all(), $this->key);
         }
 
-        if (! $result instanceof Set && $result instanceof Paginator) {
+        if (!$result instanceof Set && $result instanceof Paginator) {
             return new Set($result->all(), $this->key, [
                 'paginate' => [
                     'total' => $result->total(),
@@ -152,7 +152,7 @@ 
 
     private static function validateAction($class, $method)
     {
-        if (! class_exists($class)) {
+        if (!class_exists($class)) {
             throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
         }
 

src/Urls/UrlHelper.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
      */
     public static function onlineModels(bool $onlySingles = false, Model $ignoredModel = null): Collection
     {
-        $models = chiefMemoize('all-online-models', function () use ($onlySingles) {
+        $models = chiefMemoize('all-online-models', function() use ($onlySingles) {
             $builder = UrlRecord::whereNull('redirect_id')
                 ->select('model_type', 'model_id')
                 ->groupBy('model_type', 'model_id');
@@ -48,17 +48,17 @@ 
                 $builder->where('model_type', 'singles');
             }
 
-            return $builder->get()->mapToGroups(function ($record) {
+            return $builder->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
-            })->map(function ($record, $key) {
+            })->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->map->reject(function ($model) {
+            })->map->reject(function($model) {
                 return is_null($model) || !$model->isPublished(); // Invalid references to archived or removed models where url record still exists.
             })->flatten();
         }, [$onlySingles]);
 
         if ($ignoredModel) {
-            $models = $models->reject(function ($model) use ($ignoredModel) {
+            $models = $models->reject(function($model) use ($ignoredModel) {
                 return (get_class($model) === get_class($ignoredModel) && $model->id === $ignoredModel->id);
             });
         }