thinktomorrow / chief

src/Media/UploadMedia.php

Spacing +11 added lines, -11 removed lines

@@ -113,7 +113,7 @@ 
         if (isset(json_decode($file)->output)) {
             $image_name = json_decode($file)->output->name;
             $asset      = app(AddAsset::class)->add($model, json_decode($file)->output->image, $type, $locale, $this->sluggifyFilename($image_name));
-        } else {
+        }else {
             if ($file instanceof UploadedFile) {
                 $image_name = $file->getClientOriginalName();
                 $asset      = app(AddAsset::class)->add($model, $file, $type, $locale, $this->sluggifyFilename($image_name));
@@ -121,9 +121,9 @@ 
                 // New files are passed with their filename (instead of their id)
                 // For new files we will replace the filename with the id.
                 if (false !== ($key = array_search($image_name, $files_order))) {
-                    $files_order[$key] = (string) $asset->id;
+                    $files_order[$key] = (string)$asset->id;
                 }
-            } else {
+            }else {
                 $file       = Asset::findOrFail($file);
                 $asset      = app(AddAsset::class)->add($model, $file, $type, $locale);
             }
@@ -136,9 +136,9 @@ 
      */
     private function sluggifyFilename($filename): string
     {
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename  = substr($filename, 0, strrpos($filename, '.'));
-        $filename  = Str::slug($filename) . '.' . $extension;
+        $filename  = Str::slug($filename).'.'.$extension;
 
         return $filename;
     }
@@ -154,9 +154,9 @@ 
                 if ($file instanceof UploadedFile && !$file->isValid()) {
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
                         throw new FileTooBigException(
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
                         );
                     }
                 }
@@ -170,16 +170,16 @@ 
         foreach ($files_by_type as $type => $files) {
             foreach ($files as $locale => $_files) {
                 if (!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
 
                 if (!is_array($_files)) {
-                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a ' . gettype($_files) . ' is given.');
+                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a '.gettype($_files).' is given.');
                 }
 
                 foreach ($_files as $action => $file) {
                     if (!in_array($action, $actions)) {
-                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead ' . $action . ' is given.');
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
                     }
                 }
             }

src/Sets/Set.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
 
         // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module.
-        return $this->map(function ($item) {
+        return $this->map(function($item) {
             return ($this->viewParent && $item instanceof ViewableContract)
                 ? $item->setViewParent($this->viewParent)->renderView()
                 : ($item->content ?? '');
@@ -84,7 +84,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage);
 
         return (new \Illuminate\Pagination\Paginator($items, $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }
@@ -101,7 +101,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = '/'.request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage);
 
         return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->paginateSetting('total', $this->count()), $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }

src/Sets/SetReference.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     {
         // Constraints
         if (!isset($values['action'])) {
-            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: ' . print_r($values, true));
+            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: '.print_r($values, true));
         }
 
         return new static(
@@ -54,14 +54,14 @@ 
     {
         $sets = config('thinktomorrow.chief.sets', []);
 
-        return collect($sets)->map(function ($set, $key) {
+        return collect($sets)->map(function($set, $key) {
             return SetReference::fromArray($key, $set);
         });
     }
 
     public static function find($key): ?SetReference
     {
-        return static::all()->filter(function ($ref) use ($key) {
+        return static::all()->filter(function($ref) use ($key) {
             return $ref->key() == $key;
         })->first();
     }
@@ -78,13 +78,13 @@ 
 
         $this->validateAction($class, $method);
 
-        $result = call_user_func_array([app($class),$method], $this->parameters($class, $method, $parent));
+        $result = call_user_func_array([app($class), $method], $this->parameters($class, $method, $parent));
 
-        if (! $result instanceof Set && $result instanceof Collection) {
+        if (!$result instanceof Set && $result instanceof Collection) {
             return new Set($result->all(), $this->key);
         }
 
-        if (! $result instanceof Set && $result instanceof Paginator) {
+        if (!$result instanceof Set && $result instanceof Paginator) {
             return new Set($result->all(), $this->key, [
                 'paginate' => [
                     'total' => $result->total(),
@@ -152,7 +152,7 @@ 
 
     private static function validateAction($class, $method)
     {
-        if (! class_exists($class)) {
+        if (!class_exists($class)) {
             throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
         }
 

src/Urls/UrlHelper.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
      */
     public static function onlineModels(bool $onlySingles = false, Model $ignoredModel = null): Collection
     {
-        $models = chiefMemoize('all-online-models', function () use ($onlySingles) {
+        $models = chiefMemoize('all-online-models', function() use ($onlySingles) {
             $builder = UrlRecord::whereNull('redirect_id')
                 ->select('model_type', 'model_id')
                 ->groupBy('model_type', 'model_id');
@@ -48,17 +48,17 @@ 
                 $builder->where('model_type', 'singles');
             }
 
-            return $builder->get()->mapToGroups(function ($record) {
+            return $builder->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
-            })->map(function ($record, $key) {
+            })->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->map->reject(function ($model) {
+            })->map->reject(function($model) {
                 return is_null($model) || !$model->isPublished(); // Invalid references to archived or removed models where url record still exists.
             })->flatten();
         }, [$onlySingles]);
 
         if ($ignoredModel) {
-            $models = $models->reject(function ($model) use ($ignoredModel) {
+            $models = $models->reject(function($model) use ($ignoredModel) {
                 return (get_class($model) === get_class($ignoredModel) && $model->id === $ignoredModel->id);
             });
         }

src/Fields/Types/MediaField.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -40,7 +40,7 @@ 
             return $this;
         }
 
-        return 'files['. ($this->values['name'] ?? $this->key()).']';
+        return 'files['.($this->values['name'] ?? $this->key()).']';
     }
 
     public function sluggifyName()
@@ -58,7 +58,7 @@ 
         $images = [];
         $locale = $locale ?? app()->getLocale();
 
-        $assets = $model->assetRelation->where('pivot.type', $this->key())->filter(function ($asset) use ($locale) {
+        $assets = $model->assetRelation->where('pivot.type', $this->key())->filter(function($asset) use ($locale) {
             return $asset->pivot->locale == $locale;
         })->sortBy('pivot.order');
 

src/Management/Application/DeleteManagedModel.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
             DB::beginTransaction();
 
             // For stateful transitions we will apply this deletion as a state
-            if($model instanceof StatefulContract) {
+            if ($model instanceof StatefulContract) {
                 (new PageState($model))->apply('delete');
                 $model->save();
             }
@@ -29,17 +29,17 @@ 
             Relation::deleteRelationsOf($model->getMorphClass(), $model->id);
 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
-            if($model instanceof Module) {
+            if ($model instanceof Module) {
                 $model->update([
-                    'slug' => $model->slug . $this->appendDeleteMarker(),
+                    'slug' => $model->slug.$this->appendDeleteMarker(),
                 ]);
             }
 
-            if($model instanceof ProvidesUrl){
+            if ($model instanceof ProvidesUrl) {
                 UrlRecord::getByModel($model)->each->delete();
             }
 
-            if($model instanceof Page){
+            if ($model instanceof Page) {
                 Module::where('page_id', $model->id)->delete();
             }
 
@@ -58,6 +58,6 @@ 
 
     private function appendDeleteMarker(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

src/States/Archivable/ArchiveScope.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\States\Archivable;
 
@@ -13,6 +13,6 @@ 
 {
     public function apply(Builder $builder, Model $model)
     {
-        $builder->where('current_state','<>', PageState::ARCHIVED);
+        $builder->where('current_state', '<>', PageState::ARCHIVED);
     }
 }

src/Management/Managers.php

Spacing +6 added lines, -6 removed lines

@@ -50,7 +50,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
@@ -65,7 +65,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration)->details();
         });
     }
@@ -76,7 +76,7 @@ 
         //return array with group name and values
         $grouped = [];
 
-        $managers = $managers->map(function (Manager $item) {
+        $managers = $managers->map(function(Manager $item) {
             return [
                 'id'    => $item->details()->id,
                 'group' => $item->details()->plural,
@@ -90,7 +90,7 @@ 
     {
         $registrations = collect($this->register->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
@@ -99,7 +99,7 @@ 
     {
         $assistants = collect();
 
-        foreach($this->all() as $manager) {
+        foreach ($this->all() as $manager) {
             $assistants = $assistants->merge($manager->assistants(false));
         }
 
@@ -108,7 +108,7 @@ 
 
     public function anyRegisteredByTag($tag)
     {
-        return ! empty($this->register->filterByTag($tag)->all());
+        return !empty($this->register->filterByTag($tag)->all());
     }
 
     /**

src/States/Featurable/TestFeaturedAssistant.php

Indentation +3 added lines, -3 removed lines

@@ -13,10 +13,10 @@
 
     public static function register($app)
     {
-         Route::get('testje', [static::class, 'archive']);
-         Route::get('testje', [AssistantController, 'archive']);
+            Route::get('testje', [static::class, 'archive']);
+            Route::get('testje', [AssistantController, 'archive']);
 
-         // fetched manager
+            // fetched manager
         // propagate naar assistant::archive
     }