thinktomorrow / chief

src/Fields/Types/Field.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -27,7 +27,7 @@ 
     public function validation(...$arguments)
     {
         // If a Closure or Validator is passed, we do not want to pass it as an array.
-        if(count($arguments) == 1 && !is_array($arguments)){
+        if (count($arguments) == 1 && !is_array($arguments)) {
             $this->values['validation'] = reset($arguments);
 
             return $this;
@@ -92,7 +92,7 @@ 
      */
     public function view(string $view = null)
     {
-        if($view){
+        if ($view) {
             $this->values['view'] = $view;
             return $this;
         }
@@ -102,7 +102,7 @@ 
 
     public function viewData(array $viewData = [])
     {
-        if($viewData){
+        if ($viewData) {
             $this->values['viewData'] = $viewData;
             return $this;
         }
@@ -142,7 +142,7 @@ 
         }
 
         if (!in_array($name, ['label', 'key', 'description', 'column', 'name', 'prepend', 'append'])) {
-            throw new \InvalidArgumentException('Cannot set value by ['. $name .'].');
+            throw new \InvalidArgumentException('Cannot set value by ['.$name.'].');
         }
 
         $this->values[$name] = $arguments[0];

src/Concerns/Viewable/Viewable.php

Spacing +7 added lines, -7 removed lines

@@ -29,15 +29,15 @@ 
         try {
             return view($this->viewPath(), $this->viewData())->render();
         }
-        catch(NotFoundView $e) {
-            if(config('thinktomorrow.chief.strict')){
+        catch (NotFoundView $e) {
+            if (config('thinktomorrow.chief.strict')) {
                 throw $e;
             }
         }
 
         // 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
@@ -56,14 +56,14 @@ 
      */
     public function viewKey(): string
     {
-        if(property_exists($this, 'viewKey') && isset($this->viewKey)) return $this->viewKey;
+        if (property_exists($this, 'viewKey') && isset($this->viewKey)) return $this->viewKey;
 
-        if($this instanceof MorphableContract){
+        if ($this instanceof MorphableContract) {
             return $this->morphKey();
         }
 
-        if(config('thinktomorrow.chief.strict')) {
-            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to ' . get_class($this));
+        if (config('thinktomorrow.chief.strict')) {
+            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to '.get_class($this));
         };
 
         return '';

Braces +4 added lines, -3 removed lines

@@ -28,8 +28,7 @@ 
     {
         try {
             return view($this->viewPath(), $this->viewData())->render();
-        }
-        catch(NotFoundView $e) {
+        } catch(NotFoundView $e) {
             if(config('thinktomorrow.chief.strict')){
                 throw $e;
             }
@@ -56,7 +55,9 @@ 
      */
     public function viewKey(): string
     {
-        if(property_exists($this, 'viewKey') && isset($this->viewKey)) return $this->viewKey;
+        if(property_exists($this, 'viewKey') && isset($this->viewKey)) {
+            return $this->viewKey;
+        }
 
         if($this instanceof MorphableContract){
             return $this->morphKey();

src/Concerns/Viewable/ViewPath.php

Spacing +2 added lines, -2 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,7 +53,7 @@ 
         }
 
         foreach ($viewPaths as $viewPath) {
-            if (! view()->exists($viewPath)) {
+            if (!view()->exists($viewPath)) {
                 continue;
             }
 

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

@@ -60,15 +60,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);
         }
     }
@@ -125,7 +125,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 $field->key == 'title';
             }));
         }
@@ -144,8 +144,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();
@@ -154,7 +154,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();
         }
 
@@ -192,7 +192,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/Page.php

Spacing +7 added lines, -7 removed lines

@@ -73,8 +73,8 @@ 
     {
         $this->constructWithSnippets();
 
-        if(!isset($this->baseViewPath)) {
-            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.pages','pages');
+        if (!isset($this->baseViewPath)) {
+            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.pages', 'pages');
         }
 
         parent::__construct($attributes);
@@ -117,9 +117,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 '';
@@ -156,14 +156,14 @@ 
     /** @inheritdoc */
     public function url(string $locale = null): string
     {
-        if(!$locale) $locale = app()->getLocale();
+        if (!$locale) $locale = app()->getLocale();
 
-        try{
+        try {
             $slug = MemoizedUrlRecord::findByModel($this, $locale)->slug;
 
             return $this->resolveUrl($locale, [$slug]);
         }
-        catch(UrlRecordNotFound $e)
+        catch (UrlRecordNotFound $e)
         {
             return '';
         }

Braces +4 added lines, -3 removed lines

@@ -156,14 +156,15 @@
     /** @inheritdoc */
     public function url(string $locale = null): string
     {
-        if(!$locale) $locale = app()->getLocale();
+        if(!$locale) {
+            $locale = app()->getLocale();
+        }
 
         try{
             $slug = MemoizedUrlRecord::findByModel($this, $locale)->slug;
 
             return $this->resolveUrl($locale, [$slug]);
-        }
-        catch(UrlRecordNotFound $e)
+        } catch(UrlRecordNotFound $e)
         {
             return '';
         }

src/Sets/Set.php

Spacing +6 added lines, -6 removed lines

@@ -24,8 +24,8 @@ 
     {
         $this->viewKey = $viewKey;
 
-        if(!isset($this->baseViewPath)) {
-            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.sets','sets');
+        if (!isset($this->baseViewPath)) {
+            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.sets', 'sets');
         }
 
         $this->constructWithSnippets();
@@ -40,11 +40,11 @@ 
 
     public function renderView(): string
     {
-        if($result = $this->baseRenderView()){ return $result; }
+        if ($result = $this->baseRenderView()) { return $result; }
 
         // 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 ?? '');
@@ -77,7 +77,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);
     }
@@ -94,7 +94,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/Common/Helpers/Memoize.php

Spacing +2 added lines, -2 removed lines

@@ -16,9 +16,9 @@
     public function run(\Closure $closure, array $parameters = [])
     {
         // construct cachekey
-        $cachekey = $this->baseKey.':'.md5(implode('',$parameters));
+        $cachekey = $this->baseKey.':'.md5(implode('', $parameters));
 
-        if(isset(static::$cache[$cachekey])){
+        if (isset(static::$cache[$cachekey])) {
             return static::$cache[$cachekey];
         }
 

src/Modules/Module.php

Spacing +2 added lines, -2 removed lines

@@ -56,8 +56,8 @@
     {
         $this->constructWithSnippets();
 
-        if(!isset($this->baseViewPath)) {
-            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.modules','modules');
+        if (!isset($this->baseViewPath)) {
+            $this->baseViewPath = config('thinktomorrow.chief.base-view-paths.modules', 'modules');
         }
 
         parent::__construct($attributes);

src/Urls/UrlRecord.php

Indentation +5 added lines, -5 removed lines

@@ -63,8 +63,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
@@ -117,9 +117,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 +7 added lines, -7 removed lines

@@ -22,14 +22,14 @@ 
     public static function findBySlug(string $slug, string $locale): UrlRecord
     {
         // Clear the input from any trailing slashes.
-        if($slug != '/'){ $slug = trim($slug,'/'); }
+        if ($slug != '/') { $slug = trim($slug, '/'); }
 
         $record = static::where('slug', $slug)
                         ->where('locale', $locale)
                         ->orderBy('redirect_id', 'ASC')
                         ->first();
 
-        if(!$record){
+        if (!$record) {
             throw new UrlRecordNotFound('No url record found by slug ['.$slug.'] for locale ['.$locale.'].');
         }
 
@@ -53,7 +53,7 @@ 
             ->orderBy('redirect_id', 'ASC')
             ->first();
 
-        if(!$record){
+        if (!$record) {
             throw new UrlRecordNotFound('No url record found for model ['.$model->getMorphClass().'@'.$model->id.'] for locale ['.$locale.'].');
         }
 
@@ -82,7 +82,7 @@ 
 
     public function redirectTo(self $record = null): ?UrlRecord
     {
-        if(!$record){
+        if (!$record) {
             return $this->isRedirect() ? static::find($this->redirect_id) : null;
         }
 
@@ -106,15 +106,15 @@ 
     {
         $builder = static::where('slug', $slug);
 
-        if($locale) {
+        if ($locale) {
             $builder->where('locale', $locale);
         }
 
-        if( ! $includeRedirects){
+        if (!$includeRedirects) {
             $builder->whereNull('redirect_id');
         }
 
-        if($ignoredModel){
+        if ($ignoredModel) {
             $builder->whereNotIn('id', function($query) use($ignoredModel){
                 $query->select('id')
                       ->from('chief_urls')