thinktomorrow / chief

src/ManagedModels/ModelToManagerMap.php

Spacing +3 added lines, -3 removed lines

@@ -21,8 +21,8 @@ 
 
     public function find(string $modelClass): Manager
     {
-        if(!isset($this->map[$modelClass])) {
-            throw new \InvalidArgumentException('No manager mapped to model ' . $modelClass .'. Make sure to register the manager.');
+        if (!isset($this->map[$modelClass])) {
+            throw new \InvalidArgumentException('No manager mapped to model '.$modelClass.'. Make sure to register the manager.');
         }
 
         return app($this->map[$modelClass]);
@@ -30,7 +30,7 @@ 
 
     public function all(): array
     {
-        return array_map(function($managerClass){
+        return array_map(function($managerClass) {
             return app($managerClass);
         }, $this->map);
     }

src/Filters/Presets/RadioFilter.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
     protected function viewData(): array
     {
-        return array_merge( parent::viewData(),[
+        return array_merge(parent::viewData(), [
             'options' => $this->options,
         ]);
     }

src/PageBuilder/PageBuilder.php

Braces +3 added lines, -1 removed lines

@@ -107,7 +107,9 @@
         // Astrotomic translatable
         if($model instanceof Translatable) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
-                if(!isset($trans[$locale])) $trans[$locale] = [];
+                if(!isset($trans[$locale])) {
+                    $trans[$locale] = [];
+                }
 
                 $trans[$locale] = array_merge($trans[$locale], $model->getTranslation($locale)->toArray());
             }

Spacing +10 added lines, -10 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\PageBuilder;
 
@@ -41,7 +41,7 @@ 
         $availableChildren = AvailableChildren::forParent($model);
 
         // Only include modules of this model or shared ones.
-        $modules = $availableChildren->onlyModules()->reject(function ($module) use ($model) {
+        $modules = $availableChildren->onlyModules()->reject(function($module) use ($model) {
             return $module->owner_id != null && $module->owner_id != $model->id;
         });
 
@@ -50,7 +50,7 @@ 
         $available_sets = ModelReferencePresenter::toGroupedSelectValues($availableChildren->onlySets())->toArray();
 
         // Current sections
-        $sections = $model->children()->map(function ($section, $index) use($model) {
+        $sections = $model->children()->map(function($section, $index) use($model) {
 //            if ($section instanceof TranslatableContract) {
 //
 //            }
@@ -87,14 +87,14 @@ 
         $trans = [];
 
         // Has dynamic attributes
-        if(public_method_exists($model, 'dynamic')) {
+        if (public_method_exists($model, 'dynamic')) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
                 $trans[$locale] = [];
 
-                foreach($model->rawDynamicValues() as $key => $values) {
-                    if(is_array($values)) {
-                        foreach($values as $dynamicLocale => $dynamicValue) {
-                            if($dynamicLocale === $locale) {
+                foreach ($model->rawDynamicValues() as $key => $values) {
+                    if (is_array($values)) {
+                        foreach ($values as $dynamicLocale => $dynamicValue) {
+                            if ($dynamicLocale === $locale) {
                                 $trans[$locale][$key] = $dynamicValue;
                             }
                         }
@@ -104,9 +104,9 @@ 
         }
 
         // Astrotomic translatable
-        if($model instanceof Translatable) {
+        if ($model instanceof Translatable) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
-                if(!isset($trans[$locale])) $trans[$locale] = [];
+                if (!isset($trans[$locale])) $trans[$locale] = [];
 
                 $trans[$locale] = array_merge($trans[$locale], $model->getTranslation($locale)->toArray());
             }

src/PageBuilder/Relations/Relation.php

Spacing +8 added lines, -8 removed lines

@@ -54,7 +54,7 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) {
+        return $relations->map(function(Relation $relation) {
             $parent = $relation->parent;
             $parent->relation = $relation;
 
@@ -70,13 +70,13 @@ 
             ->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 we cannot retrieve it then the relation type is possibly wrong, 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;
@@ -88,7 +88,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();
@@ -105,10 +105,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();
@@ -120,7 +120,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();
@@ -132,7 +132,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();

src/Mediagallery/Http/MediagalleryController.php

Spacing +4 added lines, -4 removed lines

@@ -21,13 +21,13 @@ 
         $assets = Asset::with('media')->orderBy('created_at', 'DESC')->select('assets.*');
 
         if ($search) {
-            $assets->whereHas('media', function (Builder $query) use ($search) {
-                $query->where('file_name', 'LIKE', '%' . $search . '%');
+            $assets->whereHas('media', function(Builder $query) use ($search) {
+                $query->where('file_name', 'LIKE', '%'.$search.'%');
             });
         }
 
         if ($unused) {
-            $assets->whereNotExists(function ($query) {
+            $assets->whereNotExists(function($query) {
                 $query->select(DB::raw(1))
                     ->from('asset_pivots')
                     ->whereRaw('asset_pivots.asset_id = assets.id')
@@ -35,7 +35,7 @@ 
             });
         }
 
-        if($owner){
+        if ($owner) {
             $owner = ModelReference::fromString($owner)->instance();
             $modelAssets = collect();
             $modelAssets = $modelAssets->merge($owner->assets());

src/Settings/Application/ChangeHomepage.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     {
         foreach ($modelReferences as $locale => $modelReferenceString) {
             if (!$modelReferenceString) {
-                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale ' . $locale . ' is missing.');
+                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale '.$locale.' is missing.');
             }
         }
     }

src/ModelReferences/ModelReferencePresenter.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
 {
     public static function toSelectValues(Collection $collection): Collection
     {
-        return $collection->map(function (ProvidesModelReference $item) {
+        return $collection->map(function(ProvidesModelReference $item) {
             return [
                 'id'    => $item->modelReference()->get(),
                 'label' => $item->modelReferenceLabel(),
@@ -23,10 +23,10 @@ 
     {
         $grouped = [];
 
-        static::toSelectValues($collection)->each(function ($item) use (&$grouped) {
+        static::toSelectValues($collection)->each(function($item) use (&$grouped) {
             if (isset($grouped[$item['group']])) {
                 $grouped[$item['group']]['values'][] = $item;
-            } else {
+            }else {
                 $grouped[$item['group']] = ['group' => $item['group'], 'values' => [$item]];
             }
         });

src/ModelReferences/ModelReferenceCollection.php

Spacing +3 added lines, -3 removed lines

@@ -21,16 +21,16 @@
             $referenceStrings = [];
         }
 
-        return (new Collection($referenceStrings))->reject(function ($referenceString) {
+        return (new Collection($referenceStrings))->reject(function($referenceString) {
             return is_null($referenceString);
-        })->map(function ($referenceString) {
+        })->map(function($referenceString) {
             return ModelReference::fromString($referenceString)->instance();
         });
     }
 
     public function toModelReferences(): Collection
     {
-        return (new Collection($this->all()))->map(function (ProvidesModelReference $item) {
+        return (new Collection($this->all()))->map(function(ProvidesModelReference $item) {
             return $item->modelReference()->get();
         });
     }

src/ModelReferences/ModelReference.php

Spacing +3 added lines, -3 removed lines

@@ -21,12 +21,12 @@ 
     public static function fromString(string $reference): self
     {
         if (false == strpos($reference, '@')) {
-            throw new \InvalidArgumentException('Invalid reference composition. A model reference should honour schema <class>@<id>. [' . $reference . '] was passed instead.');
+            throw new \InvalidArgumentException('Invalid reference composition. A model reference should honour schema <class>@<id>. ['.$reference.'] was passed instead.');
         }
 
         list($className, $id) = explode('@', $reference);
 
-        return new static($className, (int) $id);
+        return new static($className, (int)$id);
     }
 
     /**
@@ -51,7 +51,7 @@ 
 
     public function get(): string
     {
-        return $this->className . '@' . $this->id;
+        return $this->className.'@'.$this->id;
     }
 
     public function equals($other): bool