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

Spacing +3 added lines, -3 removed lines

@@ -22,11 +22,11 @@ 
             return false;
         }
 
-        if(in_array($key, $this->dynamicKeys())) {
+        if (in_array($key, $this->dynamicKeys())) {
             return true;
         }
 
-        if(in_array('*', $this->dynamicKeys())){
+        if (in_array('*', $this->dynamicKeys())) {
             return !in_array($key, $this->dynamicKeysBlacklist());
         }
 
@@ -113,7 +113,7 @@ 
             $value = $this->dynamic($key);
 
             // If value is localized, we wont return the entire value, but instead return null since no fallback will be provided.
-            if(is_array($value) && in_array($locale, $this->dynamicLocales())) return null;
+            if (is_array($value) && in_array($locale, $this->dynamicLocales())) return null;
 
             return $value;
         }

Braces +3 added lines, -1 removed lines

@@ -113,7 +113,9 @@
             $value = $this->dynamic($key);
 
             // If value is localized, we wont return the entire value, but instead return null since no fallback will be provided.
-            if(is_array($value) && in_array($locale, $this->dynamicLocales())) return null;
+            if(is_array($value) && in_array($locale, $this->dynamicLocales())) {
+                return null;
+            }
 
             return $value;
         }

src/DynamicAttributes/DynamicAttributes.php

Spacing +2 added lines, -2 removed lines

@@ -16,11 +16,11 @@
 
     public static function fromRawValue($value): self
     {
-        if($value instanceof self) return $value;
+        if ($value instanceof self) return $value;
 
         $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

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
 
     public static function fromRawValue($value): self
     {
-        if($value instanceof self) return $value;
+        if($value instanceof self) {
+            return $value;
+        }
 
         $value = is_null($value) ? [] : (is_array($value) ? $value : json_decode($value, true));
 

src/Fragments/ManagesFragments.php

Spacing +3 added lines, -3 removed lines

@@ -21,16 +21,16 @@
         }, $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));
 
         // Save each fragment
-        foreach($fragments as $i => $fragment) {
+        foreach ($fragments as $i => $fragment) {
             $modelId = $this->existingModel()->saveFragment($fragment, $i);
 
             // Attach any asset
-            if(isset($imagePayload[$i])) {
+            if (isset($imagePayload[$i])) {
                 $fieldKey = key($imagePayload[$i]);
                 $imageField = $this->fields()->keyed($fragmentField->getKey())->first()->getFields()->keyed($fieldKey)->first();
 

src/Fragments/HasFragments.php

Spacing +2 added lines, -2 removed lines

@@ -23,8 +23,8 @@
             'key' => $fragment->getKey(), 'order' => $order,
         ], $fragment->getValues());
 
-        if($fragment->hasModelId()){
-            $model =FragmentModel::find($fragment->getModelId());
+        if ($fragment->hasModelId()) {
+            $model = FragmentModel::find($fragment->getModelId());
             $model->update($values);
 
             return $model->id;

src/Fragments/FragmentField.php

Indentation +11 added lines, -11 removed lines

@@ -60,21 +60,21 @@
                 ->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){
+                                    ->localizedFormat(':name.:locale')
+                                    ->valueResolver(function($model = null, $locale = null, $field) use($fragment){
 
-                                     if(isset($field->value)) return $field->value;
+                                        if(isset($field->value)) return $field->value;
 
-                                     if($field instanceof MediaField){
-                                         if(!$fragment->hasModelId()){
-                                             return [];
-                                         }
+                                        if($field instanceof MediaField){
+                                            if(!$fragment->hasModelId()){
+                                                return [];
+                                            }
 
-                                         return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
-                                     }
+                                            return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
+                                        }
 
-                                     return $fragment->getValue($field->getColumn(), $locale);
-                                 });
+                                        return $fragment->getValue($field->getColumn(), $locale);
+                                    });
                 }));
         }
 

Spacing +15 added lines, -15 removed lines

@@ -42,31 +42,31 @@ 
         $fragments = [Fragment::empty($this->getKey())];
 
         // Model is auto-injected by Manager::editFields() method.
-        if(($this->model)) {
+        if (($this->model)) {
 
-            if(!method_exists($this->model, 'getFragments')) {
-                throw new \RuntimeException(get_class($this->model) . ' is missing the ' . HasFragments::class . ' trait.');
+            if (!method_exists($this->model, 'getFragments')) {
+                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) {
+            if (count($modelFragments = $this->model->getFragments($this->getKey())) > 0) {
+                $fragments = $modelFragments->map(function(FragmentModel $fragmentModel) {
                     return Fragment::fromModel($fragmentModel);
                 })->all();
             }
         }
 
-        foreach($fragments as $k => $fragment) {
+        foreach ($fragments as $k => $fragment) {
             $fragments[$k] = $fragments[$k]
-                ->setModelIdInputName($this->name.'[' . $k . '][modelId]')
+                ->setModelIdInputName($this->name.'['.$k.'][modelId]')
                 ->setFields($this->fields->clone()->map(function(Field $field) use($k, $fragment){
-                    return $field->name($this->name.'.' . $k . '.' . $field->getName())
+                    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 (isset($field->value)) return $field->value;
 
-                                     if($field instanceof MediaField){
-                                         if(!$fragment->hasModelId()){
+                                     if ($field instanceof MediaField) {
+                                         if (!$fragment->hasModelId()) {
                                              return [];
                                          }
 
@@ -83,12 +83,12 @@ 
 
     public function getDuplicatableFields(): array
     {
-        if(count($this->getFragments()) < 1) return [];
+        if (count($this->getFragments()) < 1) return [];
 
         // 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){
-                if($field instanceof \Thinktomorrow\Chief\Fields\Types\MediaField) { return []; }
+        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 []; }
                 return null;
             })->render();
         }, $this->getFragments()[0]->getFields()->clone()->all());

Braces +6 added lines, -2 removed lines

@@ -63,7 +63,9 @@ 
                                  ->localizedFormat(':name.:locale')
                                  ->valueResolver(function($model = null, $locale = null, $field) use($fragment){
 
-                                     if(isset($field->value)) return $field->value;
+                                     if(isset($field->value)) {
+                                         return $field->value;
+                                     }
 
                                      if($field instanceof MediaField){
                                          if(!$fragment->hasModelId()){
@@ -83,7 +85,9 @@ 
 
     public function getDuplicatableFields(): array
     {
-        if(count($this->getFragments()) < 1) return [];
+        if(count($this->getFragments()) < 1) {
+            return [];
+        }
 
         // Take the fields from the first fragment as a starting point for duplication
         return array_map(function(\Thinktomorrow\Chief\Fields\Types\Field $field){

src/Fragments/Fragment.php

Spacing +6 added lines, -6 removed lines

@@ -33,7 +33,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
@@ -48,14 +48,14 @@ 
 
     public static function fromRequestPayload(string $key, array $payload): self
     {
-        if(!isset($payload['modelId'])) {
+        if (!isset($payload['modelId'])) {
             return static::fromNew($key, $payload);
         }
 
         $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
@@ -72,13 +72,13 @@ 
 
     public function getValue(string $key, ?string $locale = null)
     {
-        if(!isset($this->values[$key])) {
+        if (!isset($this->values[$key])) {
             return null;
         }
 
         // When the value is an array, it is assumed that this is an array of locales, each representing the value for that locale
-        if(is_array($this->values[$key])) {
-            if($locale && array_key_exists($locale, $this->values[$key])) {
+        if (is_array($this->values[$key])) {
+            if ($locale && array_key_exists($locale, $this->values[$key])) {
                 return $this->values[$key][$locale];
             }
         }