thinktomorrow / chief

src/FlatReferences/FlatReferencePresenter.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
 {
     public static function toSelectValues(Collection $collection): Collection
     {
-        return $collection->map(function (ProvidesFlatReference $item) {
+        return $collection->map(function(ProvidesFlatReference $item) {
             return [
                 'id'    => $item->flatReference()->get(),
                 'label' => $item->flatReferenceLabel(),
@@ -21,10 +21,10 @@ 
     {
         $grouped = [];
 
-        static::toSelectValues($collection)->each(function ($item) use (&$grouped) {
+        static::toSelectValues($collection)->each(function($item) use (&$grouped) {
             if (isset($grouped[$item['group']])) {
                 $grouped[$item['group']]['values'][] = $item;
-            } else {
+            }else {
                 $grouped[$item['group']] = ['group' => $item['group'], 'values' => [$item]];
             }
         });

src/FlatReferences/FlatReferenceCollection.php

Spacing +3 added lines, -3 removed lines

@@ -21,16 +21,16 @@
             $referenceStrings = [];
         }
 
-        return (new Collection($referenceStrings))->reject(function ($referenceString) {
+        return (new Collection($referenceStrings))->reject(function($referenceString) {
             return is_null($referenceString);
-        })->map(function ($referenceString) {
+        })->map(function($referenceString) {
             return FlatReferenceFactory::fromString($referenceString)->instance();
         });
     }
 
     public function toFlatReferences(): Collection
     {
-        return (new Collection($this->all()))->map(function (ProvidesFlatReference $item) {
+        return (new Collection($this->all()))->map(function(ProvidesFlatReference $item) {
             return $item->flatReference()->get();
         });
     }

src/FlatReferences/FlatReferenceFactory.php

Spacing +3 added lines, -3 removed lines

@@ -7,7 +7,7 @@ 
     public static function fromString(string $reference): FlatReference
     {
         if (false == strpos($reference, '@')) {
-            throw new \InvalidArgumentException('Invalid reference composition. A flat reference should honour schema <class>@<id>. [' . $reference . '] was passed instead.');
+            throw new \InvalidArgumentException('Invalid reference composition. A flat reference should honour schema <class>@<id>. ['.$reference.'] was passed instead.');
         }
 
         list($className, $id) = explode('@', $reference);
@@ -15,8 +15,8 @@ 
         $instance = app($className);
         $instance->{$instance->getKeyName()} = $id;
 
-        if (! method_exists($instance, 'flatReference')) {
-            throw new \InvalidArgumentException('Instance created from model reference [' . $reference . '] was expected to have a method of flatReference() but is has not.');
+        if (!method_exists($instance, 'flatReference')) {
+            throw new \InvalidArgumentException('Instance created from model reference ['.$reference.'] was expected to have a method of flatReference() but is has not.');
         }
 
         return $instance->flatReference();

src/Pages/PageManager.php

Indentation +5 added lines, -5 removed lines

@@ -90,11 +90,11 @@
         return new Fields([
             $this->pageBuilderField(),
             InputField::make('title')->translatable($this->model->availableLocales())
-                                     ->validation('required-fallback-locale|max:200', [], [
-                                         'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
-                                     ])
-                                     ->label('De titel van je '.$this->model->labelSingular ?? 'pagina')
-                                     ->description('Dit is de titel die zal worden getoond in de overzichten en modules.<br> Deze zal gebruikt worden als interne titel en slug van de nieuwe pagina.'),
+                                        ->validation('required-fallback-locale|max:200', [], [
+                                            'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
+                                        ])
+                                        ->label('De titel van je '.$this->model->labelSingular ?? 'pagina')
+                                        ->description('Dit is de titel die zal worden getoond in de overzichten en modules.<br> Deze zal gebruikt worden als interne titel en slug van de nieuwe pagina.'),
             InputField::make('slug')
                 ->translatable($this->model->availableLocales())
                 ->validation($this->model->id

Spacing +11 added lines, -11 removed lines

@@ -39,7 +39,7 @@ 
     {
         parent::__construct($registration);
 
-        $this->uniqueSlug = UniqueSlug::make(new PageTranslation)->slugResolver(function ($value) {
+        $this->uniqueSlug = UniqueSlug::make(new PageTranslation)->slugResolver(function($value) {
             return str_slug_slashed($value);
         });
     }
@@ -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);
         }
     }
@@ -98,13 +98,13 @@ 
             InputField::make('slug')
                 ->translatable($this->model->availableLocales())
                 ->validation($this->model->id
-                    ? 'required-fallback-locale|unique:page_translations,slug,' . $this->model->id . ',page_id'
+                    ? 'required-fallback-locale|unique:page_translations,slug,'.$this->model->id.',page_id'
                     : 'required-fallback-locale|unique:page_translations,slug', [], [
                     'trans.'.config('app.fallback_locale', 'nl').'.slug' => 'slug'
                 ])
                 ->label('Link')
                 ->description('De unieke url verwijzing naar deze pagina.')
-                ->prepend(collect($this->model->availableLocales())->mapWithKeys(function ($locale) {
+                ->prepend(collect($this->model->availableLocales())->mapWithKeys(function($locale) {
                     return [$locale => url($this->model->baseUrlSegment($locale)).'/'];
                 })->all()),
             InputField::make('seo_title')
@@ -140,7 +140,7 @@ 
     public function fieldArrangement($key = null): FieldArrangement
     {
         if ($key == 'create') {
-            return new FieldArrangement($this->fields()->filterBy(function ($field) {
+            return new FieldArrangement($this->fields()->filterBy(function($field) {
                 return $field->key == 'title';
             }));
         }
@@ -159,8 +159,8 @@ 
         if ($this->model->id) {
             return parent::details()
                 ->set('title', $this->model->title)
-                ->set('intro', 'laatst aangepast op ' . $this->model->updated_at->format('d/m/Y H:i'))
-                ->set('context', '<span class="inline-s">' . $this->assistant('publish')->publicationStatusAsLabel() . '</span>');
+                ->set('intro', 'laatst aangepast op '.$this->model->updated_at->format('d/m/Y H:i'))
+                ->set('context', '<span class="inline-s">'.$this->assistant('publish')->publicationStatusAsLabel().'</span>');
         }
 
         return parent::details();
@@ -169,7 +169,7 @@ 
     public function saveFields(): Manager
     {
         // Store the morph_key upon creation
-        if (! $this->model->morph_key) {
+        if (!$this->model->morph_key) {
             $this->model->morph_key = $this->model->morphKey();
         }
 
@@ -208,7 +208,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;

src/Pages/PublishedFilter.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function apply($value = null): Closure
     {
-        return function ($query) use ($value) {
+        return function($query) use ($value) {
             if ($value == 'all') {
                 return $query;
             }

src/Pages/Console/GeneratePage.php

Spacing +17 added lines, -17 removed lines

@@ -37,10 +37,10 @@ 
 
         // Set required paths
         $this->dirs = ['base' => $this->settings['base_path'] ?? base_path()];
-        $this->dirs['model'] = $this->settings['model_path'] ?? $this->dirs['base'] .'/'. config('thinktomorrow.chief.domain.path', 'src');
-        $this->dirs['views'] = $this->settings['views_path'] ?? $this->dirs['base'] . '/resources/views';
-        $this->dirs['controller'] = $this->settings['controller_path'] ?? $this->dirs['base'] . '/app/Http/Controllers';
-        $this->files['routes'] = $this->settings['routes_file'] ?? $this->dirs['base'] . '/routes/web.php';
+        $this->dirs['model'] = $this->settings['model_path'] ?? $this->dirs['base'].'/'.config('thinktomorrow.chief.domain.path', 'src');
+        $this->dirs['views'] = $this->settings['views_path'] ?? $this->dirs['base'].'/resources/views';
+        $this->dirs['controller'] = $this->settings['controller_path'] ?? $this->dirs['base'].'/app/Http/Controllers';
+        $this->files['routes'] = $this->settings['routes_file'] ?? $this->dirs['base'].'/routes/web.php';
     }
 
     public function handle()
@@ -62,8 +62,8 @@ 
     private function publishModel()
     {
         $this->publishFile(
-            __DIR__ . '/stubs/model.php.stub',
-            $to = $this->dirs['model'] . '/' . ucfirst($this->plural) . '/' . ucfirst($this->singular) . '.php',
+            __DIR__.'/stubs/model.php.stub',
+            $to = $this->dirs['model'].'/'.ucfirst($this->plural).'/'.ucfirst($this->singular).'.php',
             'model'
         );
 
@@ -73,8 +73,8 @@ 
     private function publishController()
     {
         $this->publishFile(
-            __DIR__ . '/stubs/controller.php.stub',
-            $to = $this->dirs['controller'] . '/' . ucfirst($this->plural) . '/' . ucfirst($this->singular) . '.php',
+            __DIR__.'/stubs/controller.php.stub',
+            $to = $this->dirs['controller'].'/'.ucfirst($this->plural).'/'.ucfirst($this->singular).'.php',
             'controller'
         );
     }
@@ -113,8 +113,8 @@ 
     private function modelTraits()
     {
         return [
-            '\\' . Publishable::class,
-            '\\' . Sortable::class,
+            '\\'.Publishable::class,
+            '\\'.Sortable::class,
             'q' => 'Proceed.',
         ];
     }
@@ -129,7 +129,7 @@ 
     protected function publishFile($from, $to, $type)
     {
         if ($this->filesystem->exists($to) && !$this->option('force')) {
-            if (!$this->confirm('File [' . $to . '] already exists? Overwrite this file?')) {
+            if (!$this->confirm('File ['.$to.'] already exists? Overwrite this file?')) {
                 return;
             }
         }
@@ -168,7 +168,7 @@ 
 
         $to = str_replace($this->dirs['base'], '', realpath($to));
 
-        $this->line('<info>Copied ' . $type . '</info> <comment>[' . $from . ']</comment> <info>To</info> <comment>[' . $to . ']</comment>');
+        $this->line('<info>Copied '.$type.'</info> <comment>['.$from.']</comment> <info>To</info> <comment>['.$to.']</comment>');
     }
 
     protected function replacePlaceholders($content)
@@ -187,16 +187,16 @@ 
     private function generateImportStatements(): string
     {
         return collect(['\\'.Page::class])
-            ->map(function ($statement) {
-                return 'use ' . $statement . ";\n    ";
+            ->map(function($statement) {
+                return 'use '.$statement.";\n    ";
             })->implode('');
     }
 
     private function generateTraitStatements(): string
     {
         return collect($this->chosenTraits)
-            ->map(function ($statement) {
-                return 'use ' . $statement . ";\n    ";
+            ->map(function($statement) {
+                return 'use '.$statement.";\n    ";
             })->implode('');
     }
 
@@ -215,7 +215,7 @@ 
 
         // We make an estimated guess based on the project name. At Think Tomorrow, we
         // have a src folder which is PSR-4 namespaced by the project name itself.
-        return str_replace('\\\\', '\\', ucfirst(config('thinktomorrow.chief.domain.namespace', 'App')).'\\'. ucfirst($this->plural));
+        return str_replace('\\\\', '\\', ucfirst(config('thinktomorrow.chief.domain.namespace', 'App')).'\\'.ucfirst($this->plural));
     }
 
     private function addToConfig($configKey, $value)

src/Pages/Page.php

Spacing +4 added lines, -4 removed lines

@@ -115,9 +115,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 '';
@@ -175,7 +175,7 @@ 
             return '';
         }
 
-        $parameters = trim($this->baseUrlSegment($locale) . '/' . $slug, '/');
+        $parameters = trim($this->baseUrlSegment($locale).'/'.$slug, '/');
 
         $routeName = Homepage::is($this)
                         ? config('thinktomorrow.chief.routes.pages-home', 'pages.home')
@@ -238,7 +238,7 @@ 
         ];
 
         foreach ($viewPaths as $viewPath) {
-            if (! view()->exists($viewPath)) {
+            if (!view()->exists($viewPath)) {
                 continue;
             }
 

src/Sets/Set.php

Spacing +10 added lines, -10 removed lines

@@ -57,23 +57,23 @@ 
     private function presentRawValueForParent(ActsAsParent $parent): string
     {
         $viewPaths = [
-            'front.modules.'. $parent->viewKey().'.'.$this->viewKey(),
+            'front.modules.'.$parent->viewKey().'.'.$this->viewKey(),
             'front.modules.'.$this->viewKey(),
-            'modules.'. $parent->viewKey().'.'.$this->viewKey(),
+            'modules.'.$parent->viewKey().'.'.$this->viewKey(),
             'modules.'.$this->viewKey(),
         ];
 
         // 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->first() instanceof PresentForParent) {
-            $viewPaths[] = 'front.modules.'. $parent->viewKey().'.'.$this->first()->viewKey();
-            $viewPaths[] = 'front.modules.'. $this->first()->viewKey();
-            $viewPaths[] = 'modules.'. $parent->viewKey().'.'.$this->first()->viewKey();
-            $viewPaths[] = 'modules.'. $this->first()->viewKey();
+            $viewPaths[] = 'front.modules.'.$parent->viewKey().'.'.$this->first()->viewKey();
+            $viewPaths[] = 'front.modules.'.$this->first()->viewKey();
+            $viewPaths[] = 'modules.'.$parent->viewKey().'.'.$this->first()->viewKey();
+            $viewPaths[] = 'modules.'.$this->first()->viewKey();
         }
 
         foreach ($viewPaths as $viewPath) {
-            if (! view()->exists($viewPath)) {
+            if (!view()->exists($viewPath)) {
                 continue;
             }
 
@@ -88,7 +88,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) use ($parent) {
+        return $this->map(function($item) use ($parent) {
             return ($item instanceof PresentForParent)
                 ? $item->presentForParent($parent)
                 : ($item->content ?? '');
@@ -122,7 +122,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, $currentPage))->setPath($path);
     }
@@ -139,7 +139,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->count(), $perPage, $currentPage))->setPath($path);
     }

src/Sets/SetReference.php

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,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(
@@ -49,14 +49,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();
     }
@@ -73,9 +73,9 @@ 
 
         $this->validateAction($class, $method);
 
-        $result = call_user_func_array([app($class),$method], $this->parameters);
+        $result = call_user_func_array([app($class), $method], $this->parameters);
 
-        if (! $result instanceof Set && $result instanceof Collection) {
+        if (!$result instanceof Set && $result instanceof Collection) {
             return new Set($result->all(), $this->key);
         }
 
@@ -107,7 +107,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.');
         }