thinktomorrow / chief

src/ManagedModels/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/ManagedModels/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/ManagedModels/Application/SortModels.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $params = [];
 
         foreach ($indices as $index => $modelId) {
-            $id = (int) $modelId;
+            $id = (int)$modelId;
             $cases[] = "WHEN {$id} then ?";
             $params[] = $index;
             $ids[] = $id;

src/ManagedModels/Fields/Types/FileField.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
     {
         return (new static(new FieldType(FieldType::FILE), $key))
             ->locales([config('app.fallback_locale', 'nl')])
-            ->valueResolver(function ($model = null, $locale = null, FileField $field) {
+            ->valueResolver(function($model = null, $locale = null, FileField $field) {
                 return $field->getMedia($model, $locale);
             });
     }
@@ -37,7 +37,7 @@ 
         $files = [];
         $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');
 
@@ -47,7 +47,7 @@ 
             $thumbUrl = $asset->url('thumb');
 
             // If the conversions haven't run yet, we'll use the original image until they are uploaded
-            if(!file_exists(public_path($thumbUrl))) {
+            if (!file_exists(public_path($thumbUrl))) {
                 $thumbUrl = $asset->url();
             }
 

src/ManagedModels/Fields/Types/AllowsTags.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
 
     public function tagged($tag): bool
     {
-        $tags = (array) $tag;
+        $tags = (array)$tag;
 
         return count(array_intersect($this->tags, $tags)) > 0;
     }
@@ -27,8 +27,8 @@ 
 
     public function untag($tag)
     {
-        foreach((array) $tag as $_tag) {
-            if(false !== ($k = array_search($_tag, $this->tags))) {
+        foreach ((array)$tag as $_tag) {
+            if (false !== ($k = array_search($_tag, $this->tags))) {
                 unset($this->tags[$k]);
             }
         }

src/Admin/HealthMonitor/Notifiers/ToastNotifier.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
 {
     public function onFailure(HealthCheck $healthCheck)
     {
-        session()->now($this->generateUniqueSessionKey(), ['type' => 'warning', 'message' => $healthCheck->message() ]);
+        session()->now($this->generateUniqueSessionKey(), ['type' => 'warning', 'message' => $healthCheck->message()]);
     }
 
     public function onSuccess(HealthCheck $healthCheck)
@@ -19,10 +19,10 @@ 
 
     private function generateUniqueSessionKey(): string
     {
-        $toastId = 'toast_messages.' . mt_rand(0, 9999);
+        $toastId = 'toast_messages.'.mt_rand(0, 9999);
 
         while (session()->get($toastId)) {
-            $toastId = 'toast_messages.' . mt_rand(0, 9999);
+            $toastId = 'toast_messages.'.mt_rand(0, 9999);
         }
 
         return $toastId;

src/Admin/HealthMonitor/Monitor.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                     app($notifier)->onFailure($checkInstance);
                 }
                 break;
-            } else {
+            }else {
                 foreach ($notifiers as $notifier) {
                     app($notifier)->onSuccess($checkInstance);
                 }

src/ManagedModels/States/Publishable/PreviewMode.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 
     public static function fromRequest()
     {
-        if(!config('chief.preview-mode') || Str::startsWith(request()->path(), 'admin/')) {
+        if (!config('chief.preview-mode') || Str::startsWith(request()->path(), 'admin/')) {
             return new static(false);
         }
 
@@ -40,7 +40,7 @@ 
     {
         $mode = config('chief.preview-mode');
 
-        if(!$mode || $mode == 'live') return false;
+        if (!$mode || $mode == 'live') return false;
 
         return ($mode == 'preview');
     }

Braces +3 added lines, -1 removed lines

@@ -40,7 +40,9 @@
     {
         $mode = config('chief.preview-mode');
 
-        if(!$mode || $mode == 'live') return false;
+        if(!$mode || $mode == 'live') {
+            return false;
+        }
 
         return ($mode == 'preview');
     }

src/Managers/Assistants/FragmentAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -32,18 +32,18 @@ 
 
     public function routeFragmentAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store', 'fragment-status'])) {
+        if (!in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store', 'fragment-status'])) {
             return null;
         }
 
         $modelKey = $this->managedModelClass()::managedModelKey();
 
-        if(in_array($action, ['fragment-create', 'fragment-store'])) {
+        if (in_array($action, ['fragment-create', 'fragment-store'])) {
             if (!$model || !$model instanceof FragmentsOwner) {
                 throw new \Exception('Fragment route definition for '.$action.' requires the owning Model as second argument.');
             }
 
-            return route('chief.' . $modelKey . '.' . $action, array_merge([
+            return route('chief.'.$modelKey.'.'.$action, array_merge([
                 $model::managedModelKey(),
                 $model->modelReference()->id(),
             ], $parameters));
@@ -53,12 +53,12 @@ 
             throw new \Exception('Fragment route definition for '.$action.' requires the fragment model as second argument.');
         }
 
-        return route('chief.' . $modelKey . '.' . $action, $model->fragmentModel()->id);
+        return route('chief.'.$modelKey.'.'.$action, $model->fragmentModel()->id);
     }
 
     public function canFragmentAssistant(string $action, $model = null): bool
     {
-        return in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store', 'fragment-status']);
+        return in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store', 'fragment-status']);
     }
 
     public function fragmentCreate(Request $request, string $ownerKey, $ownerId)
@@ -159,13 +159,13 @@ 
         $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.');
     }
 
     private function owner(string $ownerKey, $ownerId): FragmentsOwner