thinktomorrow / chief

src/Site/Urls/ValidationRules/UniqueUrlSlugRule.php

Spacing +1 added lines, -1 removed lines

@@ -60,6 +60,6 @@
      */
     public function message()
     {
-        return 'De \'' . $this->failedDetails['slug'] . '\' link wordt in het ' . $this->failedDetails['locale'] . ' al door een andere pagina gebruikt.';
+        return 'De \''.$this->failedDetails['slug'].'\' link wordt in het '.$this->failedDetails['locale'].' al door een andere pagina gebruikt.';
     }
 }

src/Site/Urls/ProvidesUrl/ProvidingUrl.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
     /** @inheritdoc */
     public function baseUrlSegment(string $locale = null): string
     {
-        return BaseUrlSegment::find(isset(static::$baseUrlSegment) ? (array) static::$baseUrlSegment : [], $locale);
+        return BaseUrlSegment::find(isset(static::$baseUrlSegment) ? (array)static::$baseUrlSegment : [], $locale);
     }
 }

src/Site/Urls/ProvidesUrl/BaseUrlSegment.php

Spacing +3 added lines, -3 removed lines

@@ -10,8 +10,8 @@ 
 {
     public static function find(array $segments, string $locale = null)
     {
-        if(count($segments) < 1) return '/';
-        if(count($segments) == 1) return reset($segments);
+        if (count($segments) < 1) return '/';
+        if (count($segments) == 1) return reset($segments);
 
         // Localized value
         if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) return $segments[$key];
@@ -31,7 +31,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.

Braces +12 added lines, -4 removed lines

@@ -10,14 +10,22 @@
 {
     public static function find(array $segments, string $locale = null)
     {
-        if(count($segments) < 1) return '/';
-        if(count($segments) == 1) return reset($segments);
+        if(count($segments) < 1) {
+            return '/';
+        }
+        if(count($segments) == 1) {
+            return reset($segments);
+        }
 
         // Localized value
-        if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) return $segments[$key];
+        if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) {
+            return $segments[$key];
+        }
 
         // Fallback localized value
-        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) return $segments[$fallback_locale];
+        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) {
+            return $segments[$fallback_locale];
+        }
 
         // Fallback to first entry
         return reset($segments);

src/Site/Urls/UrlHelper.php

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
 
     public static function modelsByType(array $types, Model $ignoredModel = null, bool $online = true)
     {
-        $models = chiefMemoize('all-online-models-' . implode('_', $types), function () use ($types, $online) {
+        $models = chiefMemoize('all-online-models-'.implode('_', $types), function() use ($types, $online) {
             $builder = UrlRecord::whereNull('redirect_id')
                 ->select('model_type', 'model_id')
                 ->groupBy('model_type', 'model_id');
@@ -98,11 +98,11 @@ 
                 $builder->whereIn('model_type', $types);
             }
 
-            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) use ($online) {
+            })->map->reject(function($model) use ($online) {
                 if ($online) {
                     return is_null($model) || (public_method_exists($model, 'isPublished') && !$model->isPublished());
                 } // Invalid references to archived or removed models where url record still exists.
@@ -112,7 +112,7 @@ 
         });
 
         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);
             });
         }

src/Shared/Concerns/Translatable/TranslatableController.php

Spacing +3 added lines, -3 removed lines

@@ -23,15 +23,15 @@
             }
 
             foreach ($rules as $key => $rule) {
-                $translationrules['trans.' . $locale . '.' . $key] = $rule;
+                $translationrules['trans.'.$locale.'.'.$key] = $rule;
             }
 
             foreach ($attributes as $key => $attribute) {
-                $translationattributes['trans.' . $locale . '.' . $key] = $attribute;
+                $translationattributes['trans.'.$locale.'.'.$key] = $attribute;
             }
 
             foreach ($messages as $key => $message) {
-                $translationmessages['trans.' . $locale . '.' . $key] = $message;
+                $translationmessages['trans.'.$locale.'.'.$key] = $message;
             }
         }
 

src/Shared/Concerns/Translatable/Translatable.php

Spacing +2 added lines, -2 removed lines

@@ -146,7 +146,7 @@ 
         $available_locales = static::availableLocales();
         $current_locales = $this->translations->pluck('locale')->toArray();
 
-        return array_filter($available_locales, function ($v) use ($current_locales, $available) {
+        return array_filter($available_locales, function($v) use ($current_locales, $available) {
             return $available ? in_array($v, $current_locales) : !in_array($v, $current_locales);
         }, ARRAY_FILTER_USE_BOTH);
     }
@@ -159,7 +159,7 @@ 
     private function validateLocale($locale)
     {
         if (!in_array($locale, static::availableLocales())) {
-            throw new InvalidArgumentException('Locale [' . $locale . '] is not available');
+            throw new InvalidArgumentException('Locale ['.$locale.'] is not available');
         }
     }
 

src/Shared/Concerns/Viewable/ViewPath.php

Spacing +9 added lines, -9 removed lines

@@ -34,22 +34,22 @@ 
      */
     public function get(): string
     {
-        $basePath = $this->viewBasePath ? $this->viewBasePath . '.' : '';
+        $basePath = $this->viewBasePath ? $this->viewBasePath.'.' : '';
         $guessedParentViewName = $this->parent ? $this->parent->viewKey() : '';
         $guessedViewName = $this->viewable->viewKey();
 
         $viewPaths = [
-            $basePath . $guessedParentViewName . '.' . $guessedViewName,
-            $basePath . $guessedViewName,
-            $basePath . $guessedViewName . '.show',
-            $basePath . 'show',
+            $basePath.$guessedParentViewName.'.'.$guessedViewName,
+            $basePath.$guessedViewName,
+            $basePath.$guessedViewName.'.show',
+            $basePath.'show',
         ];
 
         // In case the collection set is made out of pages, we'll also allow to use the
         // generic collection page view for these sets as well as a fallback view
         if ($this->viewable instanceof Set && $this->viewable->first() instanceof ViewableContract) {
-            $viewPaths[] = $basePath . $guessedParentViewName . '.' . $this->viewable->first()->viewKey();
-            $viewPaths[] = $basePath . $this->viewable->first()->viewKey();
+            $viewPaths[] = $basePath.$guessedParentViewName.'.'.$this->viewable->first()->viewKey();
+            $viewPaths[] = $basePath.$this->viewable->first()->viewKey();
         }
 
         foreach ($viewPaths as $viewPath) {
@@ -60,8 +60,8 @@ 
             return $viewPath;
         }
 
-        if (!view()->exists($basePath . 'show')) {
-            throw new NotFoundView('Viewfile not found for [' . get_class($this->viewable) . ']. Make sure the view [' . $basePath . $guessedViewName . '] or the default view [' . $basePath . 'show] exists.');
+        if (!view()->exists($basePath.'show')) {
+            throw new NotFoundView('Viewfile not found for ['.get_class($this->viewable).']. Make sure the view ['.$basePath.$guessedViewName.'] or the default view ['.$basePath.'show] exists.');
         }
     }
 }

src/Shared/Concerns/Sortable.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public static function reorderAgainstSiblings(array $sequence)
     {
-        array_walk($sequence, function ($id, $i) {
+        array_walk($sequence, function($id, $i) {
             self::findOrFail($id)->reorder($i);
         });
     }

src/Shared/Concerns/Morphable/Morphables.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         }
 
         if (!class_exists($class)) {
-            throw new NotFoundMorphKey('No class found by morphkey [' . $class . ']. Make sure that the morphkey is a valid class reference.');
+            throw new NotFoundMorphKey('No class found by morphkey ['.$class.']. Make sure that the morphkey is a valid class reference.');
         }
 
         return new $class($attributes);