thinktomorrow / chief

src/Management/Assistants/UrlAssistant.php

Spacing +5 added lines, -5 removed lines

@@ -23,10 +23,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();
@@ -54,14 +54,14 @@ 
             InputField::make('url-slugs')
                 ->validation(
                     [
-                        'url-slugs' => ['array', 'min:1', new UniqueUrlSlugRule(($this->model && $this->model->exists) ? $this->model : null),],
+                        'url-slugs' => ['array', 'min:1', new UniqueUrlSlugRule(($this->model && $this->model->exists) ? $this->model : null), ],
                     ],
                     [],
                     [
                         'url-slugs.*' => 'taalspecifieke link',
                     ])
                 ->view('chief::back._fields.url-slugs')
-                ->viewData(['fields' => UrlSlugFields::fromModel($this->model) ]),
+                ->viewData(['fields' => UrlSlugFields::fromModel($this->model)]),
         ]);
     }
 

src/Management/AbstractManager.php

Spacing +6 added lines, -6 removed lines

@@ -72,11 +72,11 @@ 
             $this->filters()->apply($builder);
 
             $results = $builder->get();
-        } else {
+        }else {
             $results = $model::all();
         }
 
-        return $results->map(function ($model) {
+        return $results->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -144,7 +144,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -167,7 +167,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }
 
@@ -206,7 +206,7 @@ 
 
                 // Could be translatable field
                 if ($this->isTranslatableKey($field)) {
-                    $attribute = substr($field, strrpos($field, '.') + 1);
+                    $attribute = substr($field, strrpos($field, '.')+1);
                     $locale = substr($field, strlen('trans.'), 2);
 
                     return $this->model->getTranslationFor($attribute, $locale);
@@ -244,7 +244,7 @@ 
 
             // Make our media fields able to be translatable as well...
             if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) {
-                throw new \Exception('Cannot process the ' . $field->key . ' media field. Currently no support for translatable media files. We should fix this!');
+                throw new \Exception('Cannot process the '.$field->key.' media field. Currently no support for translatable media files. We should fix this!');
             }
 
             // Okay so this is a bit odd but since all translations are expected to be inside the trans

src/Nav/Nav.php

Spacing +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
 
     public static function fromKeys($keys)
     {
-        $keys = (array) $keys;
+        $keys = (array)$keys;
         $collection = collect();
 
         /** @var Managers */
@@ -33,7 +33,7 @@ 
 
     public static function fromTags($tags)
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
         $collection = collect();
 
         /** @var Managers */
@@ -52,9 +52,9 @@ 
 
     private static function fromManagers(Collection $collection)
     {
-        return new static(...$collection->reject(function ($manager) {
+        return new static(...$collection->reject(function($manager) {
             return !$manager->can('index');
-        })->map(function ($manager) {
+        })->map(function($manager) {
             return new NavItem($manager->details()->plural, $manager->route('index'), [
                 'key' => $manager->details()->key,
                 'tags' => app(Register::class)->filterByKey($manager->details()->key)->first()->tags()
@@ -64,7 +64,7 @@ 
 
     public function rejectKeys($keys)
     {
-        $keys = (array) $keys;
+        $keys = (array)$keys;
 
         foreach ($this->items as $k => $item) {
             if (in_array($item->details('key', ''), $keys)) {
@@ -77,7 +77,7 @@ 
 
     public function rejectTags($tags)
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         foreach ($this->items as $k => $item) {
             if (count(array_intersect($item->details('tags', []), $tags)) > 0) {
@@ -119,7 +119,7 @@ 
         $output = '';
 
         foreach ($this->items as $item) {
-            $output .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="'.$item->url().'">';
+            $output .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $output .= $title ?? ucfirst($item->title());
             $output .= '</a>';
         }
@@ -146,14 +146,14 @@ 
 
         $items = '';
         foreach ($this->items as $item) {
-            $items .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="'.$item->url().'">';
+            $items .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $items .= $item->title();
             $items .= '</a>';
         }
 
         // Surround within vue dropdown
         $output = '<dropdown>';
-        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'. ($title ?? 'Collecties') .'</span>';
+        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'.($title ?? 'Collecties').'</span>';
         $output .= '<div v-cloak class="dropdown-box inset-s">';
         $output .= $items;
         $output .= '</div>';

src/Filters/Filters.php

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
     public function anyApplied(): bool
     {
         foreach ($this->all() as $filter) {
-            if(request()->filled($filter->name)) {
+            if (request()->filled($filter->name)) {
                 return true;
             }
         }
@@ -48,8 +48,8 @@ 
     {
         $requestInput = request()->all();
 
-        return array_reduce($this->all(), function ($carry, Filter $filter) use ($requestInput) {
-            return $carry . $filter->render($requestInput);
+        return array_reduce($this->all(), function($carry, Filter $filter) use ($requestInput) {
+            return $carry.$filter->render($requestInput);
         }, '');
     }
 
@@ -65,14 +65,14 @@ 
 
     public function keys(): array
     {
-        return array_map(function (Filter $filter) {
+        return array_map(function(Filter $filter) {
             return $filter->key();
         }, $this->filters);
     }
 
     private function validateFilters(array $filters)
     {
-        array_map(function (Filter $filter) {
+        array_map(function(Filter $filter) {
         }, $filters);
     }
 
@@ -97,8 +97,8 @@ 
 
     public function offsetSet($offset, $value)
     {
-        if (! $value instanceof Filter) {
-            throw new \InvalidArgumentException('Passed value must be of type ' . Filter::class);
+        if (!$value instanceof Filter) {
+            throw new \InvalidArgumentException('Passed value must be of type '.Filter::class);
         }
 
         $this->filters[$offset] = $value;
@@ -116,7 +116,7 @@ 
 
     private function sanitizeFilters(array $filters)
     {
-        return array_map(function ($filter) {
+        return array_map(function($filter) {
             if (is_string($filter) && class_exists($filter)) {
                 return $filter::init();
             }

src/Concerns/Viewable/ViewPath.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
      */
     public function get(): string
     {
-        $basePath = $this->viewBasePath ? $this->viewBasePath . '.' : '';
+        $basePath = $this->viewBasePath ? $this->viewBasePath.'.' : '';
         $guessedParentViewName = $this->parent ? $this->parent->viewKey() : '';
         $guessedViewName = $this->viewable->viewKey();
 
@@ -53,14 +53,14 @@ 
         }
 
         foreach ($viewPaths as $viewPath) {
-            if (! view()->exists($viewPath)) {
+            if (!view()->exists($viewPath)) {
                 continue;
             }
 
             return $viewPath;
         }
 
-        if (! view()->exists($basePath.'show')){
+        if (!view()->exists($basePath.'show')) {
             throw new NotFoundView('Viewfile not found for ['.get_class($this->viewable).']. Make sure the default view ['.$basePath.'show] exists.');
         }
     }

src/Urls/UrlSlugField.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
         // If this is a '/' slug, it indicates the homepage for this locale. In this case,
         // we wont be trimming the slash
-        if($slug === '/') return $slug;
+        if ($slug === '/') return $slug;
 
         if ($this->startsWithBaseUrlSegment($slug)) {
             $slug = trim(substr($slug, strlen($this->baseUrlSegment)), '/');

Braces +3 added lines, -1 removed lines

@@ -39,7 +39,9 @@
 
         // If this is a '/' slug, it indicates the homepage for this locale. In this case,
         // we wont be trimming the slash
-        if($slug === '/') return $slug;
+        if($slug === '/') {
+            return $slug;
+        }
 
         if ($this->startsWithBaseUrlSegment($slug)) {
             $slug = trim(substr($slug, strlen($this->baseUrlSegment)), '/');

src/Management/Assistants/PublishAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
 
     public function findAll(): Collection
     {
-        return $this->model->published()->get()->map(function ($model) {
+        return $this->model->published()->get()->map(function($model) {
             return $this->managers->findByModel($model);
         });
     }
@@ -93,7 +93,7 @@ 
 
     public function guard($verb): Assistant
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this->manager);
         }
 
@@ -109,7 +109,7 @@ 
     {
         $label = $this->publicationStatusAsPlainLabel();
 
-        if ($plain ||  ! $this->hasPreviewUrl()) {
+        if ($plain || !$this->hasPreviewUrl()) {
             return $label;
         }
 
@@ -144,7 +144,7 @@ 
     public function previewUrl(): string
     {
         if (!$this->hasPreviewUrl()) {
-            throw new \Exception('Managed model ' . get_class($this->model) . ' should implement ' . ProvidesUrl::class);
+            throw new \Exception('Managed model '.get_class($this->model).' should implement '.ProvidesUrl::class);
         }
 
         return $this->model->previewUrl();