thinktomorrow / chief

src/Managers/Assistants/ManagerDefaults.php

Spacing +5 added lines, -5 removed lines

@@ -28,8 +28,8 @@ 
 
     public function route(string $action, $model = null, ...$parameters): string
     {
-        foreach(DiscoverTraitMethods::belongingTo(static::class, 'route') as $method) {
-            if(null !== ($route = $this->$method($action, $model, ...$parameters))){
+        foreach (DiscoverTraitMethods::belongingTo(static::class, 'route') as $method) {
+            if (null !== ($route = $this->$method($action, $model, ...$parameters))) {
                 return $route;
             }
         }
@@ -39,8 +39,8 @@ 
 
     public function can(string $action, $model = null): bool
     {
-        foreach(DiscoverTraitMethods::belongingTo(static::class, 'can') as $method) {
-            if(true === $this->$method($action, $model)){
+        foreach (DiscoverTraitMethods::belongingTo(static::class, 'can') as $method) {
+            if (true === $this->$method($action, $model)) {
                 return true;
             }
         }
@@ -56,7 +56,7 @@ 
             $parameters = array_merge((array)$modelId, $parameters);
         }
 
-        return route('chief.' . $this->managedModelClass()::managedModelKey() . '.' . $action, $parameters);
+        return route('chief.'.$this->managedModelClass()::managedModelKey().'.'.$action, $parameters);
     }
 
     private function guard(string $action)

src/Managers/Assistants/FragmentAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -32,18 +32,18 @@ 
 
     public function routeFragmentAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store'])) {
+        if (!in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store'])) {
             return null;
         };
 
         $modelKey = $this->managedModelClass()::managedModelKey();
 
-        if(in_array($action, ['fragment-create', 'fragment-store'])) {
+        if (in_array($action, ['fragment-create', 'fragment-store'])) {
             if (!$model || !$model instanceof FragmentsOwner) {
                 throw new \Exception('Fragment route definition for '.$action.' requires the owning Model as second argument.');
             }
 
-            return route('chief.' . $modelKey . '.' . $action, array_merge([
+            return route('chief.'.$modelKey.'.'.$action, array_merge([
                 $model::managedModelKey(),
                 $model->modelReference()->id(),
             ], $parameters));
@@ -53,12 +53,12 @@ 
             throw new \Exception('Fragment route definition for '.$action.' requires the fragment model as second argument.');
         }
 
-        return route('chief.' . $modelKey . '.' . $action, $model->fragmentModel()->id);
+        return route('chief.'.$modelKey.'.'.$action, $model->fragmentModel()->id);
     }
 
     public function canFragmentAssistant(string $action, $model = null): bool
     {
-        return in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store']);
+        return in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store']);
     }
 
     public function fragmentCreate(Request $request, string $ownerKey, $ownerId)
@@ -145,13 +145,13 @@ 
         $model = $this->managedModelClass()::findOrFail($id);
 
         if ($request->get('deleteconfirmation') !== 'DELETE') {
-            return redirect()->back()->with('messages.warning', $model->adminLabel('title') . ' is niet verwijderd.');
+            return redirect()->back()->with('messages.warning', $model->adminLabel('title').' is niet verwijderd.');
         }
 
         app(DeleteManagedModel::class)->handle($model);
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is verwijderd.');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is verwijderd.');
     }
 
     private function owner(string $ownerKey, $ownerId): FragmentsOwner

src/Managers/Assistants/CrudAssistant.php

Spacing +6 added lines, -6 removed lines

@@ -100,14 +100,14 @@ 
         // Default sorted by publish date
         $modelClass = $this->managedModelClass();
         if (Schema::hasColumn((new $modelClass)->getTable(), 'published_at')) {
-            $filters = $filters->add(HiddenFilter::make('publish', function ($query) {
+            $filters = $filters->add(HiddenFilter::make('publish', function($query) {
                 return $query->orderBy('published_at', 'DESC');
             }));
         }
 
         // if model has no timestamps, updated_at doesn't exist
         if ((new $modelClass)->timestamps) {
-            $filters = $filters->add(HiddenFilter::make('updated', function ($query) {
+            $filters = $filters->add(HiddenFilter::make('updated', function($query) {
                 return $query->orderBy('updated_at', 'DESC');
             }));
         }
@@ -141,7 +141,7 @@ 
         $model->saveFields($model->fields()->tagged('create'), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('edit', $model))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is toegevoegd');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is toegevoegd');
     }
 
     public function edit(Request $request, $id)
@@ -172,7 +172,7 @@ 
         $model->saveFields($model->fields(), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="' . $this->route('edit', $model) . '">' . $model->adminLabel('title') . '</a> is aangepast');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="'.$this->route('edit', $model).'">'.$model->adminLabel('title').'</a> is aangepast');
     }
 
     public function delete(Request $request, $id)
@@ -182,12 +182,12 @@ 
         $model = $this->managedModelClass()::findOrFail($id);
 
         if ($request->get('deleteconfirmation') !== 'DELETE') {
-            return redirect()->back()->with('messages.warning', $model->adminLabel('title') . ' is niet verwijderd.');
+            return redirect()->back()->with('messages.warning', $model->adminLabel('title').' is niet verwijderd.');
         }
 
         app(DeleteManagedModel::class)->handle($model);
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is verwijderd.');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is verwijderd.');
     }
 }

src/Legacy/Fragments/Fragment.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 
     public static function fromModel(FragmentModel $fragmentModel): self
     {
-        return new static($fragmentModel->key, $fragmentModel->values->all(), new Fields(), (int) $fragmentModel->id);
+        return new static($fragmentModel->key, $fragmentModel->values->all(), new Fields(), (int)$fragmentModel->id);
     }
 
     public static function fromNew(string $key, array $values): self
@@ -56,7 +56,7 @@ 
         $modelId = $payload['modelId'];
         unset($payload['modelId']);
 
-        return new static($key, $payload, new Fields(), (int) $modelId);
+        return new static($key, $payload, new Fields(), (int)$modelId);
     }
 
     public function setFields(Fields $fields): self

src/Legacy/Fragments/ManagesFragments.php

Spacing +3 added lines, -3 removed lines

@@ -13,15 +13,15 @@
     public function saveFragmentFields(FragmentField $fragmentField, Request $request)
     {
         $payload = $request->input($fragmentField->getDottedName(), []);
-        $imagePayload = $request->input('images.' . $fragmentField->getDottedName(), []);
+        $imagePayload = $request->input('images.'.$fragmentField->getDottedName(), []);
 
         // Compose Fragment instances for each payload entry
-        $fragments = array_map(function ($fragmentPayload) use ($fragmentField) {
+        $fragments = array_map(function($fragmentPayload) use ($fragmentField) {
             return Fragment::fromRequestPayload($fragmentField->getKey(), $fragmentPayload); // (new Fragment($field->getKey(), $fragmentPayload));
         }, $payload);
 
         // remove all dead fragments
-        $this->existingModel()->removeAllFragments($fragmentField->getKey(), array_map(function (Fragment $fragment) {
+        $this->existingModel()->removeAllFragments($fragmentField->getKey(), array_map(function(Fragment $fragment) {
             return $fragment->hasModelId() ? $fragment->getModelId() : null;
         }, $fragments));
 

src/Legacy/Fragments/FragmentField.php

Indentation +16 added lines, -16 removed lines

@@ -63,22 +63,22 @@
                 ->setModelIdInputName($this->name . '[' . $k . '][modelId]')
                 ->setFields($this->fields->clone()->map(function (Field $field) use ($k, $fragment) {
                     return $field->name($this->name . '.' . $k . '.' . $field->getName())
-                                 ->localizedFormat(':name.:locale')
-                                 ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
-                                     if (isset($field->value)) {
-                                         return $field->value;
-                                     }
-
-                                     if ($field instanceof MediaField) {
-                                         if (!$fragment->hasModelId()) {
-                                             return [];
-                                         }
-
-                                         return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
-                                     }
-
-                                     return $fragment->getValue($field->getColumn(), $locale);
-                                 });
+                                    ->localizedFormat(':name.:locale')
+                                    ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
+                                        if (isset($field->value)) {
+                                            return $field->value;
+                                        }
+
+                                        if ($field instanceof MediaField) {
+                                            if (!$fragment->hasModelId()) {
+                                                return [];
+                                            }
+
+                                            return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
+                                        }
+
+                                        return $fragment->getValue($field->getColumn(), $locale);
+                                    });
                 }));
         }
 

Spacing +8 added lines, -8 removed lines

@@ -48,11 +48,11 @@ 
         // Model is auto-injected by Manager::editFields() method.
         if (($this->model)) {
             if (!method_exists($this->model, 'getFragments')) {
-                throw new \RuntimeException(get_class($this->model) . ' is missing the ' . HasFragments::class . ' trait.');
+                throw new \RuntimeException(get_class($this->model).' is missing the '.HasFragments::class.' trait.');
             }
 
             if (count($modelFragments = $this->model->getFragments($this->getKey())) > 0) {
-                $fragments = $modelFragments->map(function (FragmentModel $fragmentModel) {
+                $fragments = $modelFragments->map(function(FragmentModel $fragmentModel) {
                     return Fragment::fromModel($fragmentModel);
                 })->all();
             }
@@ -60,11 +60,11 @@ 
 
         foreach ($fragments as $k => $fragment) {
             $fragments[$k] = $fragments[$k]
-                ->setModelIdInputName($this->name . '[' . $k . '][modelId]')
-                ->setFields($this->fields->clone()->map(function (Field $field) use ($k, $fragment) {
-                    return $field->name($this->name . '.' . $k . '.' . $field->getName())
+                ->setModelIdInputName($this->name.'['.$k.'][modelId]')
+                ->setFields($this->fields->clone()->map(function(Field $field) use ($k, $fragment) {
+                    return $field->name($this->name.'.'.$k.'.'.$field->getName())
                                  ->localizedFormat(':name.:locale')
-                                 ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
+                                 ->valueResolver(function($model = null, $locale = null, $field) use ($fragment) {
                                      if (isset($field->value)) {
                                          return $field->value;
                                      }
@@ -110,8 +110,8 @@ 
         }
 
         // Take the fields from the first fragment as a starting point for duplication
-        return array_map(function (\Thinktomorrow\Chief\ManagedModels\Fields\Types\Field $field) {
-            return $field->valueResolver(function ($model = null, $locale = null, $field) {
+        return array_map(function(\Thinktomorrow\Chief\ManagedModels\Fields\Types\Field $field) {
+            return $field->valueResolver(function($model = null, $locale = null, $field) {
                 if ($field instanceof \Thinktomorrow\Chief\ManagedModels\Fields\Types\MediaField) {
                     return [];
                 }

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