thinktomorrow / chief

src/Fields/ValidationRules/ValidatesExistingAssetAttributes.php

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
     protected function refersToExistingAsset($value): bool
     {
         // Faster check to see if there is an asset id passed or not.
-        if(is_string($value) && $this->isValidJson($value)) return false;
+        if (is_string($value) && $this->isValidJson($value)) return false;
 
         return !is_null($this->existingAsset($value));
     }
@@ -35,7 +35,7 @@ 
     {
         $filepath = $asset->media->first()->getPath();
 
-        if (! $sizeDetails = @getimagesize($filepath)) {
+        if (!$sizeDetails = @getimagesize($filepath)) {
             return false;
         }
 

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
     protected function refersToExistingAsset($value): bool
     {
         // Faster check to see if there is an asset id passed or not.
-        if(is_string($value) && $this->isValidJson($value)) return false;
+        if(is_string($value) && $this->isValidJson($value)) {
+            return false;
+        }
 
         return !is_null($this->existingAsset($value));
     }

src/Fields/ValidationRules/FileFieldMaxRule.php

Spacing +8 added lines, -8 removed lines

@@ -11,19 +11,19 @@ 
     {
         $value = $this->normalizePayload($value);
 
-        foreach([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
-            foreach($value[$type] as $file) {
-                if($file && false !== $this->validateMax($attribute, $file, $params)) {
+        foreach ([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
+            foreach ($value[$type] as $file) {
+                if ($file && false !== $this->validateMax($attribute, $file, $params)) {
                     return true;
                 }
             }
         }
 
         $validator->setCustomMessages([
-            'filefield_max' => 'De :attribute is te groot en dient kleiner te zijn dan ' . implode(',',$params) .'Kb.',
+            'filefield_max' => 'De :attribute is te groot en dient kleiner te zijn dan '.implode(',', $params).'Kb.',
         ]);
 
-        if(!isset($validator->customAttributes[$attribute])) {
+        if (!isset($validator->customAttributes[$attribute])) {
             $validator->addCustomAttributes([
                 $attribute => 'afbeelding',
             ]);
@@ -35,7 +35,7 @@ 
 
     public function validateMax($attribute, $value, $parameters)
     {
-        if($this->refersToExistingAsset($value)) {
+        if ($this->refersToExistingAsset($value)) {
             return $this->validateAssetMax($this->existingAsset($value), $parameters);
         }
 
@@ -51,8 +51,8 @@ 
      */
     protected function getSize($attribute, $value)
     {
-        if(!$value instanceof File) {
-            throw new \InvalidArgumentException('Value is expected to be of type ' . File::class);
+        if (!$value instanceof File) {
+            throw new \InvalidArgumentException('Value is expected to be of type '.File::class);
         }
 
         return $value->getSize() / 1024;

src/Fields/ValidationRules/FileFieldRequiredRule.php

Spacing +3 added lines, -3 removed lines

@@ -10,8 +10,8 @@ 
     {
         $value = $this->normalizePayload($value);
 
-        foreach([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
-            if(is_array($value[$type]) && !empty($value[$type])) {
+        foreach ([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
+            if (is_array($value[$type]) && !empty($value[$type])) {
                 return true;
             }
         }
@@ -20,7 +20,7 @@ 
             'filefield_required' => 'De :attribute is verplicht.',
         ]);
 
-        if(!isset($validator->customAttributes[$attribute])) {
+        if (!isset($validator->customAttributes[$attribute])) {
             $validator->addCustomAttributes([
                 $attribute => 'afbeelding',
             ]);

src/Fields/Types/FileField.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,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');
 

src/Fields/Types/FieldType.php

Spacing +14 added lines, -14 removed lines

@@ -4,21 +4,21 @@ 
 
 class FieldType
 {
-    const INPUT = 'input';   // oneliner text (input)
-    const TEXT = 'text';    // Plain text (textarea)
+    const INPUT = 'input'; // oneliner text (input)
+    const TEXT = 'text'; // Plain text (textarea)
     const NUMBER = 'number'; // number
     const RANGE = 'range'; // range slider
-    const DATE = 'date';    // Timestamp input
-    const PHONENUMBER = 'phonenumber';    // Timestamp input
-    const HTML = 'html';    // Html text (wysiwyg)
-    const SELECT = 'select';  // Select options
+    const DATE = 'date'; // Timestamp input
+    const PHONENUMBER = 'phonenumber'; // Timestamp input
+    const HTML = 'html'; // Html text (wysiwyg)
+    const SELECT = 'select'; // Select options
     //const MEDIA = 'media';  // media file
-    const FILE = 'file';  // regular file
-    const IMAGE = 'image';  // image (slim uploader)
-    const DOCUMENT = 'document';  // documents
-    const RADIO = 'radio';  // radio select
-    const CHECKBOX = 'checkbox';  // checkbox select
-    const PAGEBUILDER = 'pagebuilder';  // the most special field there is...
+    const FILE = 'file'; // regular file
+    const IMAGE = 'image'; // image (slim uploader)
+    const DOCUMENT = 'document'; // documents
+    const RADIO = 'radio'; // radio select
+    const CHECKBOX = 'checkbox'; // checkbox select
+    const PAGEBUILDER = 'pagebuilder'; // the most special field there is...
 
     /** @var string */
     private $type;
@@ -48,7 +48,7 @@ 
     public function equalsAny(array $types): bool
     {
         foreach ($types as $type) {
-            if ($this->equals( FieldType::fromString($type) )) {
+            if ($this->equals(FieldType::fromString($type))) {
                 return true;
             }
         }
@@ -58,7 +58,7 @@ 
 
     public function equals($type): bool
     {
-        return ((string) $type === (string) $this->type && static::class === get_class($type));
+        return ((string)$type === (string)$this->type && static::class === get_class($type));
     }
 
     public function __toString()

src/Fields/Types/AbstractField.php

Spacing +8 added lines, -8 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -104,7 +104,7 @@ 
 
     protected function getValidationNameFormat(): string
     {
-        if($this->isLocalized()) {
+        if ($this->isLocalized()) {
             return $this->getLocalizedNameFormat();
         }
 
@@ -113,7 +113,7 @@ 
 
     protected function getLocalizedNameFormat(): string
     {
-        if(false !== strpos($this->name, ':locale')){
+        if (false !== strpos($this->name, ':locale')) {
             return $this->name;
         }
 
@@ -237,9 +237,9 @@ 
      */
     private function defaultEloquentValueResolver(): \Closure
     {
-        return function (Model $model = null, $locale = null) {
+        return function(Model $model = null, $locale = null) {
 
-            if(!$model) return $this->value;
+            if (!$model) return $this->value;
 
             if ($locale && $this->isLocalized()) {
                 return $model->getTranslationFor($this->getColumn(), $locale);
@@ -272,7 +272,7 @@ 
             return false;
         }
 
-        if($this->getValidationParameters() instanceof ValidationParameters) {
+        if ($this->getValidationParameters() instanceof ValidationParameters) {
             foreach ($this->getValidationParameters()->getRules() as $rule) {
                 if (false !== strpos($rule, 'required')) {
                     return true;
@@ -285,7 +285,7 @@ 
 
     public function optional(): bool
     {
-        return ! $this->required();
+        return !$this->required();
     }
 
     public function getValidationNames(array $payload = []): array
@@ -391,7 +391,7 @@ 
 
     public function getElementView(): string
     {
-        if($this->elementView) return $this->elementView;
+        if ($this->elementView) return $this->elementView;
 
         return $this->isLocalized()
             ? 'chief::back._fields.translatable'

src/Fields/Types/ImageField.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,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');
 

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,10 +58,10 @@ 
 
     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 ([$saveMethodByKey, $saveMethodByType] as $saveMethod) {
 
             foreach ($this->assistants() as $assistant) {
                 if (method_exists($assistant, $saveMethod)) {
@@ -102,7 +102,7 @@ 
 
             // Make our media fields able to be translatable as well...
             if ($field->ofType(FieldType::FILE, FieldType::IMAGE, FieldType::DOCUMENT)) {
-                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

src/Media/Application/AbstractMediaFieldHandler.php

Spacing +3 added lines, -3 removed lines

@@ -31,11 +31,11 @@
      */
     protected function sluggifyFilename(string $filename): string
     {
-        if(false === strpos($filename, '.')) return $filename;
+        if (false === strpos($filename, '.')) return $filename;
 
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename = substr($filename, 0, strrpos($filename, '.'));
 
-        return Str::slug($filename) . '.' . $extension;
+        return Str::slug($filename).'.'.$extension;
     }
 }

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
      */
     protected function sluggifyFilename(string $filename): string
     {
-        if(false === strpos($filename, '.')) return $filename;
+        if(false === strpos($filename, '.')) {
+            return $filename;
+        }
 
         $extension = substr($filename, strrpos($filename, '.') + 1);
         $filename = substr($filename, 0, strrpos($filename, '.'));