thinktomorrow / chief

src/PageBuilder/UpdateSections.php

Spacing +7 added lines, -7 removed lines

@@ -91,7 +91,7 @@ 
     private function removeExistingSets()
     {
         foreach ($this->model->children() as $instance) {
-            if (! $instance instanceof StoredSetReference) {
+            if (!$instance instanceof StoredSetReference) {
                 continue;
             }
             $this->model->rejectChild($instance);
@@ -145,7 +145,7 @@ 
         }
 
         foreach ($this->text_modules['replace'] as $text_module) {
-            if (! $module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
+            if (!$module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
                 continue;
             }
             // Do not update if content of text is completely empty. We will remove this module instead
@@ -154,10 +154,10 @@ 
                 continue;
             }
            
-            foreach($text_module['trans'] as $locale => $content)
+            foreach ($text_module['trans'] as $locale => $content)
             {
                 $purifier = new HTMLPurifier();
-                $sanitized_text = $purifier->purify( $content['content'] );
+                $sanitized_text = $purifier->purify($content['content']);
 
                 $text_module['trans'][$locale]['content'] = $sanitized_text;
             }
@@ -171,7 +171,7 @@ 
 
     private function removeTextualModule($module)
     {
-        if (! $module instanceof TextModule && ! $module instanceof PagetitleModule) {
+        if (!$module instanceof TextModule && !$module instanceof PagetitleModule) {
             throw new \Exception('Invalid request to remove non textual module');
         }
 
@@ -193,7 +193,7 @@ 
                 continue;
             }
 
-            $child = $children->first(function ($c) use ($reference) {
+            $child = $children->first(function($c) use ($reference) {
                 return $c->flatReference()->get() == $reference;
             });
 
@@ -244,7 +244,7 @@ 
     private function stripTagsBlacklist($value, $blacklist = [])
     {
         foreach ($blacklist as $tag) {
-            $value = preg_replace('/<\/?' . $tag . '(.|\s)*?>/', '', $value);
+            $value = preg_replace('/<\/?'.$tag.'(.|\s)*?>/', '', $value);
         }
 
         return $value;

src/Media/UploadMedia.php

Spacing +14 added lines, -14 removed lines

@@ -50,7 +50,7 @@ 
 
     private function addFiles(HasMedia $model, string $type, array $files, array &$files_order, string $locale = null)
     {
-        if(!$this->actionExists($files, 'new')) return;
+        if (!$this->actionExists($files, 'new')) return;
 
         foreach ($files['new'] as $id => $file) {
             if (!$file) {
@@ -68,7 +68,7 @@ 
      */
     private function replaceFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'replace')) return;
+        if (!$this->actionExists($files, 'replace')) return;
 
         foreach ($files['replace'] as $id => $file) {
             if (!$file) {
@@ -86,7 +86,7 @@ 
      */
     private function removeFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'delete')) return;
+        if (!$this->actionExists($files, 'delete')) return;
 
         foreach ($model->assets()->whereIn('id', $files['delete'])->get() as $asset) {
             $asset->delete();
@@ -103,7 +103,7 @@ 
         if (is_string($file)) {
             $image_name = json_decode($file)->name;
             $asset      = $this->addAsset(json_decode($file)->data, $type, $locale, $image_name, $model);
-        } else {
+        }else {
             $image_name = $file->getClientOriginalName();
             $asset      = $this->addAsset($file, $type, $locale, $image_name, $model);
         }
@@ -126,7 +126,7 @@ 
 
         if (is_string($file)) {
             $asset = AssetUploader::uploadFromBase64($file, $filename);
-        } else {
+        }else {
             $asset = AssetUploader::upload($file, $filename);
         }
 
@@ -143,9 +143,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;
     }
@@ -161,9 +161,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'
                         );
                     }
                 }
@@ -178,16 +178,16 @@ 
         foreach ($files_by_type as $type => $files) {
             foreach ($files as $locale => $_files) {
                 if (!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
 
                 if (!is_array($_files)) {
-                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a ' . gettype($_files) . ' is given.');
+                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a '.gettype($_files).' is given.');
                 }
 
                 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.');
                     }
                 }
             }
@@ -196,7 +196,7 @@ 
         foreach ($files_order_by_type as $type => $fileIdsCollection) {
             foreach ($fileIdsCollection as $locale => $commaSeparatedFileIds) {
                 if (!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
             }
         }