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/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

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)), '/');

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,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)), '/');
@@ -79,7 +79,7 @@ 
             'baseUrlSegment' => $this->baseUrlSegment,
             'hint' => null, // Hint placeholder to show url hint when it already exists
             'is_homepage' => ($this->value() === '/'),
-            'show' => !!$this->value(),// show input field or not
+            'show' => !!$this->value(), // show input field or not
         ]);
     }
 }

src/Management/Registration.php

Spacing +2 added lines, -2 removed lines

@@ -83,12 +83,12 @@
         }
 
         $manager = new \ReflectionClass($managerClass);
-        if (! $manager->implementsInterface(Manager::class)) {
+        if (!$manager->implementsInterface(Manager::class)) {
             throw new \InvalidArgumentException('Class ['.$managerClass.'] is expected to implement the ['.Manager::class.'] contract.');
         }
 
         $model = new \ReflectionClass($modelClass);
-        if (! $model->implementsInterface(ManagedModel::class)) {
+        if (!$model->implementsInterface(ManagedModel::class)) {
             throw new \InvalidArgumentException('Class ['.$modelClass.'] is expected to implement the ['.ManagedModel::class.'] contract.');
         }
     }

src/Urls/UrlSlugFields.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
 
     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');
 
@@ -29,7 +29,7 @@ 
             $fields[$key] = UrlSlugField::make($key)
                 ->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)).'/');
         }
 
         return $fields;
@@ -61,10 +61,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);
         }
 
@@ -77,7 +77,7 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
@@ -89,7 +89,7 @@ 
             $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/Fields/Fields.php

Spacing +3 added lines, -3 removed lines

@@ -123,8 +123,8 @@ 
 
     public function offsetSet($offset, $value)
     {
-        if (! $value instanceof Field) {
-            throw new \InvalidArgumentException('Passed value must be of type ' . Field::class);
+        if (!$value instanceof Field) {
+            throw new \InvalidArgumentException('Passed value must be of type '.Field::class);
         }
 
         $this->fields[$offset] = $value;
@@ -153,7 +153,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

src/Modules/ModuleManager.php

Spacing +5 added lines, -5 removed lines

@@ -63,15 +63,15 @@ 
     {
         $permission = 'update-page';
 
-        if (in_array($verb, ['index','show'])) {
+        if (in_array($verb, ['index', 'show'])) {
             $permission = 'view-page';
-        } elseif (in_array($verb, ['create','store'])) {
+        } elseif (in_array($verb, ['create', 'store'])) {
             $permission = 'create-page';
         } elseif (in_array($verb, ['delete'])) {
             $permission = 'delete-page';
         }
 
-        if (! auth()->guard('chief')->user()->hasPermissionTo($permission)) {
+        if (!auth()->guard('chief')->user()->hasPermissionTo($permission)) {
             throw NotAllowedManagerRoute::notAllowedPermission($permission, $this);
         }
     }
@@ -103,7 +103,7 @@ 
     public function saveFields(Request $request)
     {
         // Store the morph_key upon creation
-        if (! $this->model->morph_key) {
+        if (!$this->model->morph_key) {
             $this->model->morph_key = $this->model->morphKey();
         }
 
@@ -142,7 +142,7 @@ 
             if (is_array_empty($translation)) {
 
                 // Nullify all values
-                $trans[$locale] = array_map(function ($value) {
+                $trans[$locale] = array_map(function($value) {
                     return null;
                 }, $translation);
                 continue;