thinktomorrow / chief

src/Fields/SavingFields.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
             }
 
             // Custom set methods - default is the generic setField() method.
-            $methodName = 'set' . ucfirst(Str::camel($field->getKey())) . 'Field';
+            $methodName = 'set'.ucfirst(Str::camel($field->getKey())).'Field';
             (method_exists($this, $methodName))
                 ? $this->$methodName($field, $request)
                 : $this->setField($field, $request);
@@ -58,8 +58,8 @@ 
 
     protected function detectCustomSaveMethods(Field $field): bool
     {
-        $saveMethodByKey = 'save' . ucfirst(Str::camel($field->getKey())) . 'Field';
-        $saveMethodByType = 'save' . ucfirst(Str::camel($field->getType()->get())) . 'Fields';
+        $saveMethodByKey = 'save'.ucfirst(Str::camel($field->getKey())).'Field';
+        $saveMethodByType = 'save'.ucfirst(Str::camel($field->getType()->get())).'Fields';
 
         foreach ([$saveMethodByKey, $saveMethodByType] as $saveMethod) {
             foreach ($this->assistants() as $assistant) {
@@ -107,7 +107,7 @@ 
 
             // Make our media fields able to be translatable as well...
             if ($field->ofType(FieldType::FILE, FieldType::IMAGE)) {
-                throw new \Exception('Cannot process the ' . $field->getKey() . ' media field. Currently no support for translatable media files. We should fix this!');
+                throw new \Exception('Cannot process the '.$field->getKey().' media field. Currently no support for translatable media files. We should fix this!');
             }
 
             // Okay so this is a bit odd but since all translations are expected to be inside the trans
@@ -160,6 +160,6 @@ 
 
     public function saveImageFields(ImageField $field, Request $request)
     {
-        app(ImageFieldHandler::class)->handle($this->model, $field, $request->input('images.' . $field->getName(), []), $request);
+        app(ImageFieldHandler::class)->handle($this->model, $field, $request->input('images.'.$field->getName(), []), $request);
     }
 }

src/Pages/PageManager.php

Spacing +2 added lines, -2 removed lines

@@ -88,7 +88,7 @@ 
             $this->pageBuilderField(),
             InputField::make('title')->translatable($this->model->availableLocales())
                 ->validation('required-fallback-locale|max:200', [], [
-                    'trans.' . config('app.fallback_locale', 'nl') . '.title' => 'title',
+                    'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
                 ])
                 ->label('Intern label')
                 ->description('Dit is de benaming die zal worden getoond in de admin selectie velden.')
@@ -210,7 +210,7 @@ 
             if (is_array_empty($translation)) {
 
                 // Nullify all values
-                $trans[$locale] = array_map(function ($value) {
+                $trans[$locale] = array_map(function($value) {
                     return null;
                 }, $translation);
                 continue;

src/Relations/AvailableChildren.php

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
 
     public function onlyModules(): Collection
     {
-        return $this->collection()->reject(function ($item) {
+        return $this->collection()->reject(function($item) {
             if ($item instanceof Page || $item instanceof StoredSetReference) {
                 return true;
             }
@@ -45,7 +45,7 @@ 
 
     public function onlyPages(): Collection
     {
-        return $this->collection()->filter(function ($item) {
+        return $this->collection()->filter(function($item) {
             if ($item instanceof Page) {
                 return true;
             }
@@ -75,12 +75,12 @@ 
 
         // Merging the results of all the pages and all the modules, then filter by the config
         // This prevents us from having duplicates and also reduces the query load.
-        $collection = $collection->filter(function ($page) use ($available_children_types) {
+        $collection = $collection->filter(function($page) use ($available_children_types) {
             return in_array(get_class($page), $available_children_types);
         });
 
         // Filter out our already loaded pages and modules
-        $remaining_children_types = collect($available_children_types)->reject(function ($type) {
+        $remaining_children_types = collect($available_children_types)->reject(function($type) {
             return (new $type() instanceof Page || new $type() instanceof Module);
         });
 
@@ -92,12 +92,12 @@ 
         }
 
         // Remove duplicate models
-        $collection = $collection->unique(function ($model) {
+        $collection = $collection->unique(function($model) {
             return (new FlatReference(get_class($model), $model->id))->get();
         });
 
         // Filter out our parent
-        return $this->collection = $collection->reject(function ($item) {
+        return $this->collection = $collection->reject(function($item) {
             if ($item instanceof $this->parent) {
                 return $item->id == $this->parent->id;
             }

src/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/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\Fields\Types\Field $field) {
-            return $field->valueResolver(function ($model = null, $locale = null, $field) {
+        return array_map(function(\Thinktomorrow\Chief\Fields\Types\Field $field) {
+            return $field->valueResolver(function($model = null, $locale = null, $field) {
                 if ($field instanceof \Thinktomorrow\Chief\Fields\Types\MediaField) {
                     return [];
                 }

src/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/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/DynamicAttributes/DynamicAttributes.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
         $value = is_null($value) ? [] : (is_array($value) ? $value : json_decode($value, true));
 
-        return new static((array) $value);
+        return new static((array)$value);
     }
 
     public function has(string $key): bool

src/Fields/Types/ImageField.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
     {
         return (new static(new FieldType(FieldType::IMAGE), $key))
             ->locales([config('app.fallback_locale', 'nl')])
-            ->valueResolver(function ($model = null, $locale = null, ImageField $field) {
+            ->valueResolver(function($model = null, $locale = null, ImageField $field) {
                 return $field->getMedia($model, $locale);
             });
     }
@@ -39,7 +39,7 @@ 
         $images = [];
         $locale = $locale ?? app()->getLocale();
 
-        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function ($asset) use ($locale) {
+        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function($asset) use ($locale) {
             return $asset->pivot->locale == $locale;
         })->sortBy('pivot.order');