thinktomorrow / chief

src/Modules/Application/DeleteModule.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
             $module->update([
-                'slug' => $module->slug . $this->appendDeleteMarker(),
+                'slug' => $module->slug.$this->appendDeleteMarker(),
             ]);
 
             $module->delete();
@@ -35,6 +35,6 @@ 
 
     private function appendDeleteMarker(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

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/Media/Application/AbstractMediaFieldHandler.php

Spacing +4 added lines, -4 removed lines

@@ -140,10 +140,10 @@ 
             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;
     }
 
     protected function sort(HasAsset $model, MediaField $field, array $input)
@@ -169,8 +169,8 @@ 
         $values = isset($fileIdInput[$key])
             ? $fileIdInput[$key]
             : (
-                isset($fileIdInput['files-' . $key])
-                ? $fileIdInput['files-' . $key]
+                isset($fileIdInput['files-'.$key])
+                ? $fileIdInput['files-'.$key]
                 : ''
             );
 

src/Media/Application/FileFieldHandler.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 {
     public function handle(HasAsset $model, MediaField $field, array $input, array $files): void
     {
-        foreach ([data_get($files, 'files.' . $field->getName(), []), data_get($input, 'files.' . $field->getName(), [])] as $requestPayload) {
+        foreach ([data_get($files, 'files.'.$field->getName(), []), data_get($input, 'files.'.$field->getName(), [])] as $requestPayload) {
             foreach ($requestPayload as $locale => $values) {
                 $this->handlePayload($model, $field, $locale, $values);
             }

src/Modules/ModuleManager.php

Spacing +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
             if (is_array_empty($translation)) {
 
                 // Nullify all values
-                $trans[$locale] = array_map(function ($value) {
+                $trans[$locale] = array_map(function($value) {
                     return null;
                 }, $translation);
                 continue;

src/Managers/Assistants/RedactorFileUploadAssistant.php

Spacing +3 added lines, -3 removed lines

@@ -43,16 +43,16 @@
             $filename = $filePayload['filename'];
 
             if (!$asset = AssetUploader::uploadFromBase64($base64EncodedFile, $filename)) {
-                $responseContent['file-' . rand(1 - 999)] = [
+                $responseContent['file-'.rand(1-999)] = [
                     'error'    => true,
-                    'messages' => 'Afbeelding [' . $filename . '] kan niet worden opgeladen.',
+                    'messages' => 'Afbeelding ['.$filename.'] kan niet worden opgeladen.',
                 ];
                 continue;
             }
 
             app(AddAsset::class)->add($model, $asset, MediaType::CONTENT, $request->input('locale', app()->getLocale()));
 
-            $responseContent['file-' . $asset->id] = [
+            $responseContent['file-'.$asset->id] = [
                 'url' => $asset->url(),
                 'id'  => $asset->id,
             ];

src/Managers/Assistants/PreviewAssistant.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function canPreviewAssistant(string $action, $model = null): bool
     {
-        if($action !== 'preview' || !$model) return false;
+        if ($action !== 'preview' || !$model) return false;
 
         return $model instanceof ProvidesUrl;
     }
@@ -27,7 +27,7 @@ 
      */
     public function routePreviewAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPreviewAssistant($action, $model)) return null;
+        if (!$this->canPreviewAssistant($action, $model)) return null;
 
         return $model->url();
     }

Braces +6 added lines, -2 removed lines

@@ -9,7 +9,9 @@ 
 {
     public function canPreviewAssistant(string $action, $model = null): bool
     {
-        if($action !== 'preview' || !$model) return false;
+        if($action !== 'preview' || !$model) {
+            return false;
+        }
 
         return $model instanceof ProvidesUrl;
     }
@@ -27,7 +29,9 @@ 
      */
     public function routePreviewAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPreviewAssistant($action, $model)) return null;
+        if(!$this->canPreviewAssistant($action, $model)) {
+            return null;
+        }
 
         return $model->url();
     }

src/Legacy/Pages/Page.php

Spacing +7 added lines, -7 removed lines

@@ -98,7 +98,7 @@ 
             return static::$managedModelKey;
         }
 
-        throw new \Exception('Missing required static property \'managedModelKey\' on ' . static::class . '.');
+        throw new \Exception('Missing required static property \'managedModelKey\' on '.static::class.'.');
     }
 
     /**
@@ -138,9 +138,9 @@ 
     public function modelReferenceLabel(): string
     {
         if ($this->exists) {
-            $status = !$this->isPublished() ? ' [' . $this->statusAsPlainLabel() . ']' : null;
+            $status = !$this->isPublished() ? ' ['.$this->statusAsPlainLabel().']' : null;
 
-            return $this->title ? $this->title . $status : '';
+            return $this->title ? $this->title.$status : '';
         }
 
         return '';
@@ -151,7 +151,7 @@ 
         $classKey = get_class($this);
         if (property_exists($this, 'labelSingular')) {
             $labelSingular = $this->labelSingular;
-        } else {
+        }else {
             $labelSingular = Str::singular($classKey);
         }
 
@@ -170,7 +170,7 @@ 
             $locale = app()->getLocale();
         }
         try {
-            $memoizedKey = $this->getMorphClass() . '-' . $this->id . '-' . $locale;
+            $memoizedKey = $this->getMorphClass().'-'.$this->id.'-'.$locale;
 
             if (isset(static::$cachedUrls[$memoizedKey])) {
                 return static::$cachedUrls[$memoizedKey];
@@ -226,11 +226,11 @@ 
     public function statusAsLabel()
     {
         if ($this->isPublished()) {
-            return '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>';
+            return '<a href="'.$this->url().'" target="_blank"><em>online</em></a>';
         }
 
         if ($this->isDraft()) {
-            return '<a href="' . $this->url() . '" target="_blank" class="text-error"><em>offline</em></a>';
+            return '<a href="'.$this->url().'" target="_blank" class="text-error"><em>offline</em></a>';
         }
 
         if ($this->isArchived()) {

src/Modules/Assistants/ModuleCrudAssistant.php

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
 
     public function routeModuleCrudAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['create','store'])) return null;
+        if(!in_array($action, ['create','store'])) {
+            return null;
+        }
 
         return $this->generateRoute($action, $model, ... $parameters);
     }

Spacing +6 added lines, -6 removed lines

@@ -30,7 +30,7 @@ 
 
     public function routeModuleCrudAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['create','store'])) return null;
+        if (!in_array($action, ['create', 'store'])) return null;
 
         return $this->generateRoute($action, $model, ... $parameters);
     }
@@ -59,7 +59,7 @@ 
         $model->saveFields($model->fields()->tagged('create'), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('edit', $model))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is toegevoegd');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is toegevoegd');
     }
 
     public function edit($id)
@@ -74,7 +74,7 @@ 
         return view('chief::back.managers.edit', [
             'manager'    => $this,
             'model'      => $model,
-            'fields'     => $model->fields()->map(function (Field $field) use ($model) {
+            'fields'     => $model->fields()->map(function(Field $field) use ($model) {
                 // TODO refactor so render method of field takes model and managerViewModel as arguments.
                 return $field->model($model);
             }),
@@ -93,7 +93,7 @@ 
         $model->saveFields($model->fields(), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('edit', $model))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is aangepast');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is aangepast');
     }
 
     public function delete($id, Request $request)
@@ -103,12 +103,12 @@ 
         $model = $this->managedModelClass()::findOrFail($id);
 
         if ($request->get('deleteconfirmation') !== 'DELETE') {
-            return redirect()->back()->with('messages.warning', $model->adminLabel('title') . ' is niet verwijderd.');
+            return redirect()->back()->with('messages.warning', $model->adminLabel('title').' is niet verwijderd.');
         }
 
         app(DeleteModel::class)->handle($model);
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is verwijderd.');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is verwijderd.');
     }
 }