thinktomorrow / chief

src/Media/UploadMedia.php

Spacing +8 added lines, -8 removed lines

@@ -112,7 +112,7 @@ 
         if (is_string($file) && isset(json_decode($file)->output)) {
             $image_name = json_decode($file)->output->name;
             $asset = app(AddAsset::class)->add($model, json_decode($file)->output->image, $type, $locale, $this->sluggifyFilename($image_name));
-        } else {
+        }else {
             if ($file instanceof UploadedFile) {
                 $image_name = $file->getClientOriginalName();
 
@@ -123,7 +123,7 @@ 
                 if (false !== ($key = array_search($image_name, $files_order))) {
                     $files_order[$key] = (string)$asset->id;
                 }
-            } else {
+            }else {
                 $file = Asset::find($file);
                 if ($file) {
                     if ($model->assetRelation()->where('asset_pivots.type', $type)->where('asset_pivots.locale', $locale)->get()->contains($file)) {
@@ -142,9 +142,9 @@ 
      */
     private function sluggifyFilename($filename): string
     {
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename = substr($filename, 0, strrpos($filename, '.'));
-        $filename = Str::slug($filename) . '.' . $extension;
+        $filename = Str::slug($filename).'.'.$extension;
 
         return $filename;
     }
@@ -160,9 +160,9 @@ 
                 if ($file instanceof UploadedFile && !$file->isValid()) {
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
                         throw new FileTooBigException(
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
                         );
                     }
                 }
@@ -177,7 +177,7 @@ 
             foreach ($files as $locale => $_files) {
                 foreach ($_files as $action => $file) {
                     if (!in_array($action, $actions)) {
-                        throw new \InvalidArgumentException('A valid files entry should have a key of either [' . implode(',', $actions) . ']. Instead ' . $action . ' is given.');
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
                     }
                 }
             }

src/Common/helpers.php

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
         $manifestPath = '/chief-assets/back';
 
         // Manifest expects each entry to start with a leading slash - we make sure to deduplicate the manifest path.
-        $entry = str_replace($manifestPath, '', '/' . ltrim($filepath, '/'));
+        $entry = str_replace($manifestPath, '', '/'.ltrim($filepath, '/'));
 
         try {
             // Paths should be given relative to the manifestpath so make sure to remove the basepath
@@ -28,7 +28,7 @@ 
         } catch (\Exception $e) {
             \Illuminate\Support\Facades\Log::error($e);
 
-            return $manifestPath . $entry;
+            return $manifestPath.$entry;
         }
     }
 }
@@ -114,7 +114,7 @@ 
         }
         $teaser = substr($text, 0, $max);
 
-        return strlen($text) <= $max ? $teaser : $teaser . $ending;
+        return strlen($text) <= $max ? $teaser : $teaser.$ending;
     }
 }
 
@@ -245,16 +245,16 @@ 
             'fragment',
         ], null), $parsed_url, $overrides);
 
-        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'] . '://' : null;
-        $port = $parsed_url['port'] ? ':' . $parsed_url['port'] : null;
-        $fragment = $parsed_url['fragment'] ? '#' . $parsed_url['fragment'] : null;
+        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'].'://' : null;
+        $port = $parsed_url['port'] ? ':'.$parsed_url['port'] : null;
+        $fragment = $parsed_url['fragment'] ? '#'.$parsed_url['fragment'] : null;
 
-        $baseurl = $scheme . $parsed_url['host'] . $port . $parsed_url['path'];
+        $baseurl = $scheme.$parsed_url['host'].$port.$parsed_url['path'];
         $current_query = [];
 
         $_query = explode('&', $parsed_url['query']);
 
-        array_map(function ($v) use (&$current_query) {
+        array_map(function($v) use (&$current_query) {
             if (!$v) {
                 return;
             }
@@ -272,7 +272,7 @@ 
 
         $query = urldecode(http_build_query(array_merge($current_query, $query_params)));
 
-        return $baseurl . '?' . $query . $fragment;
+        return $baseurl.'?'.$query.$fragment;
     }
 }
 

src/Common/Helpers/Memoize.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     {
         $cachableParameters = $this->convertToCachableParameters($parameters);
 
-        $cachekey = $this->baseKey . ':' . md5(implode('', $cachableParameters));
+        $cachekey = $this->baseKey.':'.md5(implode('', $cachableParameters));
 
         if (isset(static::$cache[$cachekey])) {
             return static::$cache[$cachekey];
@@ -37,7 +37,7 @@ 
     {
         foreach ($parameters as $key => $value) {
             if ($value instanceof Model) {
-                $parameters[$key] = get_class($value) . '@' . $value->id;
+                $parameters[$key] = get_class($value).'@'.$value->id;
             }
         }
 

src/Fields/Types/FieldType.php

Spacing +12 added lines, -12 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;

src/Fields/Validation/ValidationParameters.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
             }
 
             if (isset($customRules[$rule])) {
-                $rules[$k] = $customRules[$rule] . ($params ? ':' . $params : '');
+                $rules[$k] = $customRules[$rule].($params ? ':'.$params : '');
             }
         }
 

src/Fields/Validation/ValidationNames.php

Spacing +3 added lines, -3 removed lines

@@ -94,7 +94,7 @@ 
                 }
 
                 foreach ($replacements as $replacement) {
-                    $newKeySet[] = str_replace(':' . $placeholder, $replacement, $key);
+                    $newKeySet[] = str_replace(':'.$placeholder, $replacement, $key);
                 }
             }
 
@@ -142,7 +142,7 @@ 
 
             // Remove all 'trans' entries for this locale
             foreach ($filteredKeys as $i => $key) {
-                if (Str::startsWith($key, 'trans.' . $locale)) {
+                if (Str::startsWith($key, 'trans.'.$locale)) {
                     unset($filteredKeys[$i]);
                 }
             }
@@ -163,7 +163,7 @@ 
             $payload = Arr::get($this->payload, $key, '_notfound_');
 
             // If the payload is empty and this is not the entry for the required locale
-            if ($payload !== '_notfound_' && !$payload && !Str::endsWith($key, '.' . $this->requiredLocale)) {
+            if ($payload !== '_notfound_' && !$payload && !Str::endsWith($key, '.'.$this->requiredLocale)) {
                 unset($filteredKeys[$i]);
             }
         }

src/Fields/FieldName.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
             return $value;
         }
 
-        $value = str_replace('.', '][', $value) . ']';
+        $value = str_replace('.', '][', $value).']';
 
         return substr_replace($value, '', strpos($value, ']'), 1);
     }

src/Fields/SavingFields.php

Indentation +2 added lines, -2 removed lines

@@ -66,8 +66,8 @@
             foreach ($this->assistants() as $assistant) {
                 if (method_exists($assistant, $saveMethod)) {
                     $this->saveAssistantMethods[$field->getKey()] = ['field'     => $field,
-                                                                     'method'    => $saveMethod,
-                                                                     'assistant' => $assistant,
+                                                                        'method'    => $saveMethod,
+                                                                        'assistant' => $assistant,
                     ];
 
                     return true;

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,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);
@@ -48,8 +48,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) {
@@ -96,7 +96,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

src/Fields/Fields.php

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
                 continue;
             }
 
-            $method = 'get' . ucfirst($key);
+            $method = 'get'.ucfirst($key);
 
             // Reject from list if value does not match expected one
             if ($value && $value == $field->$method()) {
@@ -124,7 +124,7 @@ 
     public function offsetSet($offset, $value)
     {
         if (!$value instanceof Field) {
-            throw new \InvalidArgumentException('Passed value must be of type ' . Field::class);
+            throw new \InvalidArgumentException('Passed value must be of type '.Field::class);
         }
 
         $this->fields[$offset] = $value;
@@ -154,7 +154,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }