thinktomorrow / chief

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/Form/LinkForm.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     {
         return new static($model, \Thinktomorrow\Chief\Site\Urls\UrlRecord::getByModel($model)
             ->groupBy('locale')
-            ->map(function ($records) {
+            ->map(function($records) {
                 return $records->sortBy('redirect_id')->sortByDesc('created_at');
             }));
     }
@@ -54,7 +54,7 @@ 
             $currentRecord = $this->urlRecords->get($locale, collect())->reject->isRedirect()->first();
 
             $values[$locale] = (object)[
-                'host'         => $this->model->resolveUrl($locale, $this->model->baseUrlSegment($locale)) . '/',
+                'host'         => $this->model->resolveUrl($locale, $this->model->baseUrlSegment($locale)).'/',
                 'fixedSegment' => $this->model->baseUrlSegment($locale),
                 'value'        => $currentRecord
                     ? $this->rawSlugValue($currentRecord->slug, $this->model->baseUrlSegment($locale))

src/Shared/Concerns/Viewable/Viewable.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         }
 
         if (config('chief.strict')) {
-            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to ' . get_class($this));
+            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to '.get_class($this));
         }
 
         return '';