thinktomorrow / chief

src/Management/Details/Sections.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
     public function __toString()
     {
-        return (string) $this->get('key');
+        return (string)$this->get('key');
     }
 
     /**

src/PageBuilder/PresentSections.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
             $this->addModelToCollection($i, $child);
         }
 
-        return $this->sets->values()->map(function (ViewableContract $child) {
+        return $this->sets->values()->map(function(ViewableContract $child) {
             return ($this->withSnippets && method_exists($child, 'withSnippets'))
                 ? $child->withSnippets()->setViewParent($this->parent)->renderView()
                 : $child->setViewParent($this->parent)->renderView();
@@ -95,7 +95,7 @@ 
     {
         // Only published pages you fool!
         // TODO: check for assistant instead of method existence
-        if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
+        if (method_exists($model, 'isPublished') && !$model->isPublished()) {
             return;
         }
 

src/Pages/PageManager.php

Indentation +5 added lines, -5 removed lines

@@ -87,11 +87,11 @@
         return parent::fields()->add(
             $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.'),
+                                        ->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.'),
             InputField::make('seo_title')
                 ->translatable($this->model->availableLocales())
                 ->label('Zoekmachine titel'),

Spacing +8 added lines, -8 removed lines

@@ -61,15 +61,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);
         }
     }
@@ -130,7 +130,7 @@ 
     public function fieldArrangement($key = null): FieldArrangement
     {
         if ($key == 'create') {
-            return new FieldArrangement($this->fieldsWithAssistantFields()->filterBy(function ($field) {
+            return new FieldArrangement($this->fieldsWithAssistantFields()->filterBy(function($field) {
                 return in_array($field->key, ['title']);
             }));
         }
@@ -152,8 +152,8 @@ 
         if ($this->model->id) {
             return parent::details()
                 ->set('title', $this->model->title)
-                ->set('intro', 'Aangepast ' . $this->model->updated_at->format('d/m/Y H:i'))
-                ->set('context', '<span class="inline-s">' . $this->assistant('publish')->publicationStatusAsLabel() . '</span>');
+                ->set('intro', 'Aangepast '.$this->model->updated_at->format('d/m/Y H:i'))
+                ->set('context', '<span class="inline-s">'.$this->assistant('publish')->publicationStatusAsLabel().'</span>');
         }
 
         return parent::details();
@@ -162,7 +162,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();
         }
 
@@ -207,7 +207,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/Sets/Set.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,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) {
+        return $this->map(function($item) {
             return ($this->viewParent && $item instanceof ViewableContract)
                 ? $item->setViewParent($this->viewParent)->renderView()
                 : ($item->content ?? '');
@@ -79,7 +79,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);
     }
@@ -96,7 +96,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/Urls/UrlRecord.php

Indentation +5 added lines, -5 removed lines

@@ -65,8 +65,8 @@ 
     public static function getByModel(Model $model)
     {
         return static::where('model_type', $model->getMorphClass())
-                     ->where('model_id', $model->id)
-                     ->get();
+                        ->where('model_id', $model->id)
+                        ->get();
     }
 
     public function replaceAndRedirect(array $values): UrlRecord
@@ -119,9 +119,9 @@ 
         if ($ignoredModel) {
             $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
                 $query->select('id')
-                      ->from('chief_urls')
-                      ->where('model_type', '=', $ignoredModel->getMorphClass())
-                      ->where('model_id', '=', $ignoredModel->id);
+                        ->from('chief_urls')
+                        ->where('model_type', '=', $ignoredModel->getMorphClass())
+                        ->where('model_id', '=', $ignoredModel->id);
             });
         }
 

Spacing +5 added lines, -5 removed lines

@@ -115,12 +115,12 @@ 
             $builder->where('locale', $locale);
         }
 
-        if (! $includeRedirects) {
+        if (!$includeRedirects) {
             $builder->whereNull('redirect_id');
         }
 
         if ($ignoredModel) {
-            $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
+            $builder->whereNotIn('id', function($query) use ($ignoredModel) {
                 $query->select('id')
                       ->from('chief_urls')
                       ->where('model_type', '=', $ignoredModel->getMorphClass())
@@ -133,16 +133,16 @@ 
 
     public static function allOnlineModels(): array
     {
-        return chiefMemoize('all-online-models', function () {
+        return chiefMemoize('all-online-models', function() {
 
             $liveUrlRecords = static::whereNull('redirect_id')->select('model_type', 'model_id')->groupBy('model_type', 'model_id')->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
             });
 
             // Get model for each of these records...
-            $models = $liveUrlRecords->map(function($record, $key){
+            $models = $liveUrlRecords->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->each->reject(function ($model) {
+            })->each->reject(function($model) {
                 // Invalid references to archived or removed models where url record still exists.
                 return is_null($model);
             })->flatten();

src/Urls/MemoizedUrlRecord.php

Spacing +2 added lines, -2 removed lines

@@ -8,14 +8,14 @@
 {
     public static function findByModel(Model $model, string $locale = null): UrlRecord
     {
-        return chiefMemoize('url-records-find-by-model', function ($model, $locale = null) {
+        return chiefMemoize('url-records-find-by-model', function($model, $locale = null) {
             return parent::findByModel($model, $locale);
         }, [$model, $locale]);
     }
 
     public static function getByModel(Model $model)
     {
-        return chiefMemoize('url-records-get-by-model', function ($model) {
+        return chiefMemoize('url-records-get-by-model', function($model) {
             return parent::getByModel($model);
         }, [$model]);
     }

src/Urls/SaveUrlSlugs.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     private function saveRecord(string $locale, ?string $slug)
     {
         // Existing ones for this locale?
-        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function ($record) use ($locale) {
+        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 !$record->isRedirect()
@@ -55,14 +55,14 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
         } elseif ($nonRedirectsWithSameLocale->isEmpty()) {
             $this->createRecord($locale, $slug);
-        } else {
+        }else {
             // Only replace the existing records that differ from the current passed slugs
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) use ($slug) {
                 if ($existingRecord->slug != $slug) {
                     $existingRecord->replaceAndRedirect(['slug' => $slug]);
                 }
@@ -87,12 +87,12 @@ 
      */
     private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
     {
-        $existingRecords->filter(function ($record) use ($locale) {
+        $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 $record->isRedirect()
             );
-        })->each(function ($existingRecord) use ($slug) {
+        })->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug == $slug) {
                 $existingRecord->delete();
             }
@@ -106,7 +106,7 @@ 
      */
     private function prependBaseUrlSegment(string $slug, $locale): string
     {
-        $slugWithBaseSegment = $this->model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $this->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.

src/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -39,11 +39,11 @@ 
                 return static::createRedirect($model->url($locale));
             }
 
-            if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
+            if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
-                if (! PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request ['. $slug .'] but it is not published.');
+                if (!PreviewMode::fromRequest()->check()) {
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -54,7 +54,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request ['. $slug .'] for locale ['.$locale.'].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)

src/Concerns/Viewable/Viewable.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // If no view has been created for this model, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We consider the 'content' attribute to be a default for our copy.
-        return isset($this->content) ? (string) $this->content : '';
+        return isset($this->content) ? (string)$this->content : '';
     }
 
     public function setViewParent(ActsAsParent $parent): ViewableContract
@@ -64,7 +64,7 @@ 
         }
 
         if (config('thinktomorrow.chief.strict')) {
-            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to ' . get_class($this));
+            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to '.get_class($this));
         };
 
         return '';