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/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.');
     }
 }

src/ManagedModels/States/PageStatePresenter.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     {
         $class = $this->pageState->isOnline() ? 'text-success' : 'text-warning';
 
-        return '<span class="inline-xs stack-s ' . $class . '">' . $this->stateAsLabel() . '</span>';
+        return '<span class="inline-xs stack-s '.$class.'">'.$this->stateAsLabel().'</span>';
     }
 
     /**

src/ManagedModels/States/State/StateException.php

Spacing +5 added lines, -5 removed lines

@@ -6,26 +6,26 @@
 {
     public static function invalidState($state, $currentState, $stateMachine)
     {
-        return new self('Transition to state [' . $state . '] is not allowed from state [' . $currentState . '] or state does not exist on ' . get_class($stateMachine));
+        return new self('Transition to state ['.$state.'] is not allowed from state ['.$currentState.'] or state does not exist on '.get_class($stateMachine));
     }
 
     public static function malformedTransition($transition, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] is malformed on ' . get_class($stateMachine) . '. It should contain both a [from:array] and [to:string] value.');
+        return new self('Transition ['.$transition.'] is malformed on '.get_class($stateMachine).'. It should contain both a [from:array] and [to:string] value.');
     }
 
     public static function invalidTransitionKey($transition, $stateMachine)
     {
-        return new self('unknown transition [' . $transition . '] on ' . get_class($stateMachine));
+        return new self('unknown transition ['.$transition.'] on '.get_class($stateMachine));
     }
 
     public static function invalidTransition($transition, $state, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] cannot be applied from current state [' . $state . '] on ' . get_class($stateMachine));
+        return new self('Transition ['.$transition.'] cannot be applied from current state ['.$state.'] on '.get_class($stateMachine));
     }
 
     public static function invalidTransitionState($transition, $state, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] contains a non existing [' . $state . '] on ' . get_class($stateMachine));
+        return new self('Transition ['.$transition.'] contains a non existing ['.$state.'] on '.get_class($stateMachine));
     }
 }

src/ManagedModels/States/Publishable/CheckPreviewTrait.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
             Session::now('note.default', 'U bekijkt een preview.');
 
             return true;
-        } else {
+        }else {
             return false;
         }
     }