thinktomorrow / chief

src/ManagedModels/Application/DeleteManagedModel.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
             if ($model instanceof Module) {
                 $model->update([
-                    'slug' => $model->slug . $this->deletionAppendix(),
+                    'slug' => $model->slug.$this->deletionAppendix(),
                 ]);
             }
 
@@ -70,6 +70,6 @@ 
 
     private function deletionAppendix(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

src/ManagedModels/Assistants/SavingFields.php

Spacing +14 added lines, -14 removed lines

@@ -27,12 +27,12 @@ 
      */
     public function saveFields(Fields $fields, array $input, array $files): void
     {
-        foreach($fields as $field){
-            if($this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($this->detectCustomSaveMethod($field)) {
                 continue;
             }
 
-            if(!$field->isLocalized())
+            if (!$field->isLocalized())
             {
                 // Set standard non-localized attribute on the model
                 ($customSetMethod = $this->detectCustomSetMethod($field))
@@ -45,11 +45,11 @@ 
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             $this->foreachTranslationEntry(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if ($key !== $field->getColumn()) return;
 
-                if($this->isFieldForDynamicValue($field)) {
+                if ($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);
-                } else {
+                }else {
                     $this->{$field->getColumn().':'.$locale} = $value;
                 }
             });
@@ -58,8 +58,8 @@ 
         $this->save();
 
         // Custom save methods
-        foreach($fields as $field){
-            if($customSaveMethod = $this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($customSaveMethod = $this->detectCustomSaveMethod($field)) {
                 $this->$customSaveMethod($field, $input, $files);
             }
         }
@@ -72,8 +72,8 @@ 
 
     private function detectCustomSaveMethod(Field $field): ?string
     {
-        $saveMethodByKey = 'save' . ucfirst(Str::camel($field->getKey())) . 'Field';
-        $saveMethodByType = 'save' . ucfirst(Str::camel($field->getType()->get())) . 'Fields';
+        $saveMethodByKey = 'save'.ucfirst(Str::camel($field->getKey())).'Field';
+        $saveMethodByType = 'save'.ucfirst(Str::camel($field->getType()->get())).'Fields';
 
         foreach ([$saveMethodByKey, $saveMethodByType] as $saveMethod) {
             if (method_exists($this, $saveMethod)) {
@@ -86,7 +86,7 @@ 
 
     private function detectCustomSetMethod(Field $field): ?string
     {
-        $methodName = 'set' . ucfirst(Str::camel($field->getKey())) . 'Field';
+        $methodName = 'set'.ucfirst(Str::camel($field->getKey())).'Field';
 
         return (method_exists($this, $methodName)) ? $methodName : null;
     }
@@ -108,13 +108,13 @@ 
 
     private function saveImageFields(ImageField $field, array $input)
     {
-        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.' . $field->getName(), []), $input);
+        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.'.$field->getName(), []), $input);
     }
 
     private function foreachTranslationEntry(array $translations, \Closure $callback)
     {
-        foreach($translations as $locale => $trans){
-            foreach($trans as $key => $value){
+        foreach ($translations as $locale => $trans) {
+            foreach ($trans as $key => $value) {
                 call_user_func($callback, $locale, $key, $value);
             }
         }

Braces +3 added lines, -1 removed lines

@@ -45,7 +45,9 @@
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             $this->foreachTranslationEntry(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if($key !== $field->getColumn()) {
+                    return;
+                }
 
                 if($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);

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]];
             }
         });