thinktomorrow / chief

src/Media/Application/AbstractMediaFieldHandler.php

Spacing +4 added lines, -4 removed lines

@@ -141,10 +141,10 @@ 
             return $filename;
         }
 
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename = substr($filename, 0, strrpos($filename, '.'));
 
-        return Str::slug($filename) . '.' . $extension;
+        return Str::slug($filename).'.'.$extension;
     }
 
     protected function sort(HasAsset $model, MediaField $field, Request $request)
@@ -170,8 +170,8 @@ 
         $values = isset($fileIdInput[$key])
             ? $fileIdInput[$key]
             : (
-                isset($fileIdInput['files-' . $key])
-                ? $fileIdInput['files-' . $key]
+                isset($fileIdInput['files-'.$key])
+                ? $fileIdInput['files-'.$key]
                 : ''
             );
 

src/DynamicAttributes/DynamicAttributes.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         $value = is_null($value) ? [] : (is_array($value) ? $value : json_decode($value, true));
 
-        return new static((array) $value);
+        return new static((array)$value);
     }
 
     public function has(string $key): bool

src/Urls/UrlRecord.php

Spacing +6 added lines, -6 removed lines

@@ -37,7 +37,7 @@ 
             ->first();
 
         if (!$record) {
-            throw new UrlRecordNotFound('No url record found by slug [' . $slug . '] for locale [' . $locale . '].');
+            throw new UrlRecordNotFound('No url record found by slug ['.$slug.'] for locale ['.$locale.'].');
         }
 
         return $record;
@@ -61,7 +61,7 @@ 
             ->first();
 
         if (!$record) {
-            throw new UrlRecordNotFound('No url record found for model [' . $model->getMorphClass() . '@' . $model->id . '] for locale [' . $locale . '].');
+            throw new UrlRecordNotFound('No url record found for model ['.$model->getMorphClass().'@'.$model->id.'] for locale ['.$locale.'].');
         }
 
         return $record;
@@ -155,7 +155,7 @@ 
         }
 
         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())
@@ -175,11 +175,11 @@ 
 
         // Filter out offline urls... TODO: this should be a state that is owned by the url and not the model.
         // Because of archived pages the mapping can be null so we reject them before we check for online
-        return $records->map(function (UrlRecord $urlRecord) {
+        return $records->map(function(UrlRecord $urlRecord) {
             return Morphables::instance($urlRecord->model_type)->find($urlRecord->model_id);
-        })->reject(function ($model) {
+        })->reject(function($model) {
             return $model == null;
-        })->reject(function (ProvidesUrl $model) {
+        })->reject(function(ProvidesUrl $model) {
             return (method_exists($model, 'isPublished') && !$model->isPublished());
         });
     }

src/System/Sitemap/SitemapFiles.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     {
         $files = $this->filesystem->files($directory);
 
-        return collect($files)->filter(function ($file) {
+        return collect($files)->filter(function($file) {
             return (Str::startsWith($file->getFileName(), 'sitemap-') && Str::endsWith($file->getFileName(), '.xml'));
         });
     }

src/System/Sitemap/SitemapXml.php

Spacing +4 added lines, -4 removed lines

@@ -76,11 +76,11 @@ 
         $models = UrlRecord::allOnlineModels($locale);
 
         $this->urls = $models
-            ->reject(function (ProvidesUrl $model) use ($locale) {
+            ->reject(function(ProvidesUrl $model) use ($locale) {
                 // In case the url is not found or present for given locale.
                 return !$model->url($locale);
             })
-            ->map(function (ProvidesUrl $model) use ($locale, $alternateLocales) {
+            ->map(function(ProvidesUrl $model) use ($locale, $alternateLocales) {
                 $url = $model->url($locale);
 
                 $alternateUrls = [];
@@ -101,12 +101,12 @@ 
     {
         $pool = new Pool($this->httpClient, $this->crawlableUrlGenerator(), [
             'concurrency' => 5,
-            'fulfilled' => function (Response $response, $index) {
+            'fulfilled' => function(Response $response, $index) {
                 if ($response->getStatusCode() !== \Symfony\Component\HttpFoundation\Response::HTTP_OK) {
                     unset($this->urls[$index]);
                 }
             },
-            'rejected' => function (RequestException $reason, $index) {
+            'rejected' => function(RequestException $reason, $index) {
                 unset($this->urls[$index]);
             },
         ]);

src/Management/Assistants/UrlAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 
     public function route($verb): ?string
     {
-        if($this->manager->hasExistingModel()) {
+        if ($this->manager->hasExistingModel()) {
             $routes = [
                 'check' => route('chief.back.assistants.url.check', [
                     $this->manager->details()->key,
@@ -78,9 +78,9 @@ 
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($this->manager->existingModel())->reject(function ($record) {
+        UrlRecord::getByModel($this->manager->existingModel())->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
     }
@@ -93,7 +93,7 @@ 
     private function validateModel()
     {
         if (!$this->manager->existingModel() instanceof ProvidesUrl) {
-            throw new \Exception('UrlAssistant requires the model interfaced by ' . ProvidesUrl::class . '.');
+            throw new \Exception('UrlAssistant requires the model interfaced by '.ProvidesUrl::class.'.');
         }
     }
 }

src/Management/AbstractManager.php

Spacing +4 added lines, -4 removed lines

@@ -88,7 +88,7 @@ 
             return $this->indexPagination($builder);
         }
 
-        return $builder->get()->map(function ($model) {
+        return $builder->get()->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -116,7 +116,7 @@ 
     {
         $paginator = $builder->paginate($this->pageCount);
 
-        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function ($model) {
+        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function($model) {
             return (new static($this->registration))->manage($model);
         });
 
@@ -232,7 +232,7 @@ 
 
     public function editFields(): Fields
     {
-        return $this->fieldsWithAssistantFields()->map(function(Field $field){
+        return $this->fieldsWithAssistantFields()->map(function(Field $field) {
             return $field->model($this->model);
         });
     }
@@ -325,7 +325,7 @@ 
             static::$bootedTraitMethods[$baseMethod] = [];
 
             foreach (class_uses_recursive($class) as $trait) {
-                $method = class_basename($trait) . ucfirst($baseMethod);
+                $method = class_basename($trait).ucfirst($baseMethod);
 
                 if (method_exists($class, $method) && !in_array($method, static::$bootedTraitMethods[$baseMethod])) {
                     static::$bootedTraitMethods[$baseMethod][] = lcfirst($method);

src/Settings/SettingFieldsManager.php

Spacing +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
 
     public function editFields(): Fields
     {
-        return $this->fields()->map(function(Field $field){
-            return $field->valueResolver(function($model = null, $locale = null, $field){
+        return $this->fields()->map(function(Field $field) {
+            return $field->valueResolver(function($model = null, $locale = null, $field) {
                 return $this->settings->get($field->getKey(), $locale);
             });
         });

src/Fields/Types/AbstractField.php

Spacing +3 added lines, -3 removed lines

@@ -237,7 +237,7 @@ 
      */
     private function defaultEloquentValueResolver(): \Closure
     {
-        return function (Model $model = null, $locale = null) {
+        return function(Model $model = null, $locale = null) {
             if (!$model) {
                 return $this->value;
             }
@@ -390,7 +390,7 @@ 
 
         return $this->isLocalized()
             ? 'chief::back._formgroups.fieldgroup_translatable'
-            : 'chief::back._fields.' . $this->type->get();
+            : 'chief::back._fields.'.$this->type->get();
     }
 
     public function viewData(array $viewData = []): Field
@@ -406,6 +406,6 @@ 
             'model' => $this->getModel(),
             'field' => $this,
             'key'   => $this->getKey(),
-        ],$this->viewData);
+        ], $this->viewData);
     }
 }