thinktomorrow / chief

src/Management/Application/DeleteManagedModel.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
             DB::beginTransaction();
 
             // For stateful transitions we will apply this deletion as a state
-            if($model instanceof StatefulContract) {
+            if ($model instanceof StatefulContract) {
                 (new PageState($model))->apply('delete');
                 $model->save();
             }
@@ -29,17 +29,17 @@ 
             Relation::deleteRelationsOf($model->getMorphClass(), $model->id);
 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
-            if($model instanceof Module) {
+            if ($model instanceof Module) {
                 $model->update([
-                    'slug' => $model->slug . $this->appendDeleteMarker(),
+                    'slug' => $model->slug.$this->appendDeleteMarker(),
                 ]);
             }
 
-            if($model instanceof ProvidesUrl){
+            if ($model instanceof ProvidesUrl) {
                 UrlRecord::getByModel($model)->each->delete();
             }
 
-            if($model instanceof Page){
+            if ($model instanceof Page) {
                 Module::where('page_id', $model->id)->delete();
             }
 
@@ -58,6 +58,6 @@ 
 
     private function appendDeleteMarker(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

src/Management/Managers.php

Spacing +6 added lines, -6 removed lines

@@ -50,7 +50,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
@@ -65,7 +65,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration)->details();
         });
     }
@@ -76,7 +76,7 @@ 
         //return array with group name and values
         $grouped = [];
 
-        $managers = $managers->map(function (Manager $item) {
+        $managers = $managers->map(function(Manager $item) {
             return [
                 'id'    => $item->details()->id,
                 'group' => $item->details()->plural,
@@ -90,7 +90,7 @@ 
     {
         $registrations = collect($this->register->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
@@ -99,7 +99,7 @@ 
     {
         $assistants = collect();
 
-        foreach($this->all() as $manager) {
+        foreach ($this->all() as $manager) {
             $assistants = $assistants->merge($manager->assistants(false));
         }
 
@@ -108,7 +108,7 @@ 
 
     public function anyRegisteredByTag($tag)
     {
-        return ! empty($this->register->filterByTag($tag)->all());
+        return !empty($this->register->filterByTag($tag)->all());
     }
 
     /**

src/States/Featurable/TestFeaturedAssistant.php

Indentation +3 added lines, -3 removed lines

@@ -13,10 +13,10 @@
 
     public static function register($app)
     {
-         Route::get('testje', [static::class, 'archive']);
-         Route::get('testje', [AssistantController, 'archive']);
+            Route::get('testje', [static::class, 'archive']);
+            Route::get('testje', [AssistantController, 'archive']);
 
-         // fetched manager
+            // fetched manager
         // propagate naar assistant::archive
     }
 

src/Management/Assistants/ManagesAssistants.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     public function assistant(string $assistantKey): Assistant
     {
         if (!$this->isAssistedBy($assistantKey)) {
-            throw new MissingAssistant('No assistant [' . $assistantKey . '] registered on manager ' . static::class);
+            throw new MissingAssistant('No assistant ['.$assistantKey.'] registered on manager '.static::class);
         }
 
         $instance = app($this->getAssistantClass($assistantKey));

src/States/Archivable/Archivable.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
     public function scopeUnarchived($query)
     {
-        $query->withoutGlobalScope(ArchiveScope::class)->where(PageState::KEY,'<>', PageState::ARCHIVED);
+        $query->withoutGlobalScope(ArchiveScope::class)->where(PageState::KEY, '<>', PageState::ARCHIVED);
     }
 
     public function scopeWithArchived($query)

src/States/AbstractStateAssistant.php

Spacing +3 added lines, -3 removed lines

@@ -20,8 +20,8 @@ 
         $this->manager  = $manager;
         $this->model    = $manager->existingModel();
 
-        if(!$this->model instanceof StatefulContract){
-            throw new \InvalidArgumentException(static::class . ' requires the model to implement the ' . StatefulContract::class);
+        if (!$this->model instanceof StatefulContract) {
+            throw new \InvalidArgumentException(static::class.' requires the model to implement the '.StatefulContract::class);
         }
     }
 
@@ -36,7 +36,7 @@ 
 
     public function guard($verb): Assistant
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this->manager);
         }
 

src/States/State/StateMachine.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
     {
         $transitions = [];
 
-        foreach($this->transitions as $transitionKey => $transition){
+        foreach ($this->transitions as $transitionKey => $transition) {
             if (false !== array_search($this->statefulContract->stateOf($this->stateKey), $transition['from'])) {
                 $transitions[] = $transitionKey;
             }
@@ -107,7 +107,7 @@ 
     private function validateTransitions()
     {
         foreach ($this->transitions as $transitionKey => $transition) {
-            if (!array_key_exists('from', $transition) || !array_key_exists('to',$transition) || !is_array($transition['from'])) {
+            if (!array_key_exists('from', $transition) || !array_key_exists('to', $transition) || !is_array($transition['from'])) {
                 throw StateException::malformedTransition($transitionKey, $this);
             }
 

src/Management/Assistants/ArchiveAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -50,7 +50,7 @@ 
 
     public function findAllArchived(): Collection
     {
-        return $this->manager->modelInstance()::archived()->get()->map(function ($model) {
+        return $this->manager->modelInstance()::archived()->get()->map(function($model) {
             return $this->managers->findByModel($model);
         });
     }
@@ -58,7 +58,7 @@ 
     public function route($verb): ?string
     {
         $routes = [
-            'index' => route('chief.back.assistants.view', [$this->key(),'index', $this->manager->managerKey()]),
+            'index' => route('chief.back.assistants.view', [$this->key(), 'index', $this->manager->managerKey()]),
         ];
 
         if (array_key_exists($verb, $routes)) {
@@ -80,11 +80,11 @@ 
 
     private function guard($verb): Assistant
     {
-        if(!$this->manager->existingModel() instanceof StatefulContract){
+        if (!$this->manager->existingModel() instanceof StatefulContract) {
             throw new \InvalidArgumentException('ArchiveAssistant requires the model to implement the StatefulContract. ['.get_class($this->model).'] given instead.');
         }
 
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this->manager);
         }
 
@@ -111,7 +111,7 @@ 
 
             // Ok now get all urls from this model and point them to the new records
             foreach (UrlRecord::getByModel($this->manager->existingModel()) as $urlRecord) {
-                if ($targetRecord = $targetRecords->first(function ($record) use ($urlRecord) {
+                if ($targetRecord = $targetRecords->first(function($record) use ($urlRecord) {
                     return ($record->locale == $urlRecord->locale && !$record->isRedirect());
                 })) {
                     $urlRecord->redirectTo($targetRecord);
@@ -121,7 +121,7 @@ 
 
         app(ArchiveManagedModel::class)->handle($this->manager->existingModel());
 
-        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title .' is gearchiveerd.');
+        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title.' is gearchiveerd.');
     }
 
     public function unarchive()
@@ -130,6 +130,6 @@ 
 
         app(UnArchiveManagedModel::class)->handle($this->manager->existingModel());
 
-        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title .' is hersteld.');
+        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title.' is hersteld.');
     }
 }

src/Management/Assistants/PublishAssistant.php

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
 
         app(PublishManagedModel::class)->handle($this->manager->existingModel());
 
-        return redirect()->to($this->manager->route('edit'))->with('messages.success', $this->manager->details()->title .' is online gezet.');
+        return redirect()->to($this->manager->route('edit'))->with('messages.success', $this->manager->details()->title.' is online gezet.');
     }
 
     public function unpublish()
@@ -64,12 +64,12 @@ 
 
         app(UnpublishManagedModel::class)->handle($this->manager->existingModel());
 
-        return redirect()->to($this->manager->route('edit'))->with('messages.success', $this->manager->details()->title .' is offline gehaald.');
+        return redirect()->to($this->manager->route('edit'))->with('messages.success', $this->manager->details()->title.' is offline gehaald.');
     }
 
     public function findAll(): Collection
     {
-        return $this->manager->existingModel()->published()->get()->map(function ($model) {
+        return $this->manager->existingModel()->published()->get()->map(function($model) {
             return $this->managers->findByModel($model);
         });
     }
@@ -91,7 +91,7 @@ 
 
     private function guard($verb): Assistant
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this->manager);
         }
 
@@ -116,10 +116,10 @@ 
             $class = 'text-warning';
         }
 
-        $statusAsLabel = '<span class="font-bold '. $class .'"><em>' . $label . '</em></span>';
+        $statusAsLabel = '<span class="font-bold '.$class.'"><em>'.$label.'</em></span>';
 
         if (!$plain && $this->hasPreviewUrl()) {
-            $statusAsLabel =  '<a href="'.$this->previewUrl().'" target="_blank">'. $statusAsLabel .'</a>';
+            $statusAsLabel = '<a href="'.$this->previewUrl().'" target="_blank">'.$statusAsLabel.'</a>';
         }
 
         return $statusAsLabel;