thinktomorrow / chief

src/Management/Assistants/UrlAssistant.php

Spacing +5 added lines, -5 removed lines

@@ -32,10 +32,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();
@@ -59,7 +59,7 @@ 
             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), ],
                     ],
                     [],
                     [
@@ -67,7 +67,7 @@ 
                         '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 +7 added lines, -7 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);
         }
 
@@ -166,8 +166,8 @@ 
     {
         $fields = $this->fields();
 
-        foreach($this->assistants() as $assistant){
-            if( ! method_exists($assistant, 'fields')) continue;
+        foreach ($this->assistants() as $assistant) {
+            if (!method_exists($assistant, 'fields')) continue;
 
             $fields = $fields->merge($assistant->fields());
         }
@@ -204,7 +204,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);
@@ -242,7 +242,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

Braces +3 added lines, -1 removed lines

@@ -167,7 +167,9 @@
         $fields = $this->fields();
 
         foreach($this->assistants() as $assistant){
-            if( ! method_exists($assistant, 'fields')) continue;
+            if( ! method_exists($assistant, 'fields')) {
+                continue;
+            }
 
             $fields = $fields->merge($assistant->fields());
         }

src/Fields/Fields.php

Indentation +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function validate(array $data)
     {
         foreach($this->fields as $field){
-             $field->validator($data)->validate();
+                $field->validator($data)->validate();
         }
     }
 

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
 
     public function first(): ?Field
     {
-        if(!$this->any()) return null;
+        if (!$this->any()) return null;
 
         return reset($this->fields);
     }
@@ -46,7 +46,7 @@ 
 
     public function validate(array $data)
     {
-        foreach($this->fields as $field){
+        foreach ($this->fields as $field) {
              $field->validator($data)->validate();
         }
     }
@@ -121,8 +121,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;
@@ -142,7 +142,7 @@ 
     {
         $keyedFields = [];
 
-        foreach($fields as $field)
+        foreach ($fields as $field)
         {
             $keyedFields[$field->key] = $field;
         }
@@ -152,7 +152,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

Braces +3 added lines, -1 removed lines

@@ -24,7 +24,9 @@
 
     public function first(): ?Field
     {
-        if(!$this->any()) return null;
+        if(!$this->any()) {
+            return null;
+        }
 
         return reset($this->fields);
     }

src/Fields/Validators/FieldValidatorFactory.php

Spacing +4 added lines, -4 removed lines

@@ -19,15 +19,15 @@ 
 
     public function create(Field $field, array $data): Validator
     {
-        if(!$field->hasValidation()){
+        if (!$field->hasValidation()) {
             return new NullValidator();
         }
 
-        if($field->validation instanceof Validator){
+        if ($field->validation instanceof Validator) {
             return $field->validation;
         }
 
-        if($field->validation instanceof \Closure){
+        if ($field->validation instanceof \Closure) {
             $closure = $field->validation;
             return $closure($this->validatorFactory, $data);
         }
@@ -37,7 +37,7 @@ 
 
     private function composeValidatorFromRules(Field $field, array $data): Validator
     {
-        if(!is_array($field->validation) || !isset($field->validation[0])){
+        if (!is_array($field->validation) || !isset($field->validation[0])) {
             throw new \Exception('Invalid validation given. Rules should be passed as non-associative array.');
         }
 

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 '';
         }