thinktomorrow / chief

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/Management/Application/DeleteManagedModel.php

Spacing +2 added lines, -2 removed lines

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

src/Modules/Module.php

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,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.'.');
     }
 
     /**
@@ -79,9 +79,9 @@ 
      */
     public static function availableForCreation(): Collection
     {
-        $managers = app(Managers::class)->findByTag('module')->filter(function ($manager) {
+        $managers = app(Managers::class)->findByTag('module')->filter(function($manager) {
             return $manager->can('create');
-        })->map(function ($manager) {
+        })->map(function($manager) {
             return $manager->details();
         });
 

src/Management/Assistants/UrlAssistant.php

Spacing +3 added lines, -3 removed lines

@@ -69,9 +69,9 @@ 
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($this->manager->existingModel())->reject(function ($record) {
+        UrlRecord::getByModel($this->manager->existingModel())->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
     }
@@ -84,7 +84,7 @@ 
     private function validateModel()
     {
         if (!$this->manager->existingModel() instanceof ProvidesUrl) {
-            throw new \Exception('UrlAssistant requires the model interfaced by ' . ProvidesUrl::class . '.');
+            throw new \Exception('UrlAssistant requires the model interfaced by '.ProvidesUrl::class.'.');
         }
     }
 }

src/Management/Assistants/ArchiveAssistant.php

Spacing +5 added lines, -5 removed lines

@@ -49,7 +49,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);
         });
     }
@@ -90,7 +90,7 @@ 
     private function guard($verb): Assistant
     {
         if (!$this->manager->existingModel() instanceof StatefulContract) {
-            throw new \InvalidArgumentException('ArchiveAssistant requires the model to implement the StatefulContract. [' . get_class($this->manager->existingModel()) . '] given instead.');
+            throw new \InvalidArgumentException('ArchiveAssistant requires the model to implement the StatefulContract. ['.get_class($this->manager->existingModel()).'] given instead.');
         }
 
         if (!$this->can($verb)) {
@@ -120,7 +120,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);
@@ -137,7 +137,7 @@ 
             ->performedOn($this->manager->existingModel())
             ->log('archived');
 
-        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()
@@ -153,6 +153,6 @@ 
             ->performedOn($this->manager->existingModel())
             ->log('unarchived');
 
-        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/Registration.php

Spacing +4 added lines, -4 removed lines

@@ -75,21 +75,21 @@
     private function validate($managerClass, $modelClass)
     {
         if (!class_exists($managerClass)) {
-            throw new \InvalidArgumentException('Manager class [' . $managerClass . '] is an invalid class reference. Please make sure the class exists.');
+            throw new \InvalidArgumentException('Manager class ['.$managerClass.'] is an invalid class reference. Please make sure the class exists.');
         }
 
         if (!class_exists($modelClass)) {
-            throw new \InvalidArgumentException('Model class [' . $modelClass . '] is an invalid model reference. Please make sure the class exists.');
+            throw new \InvalidArgumentException('Model class ['.$modelClass.'] is an invalid model reference. Please make sure the class exists.');
         }
 
         $manager = new \ReflectionClass($managerClass);
         if (!$manager->implementsInterface(Manager::class)) {
-            throw new \InvalidArgumentException('Class [' . $managerClass . '] is expected to implement the [' . Manager::class . '] contract.');
+            throw new \InvalidArgumentException('Class ['.$managerClass.'] is expected to implement the ['.Manager::class.'] contract.');
         }
 
         $model = new \ReflectionClass($modelClass);
         if (!$model->implementsInterface(ManagedModel::class)) {
-            throw new \InvalidArgumentException('Class [' . $modelClass . '] is expected to implement the [' . ManagedModel::class . '] contract.');
+            throw new \InvalidArgumentException('Class ['.$modelClass.'] is expected to implement the ['.ManagedModel::class.'] contract.');
         }
     }
 }

src/Management/Register.php

Spacing +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
     private function registrationMustExistConstraint(string $key, $value, $registrations): void
     {
         if (empty($registrations) && count($registrations) != $this->registrations) {
-            throw new NonRegisteredManager('No manager found for ' . $key . ' [' . print_r($value, true) . ']. Did you perhaps forgot to register the manager?');
+            throw new NonRegisteredManager('No manager found for '.$key.' ['.print_r($value, true).']. Did you perhaps forgot to register the manager?');
         }
     }
 }

src/Management/Exceptions/NotAllowedManagerRoute.php

Spacing +2 added lines, -2 removed lines

@@ -39,11 +39,11 @@
 
     public static function notAllowedVerb($verb, Manager $manager)
     {
-        throw new self('Not allowed to ' . $verb . ' a model. ' . ucfirst($verb) . ' route is not allowed by the ' . $manager->details()->key . ' manager.');
+        throw new self('Not allowed to '.$verb.' a model. '.ucfirst($verb).' route is not allowed by the '.$manager->details()->key.' manager.');
     }
 
     public static function notAllowedPermission($permission, Manager $manager)
     {
-        throw new self('Not allowed permission for ' . $permission . ' on a model as managed by the ' . $manager->details()->key . ' manager.');
+        throw new self('Not allowed permission for '.$permission.' on a model as managed by the '.$manager->details()->key.' manager.');
     }
 }

src/Management/Details/HasDetails.php

Spacing +4 added lines, -4 removed lines

@@ -16,7 +16,7 @@ 
         $genericModelInstance = $this->modelInstance();
 
         // Generic model details
-        $id = Str::slug($this->registration->key() . ($this->hasExistingModel() ? '-' . $this->existingModel()->id : ''));
+        $id = Str::slug($this->registration->key().($this->hasExistingModel() ? '-'.$this->existingModel()->id : ''));
         $key = $this->registration->key();
         $labelSingular = property_exists($genericModelInstance, 'labelSingular') ? $genericModelInstance->labelSingular : Str::singular($key);
         $labelPlural = property_exists($genericModelInstance, 'labelPlural') ? $genericModelInstance->labelPlural : Str::plural($key);
@@ -32,10 +32,10 @@ 
         return new Details(
             $id,
             $key,
-            $labelSingular . '',
-            $labelPlural . '',
+            $labelSingular.'',
+            $labelPlural.'',
             $internal_label,
-            $title . ''
+            $title.''
         );
     }
 }