thinktomorrow / chief

src/Fields/Fields.php

Spacing +3 added lines, -3 removed lines

@@ -123,8 +123,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;
@@ -153,7 +153,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

src/Modules/ModuleManager.php

Spacing +5 added lines, -5 removed lines

@@ -63,15 +63,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);
         }
     }
@@ -103,7 +103,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();
         }
 
@@ -142,7 +142,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/Fields/SavingFields.php

Spacing +4 added lines, -4 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(Str::camel($field->key())) . 'Field';
+            $methodName = 'set'.ucfirst(Str::camel($field->key())).'Field';
             (method_exists($this, $methodName))
                 ? $this->$methodName($field, $request)
                 : $this->setField($field, $request);
@@ -53,7 +53,7 @@ 
 
     protected function detectCustomSaveMethods(Field $field): bool
     {
-        $saveMethodName = 'save'. ucfirst(Str::camel($field->key())) . 'Field';
+        $saveMethodName = 'save'.ucfirst(Str::camel($field->key())).'Field';
 
         // Custom save method on assistant
         foreach ($this->assistants() as $assistant) {
@@ -95,7 +95,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/Settings/SettingsServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
     public function register()
     {
-        $this->app->singleton(Settings::class, function ($app) {
+        $this->app->singleton(Settings::class, function($app) {
             return new Settings();
         });
     }

src/Modules/Module.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
             return static::$managedModelKey;
         }
 
-        throw new \Exception('Missing required static property \'managedModelKey\' on ' . static::class. '.');
+        throw new \Exception('Missing required static property \'managedModelKey\' on '.static::class.'.');
     }
 
     /**

src/Common/Helpers/Memoize.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         foreach ($parameters as $key => $value) {
             if ($value instanceof Model) {
-                $parameters[$key] = get_class($value) . '@' . $value->id;
+                $parameters[$key] = get_class($value).'@'.$value->id;
             }
         }
 

src/Urls/Application/SaveUrlSlugs.php

Spacing +8 added lines, -8 removed lines

@@ -56,7 +56,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()
@@ -65,7 +65,7 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
 
@@ -81,7 +81,7 @@ 
         }
 
         // 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]);
             }
@@ -126,12 +126,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();
             }
@@ -146,11 +146,11 @@ 
 
         // The old homepage url should be removed since this is no longer in effect.
         // In case of any redirect to this old homepage, the last used redirect is now back in effect.
-        $existingRecords->reject(function ($existingRecord) {
+        $existingRecords->reject(function($existingRecord) {
             return (
                 $existingRecord->model_type == $this->model->getMorphClass() &&
                 $existingRecord->model_id == $this->model->id);
-        })->each(function ($existingRecord) {
+        })->each(function($existingRecord) {
 
             // TODO: if there is a redirect to this page, we'll take this one as the new url
             $existingRecord->delete();
@@ -164,7 +164,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/UrlSlugField.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
             'baseUrlSegment' => $this->baseUrlSegment,
             'hint' => null, // Hint placeholder to show url hint when it already exists
             'is_homepage' => ($this->value() === '/'),
-            'show' => !!$this->value(),// show input field or not
+            'show' => !!$this->value(), // show input field or not
         ]);
     }
 }

src/Management/Assistants/PublishAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
 
     public function findAll(): Collection
     {
-        return $this->model->published()->get()->map(function ($model) {
+        return $this->model->published()->get()->map(function($model) {
             return $this->managers->findByModel($model);
         });
     }
@@ -93,7 +93,7 @@ 
 
     public function guard($verb): Assistant
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this->manager);
         }
 
@@ -117,10 +117,10 @@ 
             $class = 'text-warning';
         }
 
-        $statusAsLabel = '<span class="font-bold '. $class .'"><em>' . $label . '</em></span>';
+        $statusAsLabel = '<span class="font-bold '.$class.'"><em>'.$label.'</em></span>';
 
         if (!$plain && $this->hasPreviewUrl()) {
-            $statusAsLabel =  '<a href="'.$this->previewUrl().'" target="_blank">'. $statusAsLabel .'</a>';
+            $statusAsLabel = '<a href="'.$this->previewUrl().'" target="_blank">'.$statusAsLabel.'</a>';
         }
 
         return $statusAsLabel;