thinktomorrow / chief

src/Legacy/Fragments/HasFragments.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         ], $fragment->getValues());
 
         if ($fragment->hasModelId()) {
-            $model =FragmentModel::find($fragment->getModelId());
+            $model = FragmentModel::find($fragment->getModelId());
             $model->update($values);
 
             return $model->id;

src/Modules/Modules.php

Spacing +5 added lines, -5 removed lines

@@ -33,12 +33,12 @@ 
         // TODO: how to assign for pagebuilder?? with tag, via the chief relations config or is this default
         $moduleManagerClasses = $this->registry->tagged('module')->get();
 trap('working on it');
-        $managers = collect(array_map(function($moduleManagerClass){
+        $managers = collect(array_map(function($moduleManagerClass) {
             return app($moduleManagerClass);
         }, $moduleManagerClasses));
 
         return $managers
-            ->reject(function($manager){ return !$manager->can('create'); })
+            ->reject(function($manager) { return !$manager->can('create'); })
             ->map(function($manager) {
                 return [
                     'label' => $manager->adminLabel('page_title'),
@@ -53,13 +53,13 @@ 
         // Get the modules that are set to be shareable.
         $moduleManagerClasses = $this->shareableModules();
 
-        $managers = collect(array_map(function($moduleManagerClass){
+        $managers = collect(array_map(function($moduleManagerClass) {
             return app($moduleManagerClass);
         }, $moduleManagerClasses));
 
         return $managers
-            ->reject(function($manager){ return !$manager->can('create-shared'); })
-            ->map(function($manager){
+            ->reject(function($manager) { return !$manager->can('create-shared'); })
+            ->map(function($manager) {
                 return [
                     'label' => $manager->adminLabel('page_title'),
                     'value' => $manager->route('create-shared')

src/Fragments/Actions/CreateFragmentModel.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
      */
     public function create(FragmentsOwner $owner, Fragmentable $fragmentable, int $order, array $data = []): FragmentModel
     {
-        if(!$context = ContextModel::ownedBy($owner)) {
+        if (!$context = ContextModel::ownedBy($owner)) {
             $context = ContextModel::createForOwner($owner);
         }
 

src/Fragments/Actions/RenderFragments.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     public function render(Collection $fragmentables, FragmentsOwner $owner, $viewData = []): string
     {
         // Validate each entry as a valid fragment object.
-        $fragmentables->each(function(Fragmentable $fragmentable){});
+        $fragmentables->each(function(Fragmentable $fragmentable) {});
 
         // Init new loop object
         $this->loopsStack = []; $this->addLoop($fragmentables);
@@ -24,7 +24,7 @@ 
             $this->incrementLoopIndices();
             $loop = $this->getLastLoop();
 
-            return $carry . $fragmentable->renderFragment($owner, $loop, $fragmentables, $viewData);
+            return $carry.$fragmentable->renderFragment($owner, $loop, $fragmentables, $viewData);
         }, '');
     }
 }

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.');
         }
     }