thinktomorrow / chief

src/Urls/UrlSlugFields.php

Spacing +9 added lines, -9 removed lines

@@ -18,14 +18,14 @@ 
 
     public static function redirectsFromModel(ProvidesUrl $model)
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return !$record->isRedirect();
         })->sortByDesc('created_at');
 
         $fields = new static([]);
 
         foreach ($records as $record) {
-            $key = 'redirects-' . $record->locale . '-' . $record->slug;
+            $key = 'redirects-'.$record->locale.'-'.$record->slug;
             $fields[$key] = UrlSlugField::make($key)
                 ->setUrlRecord($record)
                 ->setFullUrl($model->resolveUrl($record->locale, $record->slug));
@@ -60,10 +60,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);
         }
 
@@ -76,19 +76,19 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = UrlRecord::getByModel($model)->reject(function ($record) {
+        $records = UrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
         foreach ($records as $record) {
-            if (!isset($fields['url-slugs.' . $record->locale])) {
+            if (!isset($fields['url-slugs.'.$record->locale])) {
                 continue;
             }
 
-            $fields['url-slugs.' . $record->locale]
+            $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/Relations/Relation.php

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) {
+        return $relations->map(function(Relation $relation) {
             $parent = $relation->parent;
             $parent->relation = $relation;
 
@@ -67,14 +67,14 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) use ($parent_type, $parent_id) {
+        return $relations->map(function(Relation $relation) use ($parent_type, $parent_id) {
 
             // It could be that the child itself is soft-deleted, if this is the case, we will ignore it and move on.
             if (!$child = $relation->child) {
                 if (!$relation->child()->withTrashed()->first()) {
 //                if ((!method_exists($childInstance, 'trashed')) || ! $childInstance->onlyTrashed()->find($relation->child_id)) {
                     // If we cannot retrieve it then he collection type is possibly off, this is a database inconsistency and should be addressed
-                    throw new \DomainException('Corrupt relation reference. Related child [' . $relation->child_type . '@' . $relation->child_id . '] could not be retrieved for parent [' . $parent_type . '@' . $parent_id . ']. Make sure the morph key can resolve to a valid class.');
+                    throw new \DomainException('Corrupt relation reference. Related child ['.$relation->child_type.'@'.$relation->child_id.'] could not be retrieved for parent ['.$parent_type.'@'.$parent_id.']. Make sure the morph key can resolve to a valid class.');
                 }
 
                 return null;
@@ -86,7 +86,7 @@ 
         })
 
             // In case of soft-deleted entries, this will be null and should be ignored. We make sure that keys are reset in case of removed child
-            ->reject(function ($child) {
+            ->reject(function($child) {
                 return is_null($child);
             })
             ->values();
@@ -103,10 +103,10 @@ 
 
     public static function deleteRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                 ->where('parent_id', $id);
-        })->orWhere(function ($query) use ($type, $id) {
+        })->orWhere(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();
@@ -118,7 +118,7 @@ 
 
     public static function deleteAllChildRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                 ->where('parent_id', $id);
         })->get();
@@ -130,7 +130,7 @@ 
 
     public static function deleteAllParentRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();

src/Relations/AvailableChildren.php

Spacing +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
 
     public function onlyModules(): Collection
     {
-        return $this->collection()->reject(function ($item) {
+        return $this->collection()->reject(function($item) {
             if ($item instanceof Page || $item instanceof StoredSetReference) {
                 return true;
             }
@@ -42,7 +42,7 @@ 
 
     public function onlyPages(): Collection
     {
-        return $this->collection()->filter(function ($item) {
+        return $this->collection()->filter(function($item) {
             if ($item instanceof Page) {
                 return true;
             }
@@ -72,12 +72,12 @@ 
 
         // Merging the results of all the pages and all the modules, then filter by the config
         // This prevents us from having duplicates and also reduces the query load.
-        $collection = $collection->filter(function ($page) use ($available_children_types) {
+        $collection = $collection->filter(function($page) use ($available_children_types) {
             return in_array(get_class($page), $available_children_types);
         });
 
         // Filter out our already loaded pages and modules
-        $remaining_children_types = collect($available_children_types)->reject(function ($type) {
+        $remaining_children_types = collect($available_children_types)->reject(function($type) {
             return (new $type() instanceof Page || new $type() instanceof Module);
         });
 
@@ -89,7 +89,7 @@ 
         }
 
         // Filter out our parent
-        return $this->collection = $collection->reject(function ($item) {
+        return $this->collection = $collection->reject(function($item) {
             if ($item instanceof $this->parent) {
                 return $item->id == $this->parent->id;
             }

src/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/Concerns/Sluggable/UniqueSlug.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         $this->model = $model;
         $this->blacklist = $blacklist;
 
-        $this->slugResolver = function ($slug) {
+        $this->slugResolver = function($slug) {
             return Str::slug($slug);
         };
     }
@@ -43,7 +43,7 @@ 
         $i = 1;
 
         while (!$this->isSlugUnique($slug, $entity)) {
-            $slug = $originalslug . '-' . $i;
+            $slug = $originalslug.'-'.$i;
             $i++;
         }
 

src/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/Menu/ChiefMenu.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
     {
         $collection = NodeCollection::fromArray($items);
 
-        $collection->mapRecursive(function ($node) {
+        $collection->mapRecursive(function($node) {
             return $node->replaceEntry((new MenuItem())->entry($node));
         });
 
@@ -42,7 +42,7 @@ 
     public function items(): NodeCollection
     {
         if (!$this->includeHidden) {
-            $this->collection = $this->collection->shake(function ($node) {
+            $this->collection = $this->collection->shake(function($node) {
                 return !$node->hidden_in_menu && !$node->draft;
             });
         }
@@ -55,22 +55,22 @@ 
         $this->collection = $this->items();
 
         if ($id) {
-            $this->collection = $this->collection->prune(function ($node) use ($id) {
+            $this->collection = $this->collection->prune(function($node) use ($id) {
                 return !in_array($id, $node->pluckAncestors('id'));
             });
         }
 
-        $menu = $this->collection->mapRecursive(function ($node) {
+        $menu = $this->collection->mapRecursive(function($node) {
             $entry = $node->entry();
             $label = $entry->label;
-            $entry->label = $node->depth() != 0 ? (str_repeat('-', $node->depth())) . '>' : '';
+            $entry->label = $node->depth() != 0 ? (str_repeat('-', $node->depth())).'>' : '';
             $entry->label .= $label;
 
             return $node->replaceEntry($entry);
         });
 
         $menuitems = collect();
-        $menu->flatten()->each(function ($node) use ($menuitems) {
+        $menu->flatten()->each(function($node) use ($menuitems) {
             $menuitems[] = [
                 'label' => $node->label,
                 'id'    => $node->id,

src/Menu/Application/UpdateMenu.php

Spacing +3 added lines, -3 removed lines

@@ -54,13 +54,13 @@ 
         if (in_array($menu->order, $sequence)) {
             foreach ($sequence as $id => $order) {
                 if ($order < $menu->order) {
-                } else {
+                }else {
                     $sequence[$id]++;
                 }
             }
         }
 
-        $sequence = $sequence + [$menu->id => $menu->order];
+        $sequence = $sequence+[$menu->id => $menu->order];
 
         asort($sequence);
 
@@ -69,7 +69,7 @@ 
 
     private function reorder(array $sequence)
     {
-        array_walk($sequence, function ($order, $id) {
+        array_walk($sequence, function($order, $id) {
             MenuItem::withoutGlobalScope(SoftDeletingScope::class)
                 ->where('id', $id)
                 ->update(['order' => $order]);

src/Menu/Menu.php

Spacing +4 added lines, -4 removed lines

@@ -26,14 +26,14 @@ 
     {
         $types = config('thinktomorrow.chief.menus', []);
 
-        return collect($types)->map(function ($menu, $key) {
+        return collect($types)->map(function($menu, $key) {
             return new static($key, $menu['label'], $menu['view']);
         });
     }
 
     public static function find($key): ?self
     {
-        return static::all()->filter(function ($menu) use ($key) {
+        return static::all()->filter(function($menu) use ($key) {
             return $menu->key() == $key;
         })->first();
     }
@@ -82,10 +82,10 @@ 
     {
         $menu = [];
 
-        $this->items()->each(function ($item) use (&$menu) {
+        $this->items()->each(function($item) use (&$menu) {
             $menu[] = sprintf('<li><a href="%s">%s</a></li>', $item->url, $item->label);
         });
 
-        return '<ul>' . implode('', $menu) . '</ul>';
+        return '<ul>'.implode('', $menu).'</ul>';
     }
 }