thinktomorrow / chief

src/Media/Application/ImageFieldHandler.php

Spacing +4 added lines, -4 removed lines

@@ -16,8 +16,8 @@ 
             $request->input('images.'.$field->getName(), []),
         ], $field, $request);
 
-        foreach([MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH] as $action) {
-            foreach($mediaRequest->getByKey($action) as $input) {
+        foreach ([MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH] as $action) {
+            foreach ($mediaRequest->getByKey($action) as $input) {
                 $this->$action($model, $input);
             }
         }
@@ -27,7 +27,7 @@ 
 
     private function new(HasAsset $model, MediaRequestInput $mediaRequestInput): Asset
     {
-        if($mediaRequestInput->metadata('existing_asset')) {
+        if ($mediaRequestInput->metadata('existing_asset')) {
             return $this->newExistingAsset($model, $mediaRequestInput);
         }
 
@@ -59,7 +59,7 @@ 
 
     protected function createNewAsset(HasAsset $model, MediaRequestInput $mediaRequestInput): Asset
     {
-        if($mediaRequestInput->metadata('existing_asset')) {
+        if ($mediaRequestInput->metadata('existing_asset')) {
             return Asset::find($mediaRequestInput->value());
         }
 

src/Media/Application/FileFieldHandler.php

Spacing +3 added lines, -3 removed lines

@@ -19,8 +19,8 @@ 
             $request->input('files.'.$field->getName(), []),
         ], $field, $request);
 
-        foreach([MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH] as $action) {
-            foreach($mediaRequest->getByKey($action) as $input) {
+        foreach ([MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH] as $action) {
+            foreach ($mediaRequest->getByKey($action) as $input) {
                 $this->$action($model, $input);
             }
         }
@@ -31,7 +31,7 @@ 
 
     private function new(HasAsset $model, MediaRequestInput $mediaRequestInput): Asset
     {
-        if($mediaRequestInput->metadata('existing_asset')) {
+        if ($mediaRequestInput->metadata('existing_asset')) {
             return $this->newExistingAsset($model, $mediaRequestInput);
         }
 

src/Media/Application/AbstractMediaFieldHandler.php

Spacing +8 added lines, -8 removed lines

@@ -39,15 +39,15 @@ 
     {
         $mediaRequest = new MediaRequest();
 
-        foreach($requests as $requestData){
-            foreach($requestData as $locale => $filesPerLocale) {
-                foreach($filesPerLocale as $action => $files) {
+        foreach ($requests as $requestData) {
+            foreach ($requestData as $locale => $filesPerLocale) {
+                foreach ($filesPerLocale as $action => $files) {
 
-                    if(!is_array($files) || !in_array($action, [MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH])) {
+                    if (!is_array($files) || !in_array($action, [MediaRequest::NEW, MediaRequest::REPLACE, MediaRequest::DETACH])) {
                         throw new \InvalidArgumentException('Malformed request data. Files are expected to be passed in a localized array.');
                     }
 
-                    foreach($files as $k => $file) {
+                    foreach ($files as $k => $file) {
                         $mediaRequest->add($action, new MediaRequestInput(
                             $file, $locale, $field->getKey(), [
                                 'index' => $k, // index key is used for replace method to indicate the current asset id
@@ -91,11 +91,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;
     }
 }

src/Fields/ValidationRules/ImageFieldMimetypesRule.php

Spacing +6 added lines, -6 removed lines

@@ -10,19 +10,19 @@ 
     {
         $value = $this->normalizePayload($value);
 
-        foreach([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
-            foreach($value[$type] as $file) {
-                if($file && false !== $this->validateMimetypes($attribute, $file, $params)) {
+        foreach ([MediaRequest::NEW, MediaRequest::REPLACE] as $type) {
+            foreach ($value[$type] as $file) {
+                if ($file && false !== $this->validateMimetypes($attribute, $file, $params)) {
                     return true;
                 }
             }
         }
 
         $validator->setCustomMessages([
-            'imagefield_mimetypes' => 'De :attribute is niet het juiste bestandstype. Volgende types zijn geldig: ' . implode(', ', $params),
+            'imagefield_mimetypes' => 'De :attribute is niet het juiste bestandstype. Volgende types zijn geldig: '.implode(', ', $params),
         ]);
 
-        if(!isset($validator->customAttributes[$attribute])) {
+        if (!isset($validator->customAttributes[$attribute])) {
             $validator->addCustomAttributes([
                 $attribute => 'afbeelding',
             ]);
@@ -34,7 +34,7 @@ 
 
     public function validateMimetypes($attribute, $value, $parameters)
     {
-        if($this->refersToExistingAsset($value)) {
+        if ($this->refersToExistingAsset($value)) {
             return $this->validateAssetMimetypes($this->existingAsset($value), $parameters);
         }
 

src/Fields/Validation/ValidationNames.php

Spacing +11 added lines, -11 removed lines

@@ -35,7 +35,7 @@ 
         $this->format = $format;
         $this->placeholders = $this->payload = $this->keysToBeRemoved = [];
 
-        $this->requiredLocale((string) config('app.fallback_locale', 'nl'));
+        $this->requiredLocale((string)config('app.fallback_locale', 'nl'));
     }
 
     public static function fromFormat(string $format)
@@ -47,7 +47,7 @@ 
     {
         $names = [$this->format];
 
-        foreach($this->filters as $filter){
+        foreach ($this->filters as $filter) {
             $names = call_user_func_array([$this, $filter], [$names]);
         }
 
@@ -88,13 +88,13 @@ 
             $newKeySet = [];
             foreach ($keys as $i => $key) {
 
-                if(count($replacements) < 1) {
+                if (count($replacements) < 1) {
                     $newKeySet[] = $key;
                     continue;
                 }
 
                 foreach ($replacements as $replacement) {
-                    $newKeySet[] = str_replace(':' . $placeholder, $replacement, $key);
+                    $newKeySet[] = str_replace(':'.$placeholder, $replacement, $key);
                 }
             }
 
@@ -108,7 +108,7 @@ 
     {
         $filteredKeys = $keys;
 
-        foreach($this->keysToBeRemoved as $keyToBeRemoved) {
+        foreach ($this->keysToBeRemoved as $keyToBeRemoved) {
             $pattern = preg_quote($keyToBeRemoved, '#');
 
             /* Any asterix which work as an wildcard of characters */
@@ -116,7 +116,7 @@ 
                 $pattern = str_replace('\*', '(.+)', $pattern);
             }
 
-            foreach($filteredKeys as $k => $filteredKey) {
+            foreach ($filteredKeys as $k => $filteredKey) {
                 if (preg_match("#$pattern#", $filteredKey)) {
                     unset($filteredKeys[$k]);
                 }
@@ -136,13 +136,13 @@ 
 
         // Remove locales that are considered empty in the request payload
         foreach ($this->payload['trans'] as $locale => $values) {
-            if ($locale == $this->requiredLocale || ! is_array_empty($values)) {
+            if ($locale == $this->requiredLocale || !is_array_empty($values)) {
                 continue;
             }
 
             // Remove all 'trans' entries for this locale
-            foreach($filteredKeys as $i => $key){
-                if(Str::startsWith($key, 'trans.'.$locale)) {
+            foreach ($filteredKeys as $i => $key) {
+                if (Str::startsWith($key, 'trans.'.$locale)) {
                     unset($filteredKeys[$i]);
                 }
             }
@@ -155,13 +155,13 @@ 
     {
         $filteredKeys = $keys;
 
-        foreach($filteredKeys as $i => $key) {
+        foreach ($filteredKeys as $i => $key) {
             if (!Str::startsWith($key, ['images.', 'files.'])) continue;
 
             $payload = Arr::get($this->payload, $key);
 
             // If the payload is empty and this is not the entry for the required locale
-            if(!$payload && !Str::endsWith($key, '.'.$this->requiredLocale)) {
+            if (!$payload && !Str::endsWith($key, '.'.$this->requiredLocale)) {
                 unset($filteredKeys[$i]);
             }
         }

Braces +3 added lines, -1 removed lines

@@ -156,7 +156,9 @@
         $filteredKeys = $keys;
 
         foreach($filteredKeys as $i => $key) {
-            if (!Str::startsWith($key, ['images.', 'files.'])) continue;
+            if (!Str::startsWith($key, ['images.', 'files.'])) {
+                continue;
+            }
 
             $payload = Arr::get($this->payload, $key);