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

Spacing +4 added lines, -4 removed lines

@@ -73,11 +73,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);
         });
     }
@@ -145,7 +145,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -168,7 +168,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 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/Urls/UrlSlugFields.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
 
     public static function redirectsFromModel(ProvidesUrl $model)
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return !$record->isRedirect();
         })->sortByDesc('created_at');
 
@@ -29,7 +29,7 @@ 
             $fields[$key] = UrlSlugField::make($key)
                 ->setUrlRecord($record)
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) .'/');
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
         }
 
         return $fields;
@@ -61,10 +61,10 @@ 
         $fields = new static([]);
 
         foreach ($locales as $locale) {
-            $fields['url-slugs.' . $locale] = UrlSlugField::make('url-slugs.' . $locale)
+            $fields['url-slugs.'.$locale] = UrlSlugField::make('url-slugs.'.$locale)
                                                 ->setBaseUrlSegment($model->baseUrlSegment($locale))
-                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)) .'/')
-                                                ->name('url-slugs[' . $locale . ']')
+                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)).'/')
+                                                ->name('url-slugs['.$locale.']')
                                                 ->label($locale);
         }
 
@@ -77,7 +77,7 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = UrlRecord::getByModel($model)->reject(function ($record) {
+        $records = UrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
@@ -89,7 +89,7 @@ 
             $fields['url-slugs.'.$record->locale]
                 ->setUrlRecord($record)
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) .'/');
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
         }
     }
 }

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.