thinktomorrow / chief

src/PageBuilder/UpdateSections.php

Spacing +5 added lines, -5 removed lines

@@ -90,7 +90,7 @@ 
     private function removeExistingSets()
     {
         foreach ($this->model->children() as $instance) {
-            if (! $instance instanceof StoredSetReference) {
+            if (!$instance instanceof StoredSetReference) {
                 continue;
             }
             $this->model->rejectChild($instance);
@@ -144,7 +144,7 @@ 
         }
 
         foreach ($this->text_modules['replace'] as $text_module) {
-            if (! $module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
+            if (!$module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
                 continue;
             }
 
@@ -163,7 +163,7 @@ 
 
     private function removeTextualModule($module)
     {
-        if (! $module instanceof TextModule && ! $module instanceof PagetitleModule) {
+        if (!$module instanceof TextModule && !$module instanceof PagetitleModule) {
             throw new \Exception('Invalid request to remove non textual module');
         }
 
@@ -185,7 +185,7 @@ 
                 continue;
             }
 
-            $child = $children->first(function ($c) use ($reference) {
+            $child = $children->first(function($c) use ($reference) {
                 return $c->flatReference()->get() == $reference;
             });
 
@@ -236,7 +236,7 @@ 
     private function stripTagsBlacklist($value, $blacklist = [])
     {
         foreach ($blacklist as $tag) {
-            $value = preg_replace('/<\/?' . $tag . '(.|\s)*?>/', '', $value);
+            $value = preg_replace('/<\/?'.$tag.'(.|\s)*?>/', '', $value);
         }
 
         return $value;

src/Modules/Module.php

Spacing +3 added lines, -3 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.'.');
     }
 
     /**
@@ -76,9 +76,9 @@ 
      */
     public static function availableForCreation(): Collection
     {
-        $managers = app(Managers::class)->findByTag('module')->filter(function ($manager) {
+        $managers = app(Managers::class)->findByTag('module')->filter(function($manager) {
             return $manager->can('create');
-        })->map(function ($manager) {
+        })->map(function($manager) {
             return $manager->details();
         });
 

src/Management/Nomadic.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public function nomadicCan($verb)
     {
-        if (in_array($verb, ['create', 'store', 'delete']) && ! auth()->guard('chief')->user()->hasRole('developer')) {
+        if (in_array($verb, ['create', 'store', 'delete']) && !auth()->guard('chief')->user()->hasRole('developer')) {
             throw NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 

src/Management/Managers.php

Spacing +5 added lines, -5 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,14 +90,14 @@ 
     {
         $registrations = collect($this->register->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
 
     public function anyRegisteredByTag($tag)
     {
-        return ! empty($this->register->filterByTag($tag)->all());
+        return !empty($this->register->filterByTag($tag)->all());
     }
 
     /**

src/Pages/Page.php

Spacing +5 added lines, -5 removed lines

@@ -84,7 +84,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.'.');
     }
 
     /**
@@ -124,9 +124,9 @@ 
     public function flatReferenceLabel(): string
     {
         if ($this->exists) {
-            $status = ! $this->isPublished() ? ' [' . $this->statusAsPlainLabel().']' : null;
+            $status = !$this->isPublished() ? ' ['.$this->statusAsPlainLabel().']' : null;
 
-            return $this->title ? $this->title . $status : '';
+            return $this->title ? $this->title.$status : '';
         }
 
         return '';
@@ -136,8 +136,8 @@ 
     {
         $classKey = get_class($this);
         if (property_exists($this, 'labelSingular')) {
-            $labelSingular =  $this->labelSingular;
-        } else {
+            $labelSingular = $this->labelSingular;
+        }else {
             $labelSingular = Str::singular($classKey);
         }
 

src/Fields/Types/Field.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -71,7 +71,7 @@ 
 
     public function optional(): bool
     {
-        return ! $this->required();
+        return !$this->required();
     }
 
     public function name(string $name = null)
@@ -146,7 +146,7 @@ 
 
     private function defaultValueResolver(): callable
     {
-        return function (Model $model, $locale) {
+        return function(Model $model, $locale) {
             if ($this->isTranslatable() && $locale) {
                 return $model->getTranslationFor($this->column(), $locale);
             }
@@ -214,7 +214,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/Management/AbstractManager.php

Spacing +6 added lines, -6 removed lines

@@ -84,7 +84,7 @@ 
             return $this->indexPagination($builder);
         }
         
-        return $builder->get()->map(function ($model) {
+        return $builder->get()->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -112,7 +112,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);
         });
 
@@ -186,7 +186,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -209,7 +209,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }
 
@@ -291,9 +291,9 @@ 
             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])) {
+                if (method_exists($class, $method) && !in_array($method, static::$bootedTraitMethods[$baseMethod])) {
                     static::$bootedTraitMethods[$baseMethod][] = lcfirst($method);
                 }
             }

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/Relations/Relation.php

Spacing +8 added lines, -8 removed lines

@@ -46,7 +46,7 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) {
+        return $relations->map(function(Relation $relation) {
             $parent = $relation->parent;
             $parent->relation = $relation;
             return $parent;
@@ -60,14 +60,14 @@ 
                             ->orderBy('sort', 'ASC')
                             ->get();
 
-        return $relations->map(function (Relation $relation) use ($parent_type, $parent_id) {
+        return $relations->map(function(Relation $relation) use ($parent_type, $parent_id) {
 
             // It could be that the child itself is soft-deleted, if this is the case, we will ignore it and move on.
             if (!$child = $relation->child) {
                 if (!$relation->child()->withTrashed()->first()) {
 //                if ((!method_exists($childInstance, 'trashed')) || ! $childInstance->onlyTrashed()->find($relation->child_id)) {
                     // If we cannot retrieve it then he collection type is possibly off, this is a database inconsistency and should be addressed
-                    throw new \DomainException('Corrupt relation reference. Related child ['.$relation->child_type.'@'.$relation->child_id.'] could not be retrieved for parent [' . $parent_type.'@'.$parent_id.']. Make sure the morph key can resolve to a valid class.');
+                    throw new \DomainException('Corrupt relation reference. Related child ['.$relation->child_type.'@'.$relation->child_id.'] could not be retrieved for parent ['.$parent_type.'@'.$parent_id.']. Make sure the morph key can resolve to a valid class.');
                 }
 
                 return null;
@@ -79,7 +79,7 @@ 
         })
 
         // In case of soft-deleted entries, this will be null and should be ignored. We make sure that keys are reset in case of removed child
-        ->reject(function ($child) {
+        ->reject(function($child) {
             return is_null($child);
         })
         ->values();
@@ -96,10 +96,10 @@ 
 
     public static function deleteRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                          ->where('parent_id', $id);
-        })->orWhere(function ($query) use ($type, $id) {
+        })->orWhere(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();
@@ -111,7 +111,7 @@ 
 
     public static function deleteAllChildRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                 ->where('parent_id', $id);
         })->get();
@@ -123,7 +123,7 @@ 
 
     public static function deleteAllParentRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();