thinktomorrow / chief

src/Managers/Register/Registry.php

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
 
     public function modelClass(string $key): string
     {
-        if(!isset($this->models[$key])) {
+        if (!isset($this->models[$key])) {
             throw new MissingModelRegistration('No model class registered for key ['.$key.']');
         }
 
@@ -31,7 +31,7 @@ 
 
     public function manager(string $key): Manager
     {
-        if(!isset($this->managers[$key])) {
+        if (!isset($this->managers[$key])) {
             throw new MissingManagerRegistration('No manager registered for key ['.$key.']');
         }
 
@@ -50,9 +50,9 @@ 
 
     public function managersWithTags(): array
     {
-        return collect($this->tags->get())->reject(function($tags, $key){
+        return collect($this->tags->get())->reject(function($tags, $key) {
             return !isset($this->managers[$key]);
-        })->map(function($tags, $key){
+        })->map(function($tags, $key) {
             return (object)[
                 'manager' => $this->manager($key),
                 'tags' => $tags,

src/Managers/Register/TaggedKeys.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 
     public function tagged($tags): self
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         return $this->filter(function($value, $key) use($tags){
             return count(array_intersect($value, $tags)) > 0;
@@ -30,7 +30,7 @@ 
 
     public function notTagged($tags): self
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         return $this->filter(function($value, $key) use($tags){
             return count(array_intersect($value, $tags)) === 0;
@@ -39,7 +39,7 @@ 
 
     public function untagged(): self
     {
-        return $this->filter(function($value){
+        return $this->filter(function($value) {
             return empty($value);
         });
     }

src/Managers/Register/Register.php

Spacing +3 added lines, -3 removed lines

@@ -45,8 +45,8 @@ 
     {
         // Check if model class points to ManagedModel interface
         $ref = new \ReflectionClass($modelClass);
-        if(!$ref->implementsInterface(ManagedModel::class)) {
-            throw new \DomainException('Class ' . $modelClass . ' should implement contract ' . ManagedModel::class);
+        if (!$ref->implementsInterface(ManagedModel::class)) {
+            throw new \DomainException('Class '.$modelClass.' should implement contract '.ManagedModel::class);
         }
 
         $managedModelKey = $modelClass::managedModelKey();
@@ -55,7 +55,7 @@ 
         $this->container->make(Registry::class)
             ->registerModel($managedModelKey, $modelClass)
             ->registerManager($managedModelKey, $manager)
-            ->registerTags($managedModelKey, (array) $tags);
+            ->registerTags($managedModelKey, (array)$tags);
 
         // Register routes
         $this->container->make(RegisterManagedRoutes::class)(

src/Managers/Exceptions/NotAllowedManagerAction.php

Spacing +2 added lines, -2 removed lines

@@ -40,11 +40,11 @@
 
     public static function notAllowedAction($action, string $managerKey)
     {
-        return new self('Not allowed to ' . $action . ' a model. ' . ucfirst($action) . ' route is not allowed by the ' . $managerKey . ' manager.');
+        return new self('Not allowed to '.$action.' a model. '.ucfirst($action).' route is not allowed by the '.$managerKey.' manager.');
     }
 
     public static function notAllowedPermission($permission, string $managerKey)
     {
-        return new self('Not allowed permission for ' . $permission . ' on a model as managed by the ' . $managerKey . ' manager.');
+        return new self('Not allowed permission for '.$permission.' on a model as managed by the '.$managerKey.' manager.');
     }
 }

src/Managers/Assistants/FragmentsAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@ 
 
     public function routeFragmentsAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragments-index','fragments-reorder'])) return null;
+        if (!in_array($action, ['fragments-index', 'fragments-reorder'])) return null;
 
         return $this->generateRoute($action, $model, ...$parameters);
     }
 
     public function canFragmentsAssistant(string $action, $model = null): bool
     {
-        return in_array($action, ['fragments-index','fragments-reorder']);
+        return in_array($action, ['fragments-index', 'fragments-reorder']);
     }
 
     public function fragmentsIndex(Request $request, $id)
@@ -43,7 +43,7 @@ 
         $model = $modelClass::findOrFail($id);
 
         // Current fragments
-        $fragments = app(FragmentRepository::class)->getByOwner($model)->map(function(Fragmentable $model){
+        $fragments = app(FragmentRepository::class)->getByOwner($model)->map(function(Fragmentable $model) {
             return [
                 'model'    => $model,
                 'manager'  => app(Registry::class)->manager($model::managedModelKey()),
@@ -51,7 +51,7 @@ 
         });
 
         // Available fragments
-        $allowedFragments = array_map(function($fragmentableClass){
+        $allowedFragments = array_map(function($fragmentableClass) {
             $modelClass = app(Registry::class)->modelClass($fragmentableClass::managedModelKey());
             return [
                 'manager' => app(Registry::class)->manager($fragmentableClass::managedModelKey()),

Braces +3 added lines, -1 removed lines

@@ -25,7 +25,9 @@
 
     public function routeFragmentsAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragments-index','fragments-reorder'])) return null;
+        if(!in_array($action, ['fragments-index','fragments-reorder'])) {
+            return null;
+        }
 
         return $this->generateRoute($action, $model, ...$parameters);
     }

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/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 [];
                 }