thinktomorrow / chief

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/Fields/Types/Field.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -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 +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 '';

src/Management/Application/StoringAndUpdatingFields.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 
             // Media fields are treated separately
             if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) {
-                if (! isset($this->saveMethods['_files'])) {
+                if (!isset($this->saveMethods['_files'])) {
                     $this->saveMethods['_files'] = ['field' => new Fields([$field]), 'method' => 'uploadMedia'];
                     continue;
                 }
@@ -36,7 +36,7 @@ 
             }
 
             // Custom set methods - default is the generic setField() method.
-            $methodName = 'set'. ucfirst(camel_case($field->key())) . 'Field';
+            $methodName = 'set'.ucfirst(camel_case($field->key())).'Field';
             (method_exists($manager, $methodName))
                 ? $manager->$methodName($field, $request)
                 : $manager->setField($field, $request);
@@ -48,7 +48,7 @@ 
 
     protected function detectCustomSaveMethods(Manager $manager, Field $field): bool
     {
-        $saveMethodName = 'save'. ucfirst(camel_case($field->key())) . 'Field';
+        $saveMethodName = 'save'.ucfirst(camel_case($field->key())).'Field';
 
         // Custom save method on assistant
         foreach ($manager->assistants() as $assistant) {

src/Management/Assistants/AssistedManager.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
      */
     public function isAssistedBy(string $assistant): bool
     {
-        return !! $this->getAssistantClass($assistant);
+        return !!$this->getAssistantClass($assistant);
     }
 
     public function assistants(): array
@@ -40,8 +40,8 @@ 
      */
     public function assistant(string $assistant): Assistant
     {
-        if (! $this->isAssistedBy($assistant)) {
-            throw new MissingAssistant('No assistant [' . $assistant . '] present on manager ' . get_class($this));
+        if (!$this->isAssistedBy($assistant)) {
+            throw new MissingAssistant('No assistant ['.$assistant.'] present on manager '.get_class($this));
         }
 
         $instance = app($this->getAssistantClass($assistant));

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/Management/AbstractManager.php

Spacing +6 added lines, -6 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);
         }
 
@@ -167,7 +167,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }
 
@@ -206,7 +206,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);
@@ -244,7 +244,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

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