thinktomorrow / chief

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

src/Nav/Nav.php

Spacing +7 added lines, -7 removed lines

@@ -52,9 +52,9 @@ 
 
     private static function fromManagers(Collection $collection)
     {
-        return new static(...$collection->reject(function ($manager) {
+        return new static(...$collection->reject(function($manager) {
             return !$manager->can('index');
-        })->map(function ($manager) {
+        })->map(function($manager) {
             return new NavItem($manager->details()->plural, $manager->route('index'), [
                 'key'  => $manager->details()->key,
                 'tags' => app(Register::class)->filterByKey($manager->details()->key)->first()->tags(),
@@ -119,12 +119,12 @@ 
         $output = '';
 
         foreach ($this->items as $item) {
-            $output .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="' . $item->url() . '">';
+            $output .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $output .= $title ?? ucfirst($item->title());
             $output .= '</a>';
         }
 
-        return '<li>' . $output . '</li>';
+        return '<li>'.$output.'</li>';
     }
 
     /**
@@ -146,19 +146,19 @@ 
 
         $items = '';
         foreach ($this->items as $item) {
-            $items .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="' . $item->url() . '">';
+            $items .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $items .= ucfirst($item->title());
             $items .= '</a>';
         }
 
         // Surround within vue dropdown
         $output = '<dropdown>';
-        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">' . ($title ?? 'Collecties') . '</span>';
+        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'.($title ?? 'Collecties').'</span>';
         $output .= '<div v-cloak class="dropdown-box inset-s">';
         $output .= $items;
         $output .= '</div>';
         $output .= '</dropdown>';
 
-        return '<li>' . $output . '</li>';
+        return '<li>'.$output.'</li>';
     }
 }