thinktomorrow / chief

src/Fragments/Database/FragmentRepository.php

Spacing +6 added lines, -6 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function getByOwner(FragmentsOwner $owner): Collection
     {
-        if(!$context = ContextModel::ownedBy($owner)) return collect();
+        if (!$context = ContextModel::ownedBy($owner)) return collect();
 
         $fragmentModels = $context->fragments()->get();
 
@@ -44,16 +44,16 @@ 
 
     private function prefetchRecords(Collection $fragmentModels)
     {
-        $fragmentModels->mapToGroups(function(FragmentModel $fragmentModel){
+        $fragmentModels->mapToGroups(function(FragmentModel $fragmentModel) {
             return [ModelReference::fromString($fragmentModel->model_reference)->className() => ModelReference::fromString($fragmentModel->model_reference)->id()];
-        })->reject(function($modelIds, $className){
+        })->reject(function($modelIds, $className) {
             $reflection = new ReflectionClass($className);
             return !$reflection->isSubclassOf(Model::class);
-        })->each(function($modelIds, $className){
+        })->each(function($modelIds, $className) {
             $modelIds = $modelIds->filter(fn($modelId) => $modelId !== 0);
             $records = $className::withoutGlobalScopes()->whereIn('id', $modelIds->toArray())->get();
 
-            $records->each(function($record){
+            $records->each(function($record) {
                 $this->prefetchedRecords[$record->modelReference()->get()] = $record;
             });
         });
@@ -62,7 +62,7 @@ 
     private function fragmentFactory(FragmentModel $fragmentModel): Fragmentable
     {
         return $this->prefetchedRecords
-            ->get($fragmentModel->model_reference,  fn() => ModelReference::fromString($fragmentModel->model_reference)->instance())
+            ->get($fragmentModel->model_reference, fn() => ModelReference::fromString($fragmentModel->model_reference)->instance())
             ->setFragmentModel($fragmentModel);
     }
 

Braces +3 added lines, -1 removed lines

@@ -26,7 +26,9 @@
      */
     public function getByOwner(FragmentsOwner $owner): Collection
     {
-        if(!$context = ContextModel::ownedBy($owner)) return collect();
+        if(!$context = ContextModel::ownedBy($owner)) {
+            return collect();
+        }
 
         $fragmentModels = $context->fragments()->get();
 

src/Fragments/Database/ContextModel.php

Indentation +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
     public static function ownedBy(FragmentsOwner $owner): ?ContextModel
     {
         return static::where('owner_type', $owner->modelReference()->className())
-                     ->where('owner_id', $owner->modelReference()->id())
-                     ->first();
+                        ->where('owner_id', $owner->modelReference()->id())
+                        ->first();
     }
 
     public static function createForOwner(FragmentsOwner $owner): ContextModel

src/Fragments/Assistants/FragmentableDefaults.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function renderFragment($owner, $loop, $fragments, $viewData): string
     {
         // Default (legacy) view rendering
-        if(public_method_exists($this, 'renderView')) {
+        if (public_method_exists($this, 'renderView')) {
             return $this->renderView();
         }
 
@@ -38,7 +38,7 @@ 
 
     public function fragmentModel(): FragmentModel
     {
-        if(!isset($this->fragmentModel)) {
+        if (!isset($this->fragmentModel)) {
             throw new \DomainException('FragmentModel property on ['.get_class($this).'] expected to be set, but it\'s not.');
         }
 

src/Site/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
                     $targetModel = Morphables::instance($targetUrlRecord->model_type)->find($targetUrlRecord->model_id);
 
                     if (!$targetModel) {
-                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $targetUrlRecord->model_type . '@' . $targetUrlRecord->model_id . '] has probably been archived or deleted.');
+                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$targetUrlRecord->model_type.'@'.$targetUrlRecord->model_id.'] has probably been archived or deleted.');
                     }
 
                     return static::createRedirect($targetModel->url($locale));
@@ -45,14 +45,14 @@ 
             }
 
             if (!$model) {
-                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $urlRecord->model_type . '@' . $urlRecord->model_id . '] has probably been archived or deleted.');
+                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$urlRecord->model_type.'@'.$urlRecord->model_id.'] has probably been archived or deleted.');
             }
 
             if (public_method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
                 if (!PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request [' . $slug . '] but it is not published.');
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -63,7 +63,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request [' . $slug . '] for locale [' . $locale . '].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)

src/Old/Sets/SetReference.php

Spacing +5 added lines, -5 removed lines

@@ -38,7 +38,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(
@@ -53,14 +53,14 @@ 
     {
         $sets = config('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();
     }
@@ -152,11 +152,11 @@ 
     private static function validateAction($class, $method)
     {
         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.');
+            throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
         }
 
         if (!method_exists($class, $method)) {
-            throw new \InvalidArgumentException('The method [' . $method . '] does not exist on the class [' . $class . ']. Make sure you provide a valid method to the action value in the chief-settings.sets config entry.');
+            throw new \InvalidArgumentException('The method ['.$method.'] does not exist on the class ['.$class.']. Make sure you provide a valid method to the action value in the chief-settings.sets config entry.');
         }
     }
 

src/Old/PageBuilder/PageBuilder.php

Spacing +10 added lines, -10 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Old\PageBuilder;
 
@@ -42,7 +42,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;
         });
 
@@ -51,7 +51,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) {
 //
 //            }
@@ -88,14 +88,14 @@ 
         $trans = [];
 
         // Has dynamic attributes
-        if(public_method_exists($model, 'dynamic')) {
+        if (public_method_exists($model, 'dynamic')) {
             foreach (config('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;
                             }
                         }
@@ -105,9 +105,9 @@ 
         }
 
         // Astrotomic translatable
-        if($model instanceof Translatable) {
+        if ($model instanceof Translatable) {
             foreach (config('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());
             }

Braces +3 added lines, -1 removed lines

@@ -107,7 +107,9 @@
         // Astrotomic translatable
         if($model instanceof Translatable) {
             foreach (config('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/Site/Menu/Menu.php

Spacing +4 added lines, -4 removed lines

@@ -28,14 +28,14 @@ 
     {
         $types = config('chief.menus', []);
 
-        return collect($types)->map(function ($menu, $key) {
+        return collect($types)->map(function($menu, $key) {
             return new static($key, $menu['label'], $menu['view']);
         });
     }
 
     public static function find($key): ?self
     {
-        return static::all()->filter(function ($menu) use ($key) {
+        return static::all()->filter(function($menu) use ($key) {
             return $menu->key() == $key;
         })->first();
     }
@@ -84,10 +84,10 @@ 
     {
         $menu = [];
 
-        $this->items()->each(function ($item) use (&$menu) {
+        $this->items()->each(function($item) use (&$menu) {
             $menu[] = sprintf('<li><a href="%s">%s</a></li>', $item->url, $item->label);
         });
 
-        return '<ul>' . implode('', $menu) . '</ul>';
+        return '<ul>'.implode('', $menu).'</ul>';
     }
 }

src/Site/Urls/Controllers/LinksController.php

Spacing +4 added lines, -4 removed lines

@@ -20,19 +20,19 @@
     public function update(Request $request)
     {
         /** @var ProvidesUrl|Model $model */
-        $model = (new ModelReference($request->modelClass, (int) $request->modelId))->instance();
+        $model = (new ModelReference($request->modelClass, (int)$request->modelId))->instance();
 
         $this->validate($request, ['links' => [
-            'array', 'min:1', new UniqueUrlSlugRule($model, $model),], [], ['links.*' => 'taalspecifieke link']
+            'array', 'min:1', new UniqueUrlSlugRule($model, $model), ], [], ['links.*' => 'taalspecifieke link']
         ]);
 
         (new SaveUrlSlugs())->handle($model, $request->input('links', []));
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($model)->reject(function ($record) {
+        UrlRecord::getByModel($model)->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
 

src/Site/Urls/Application/SaveUrlSlugs.php

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
     private function saveRecord(ProvidesUrl $model, string $locale, ?string $slug, Collection $existingRecords)
     {
         // Existing ones for this locale?
-        $nonRedirectsWithSameLocale = $existingRecords->filter(function ($record) use ($locale) {
+        $nonRedirectsWithSameLocale = $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 !$record->isRedirect()
@@ -47,7 +47,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();
             });
 
@@ -64,7 +64,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]);
             }
@@ -105,12 +105,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();
             }
@@ -125,11 +125,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 == $model->getMorphClass() &&
                 $existingRecord->model_id == $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();