thinktomorrow / chief

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/AbstractManager.php

Spacing +3 added lines, -3 removed lines

@@ -88,7 +88,7 @@ 
             return $this->indexPagination($builder);
         }
 
-        return $builder->get()->map(function ($model) {
+        return $builder->get()->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -116,7 +116,7 @@ 
     {
         $paginator = $builder->paginate($this->pageCount);
 
-        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function ($model) {
+        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function($model) {
             return (new static($this->registration))->manage($model);
         });
 
@@ -307,7 +307,7 @@ 
             static::$bootedTraitMethods[$baseMethod] = [];
 
             foreach (class_uses_recursive($class) as $trait) {
-                $method = class_basename($trait) . ucfirst($baseMethod);
+                $method = class_basename($trait).ucfirst($baseMethod);
 
                 if (method_exists($class, $method) && !in_array($method, static::$bootedTraitMethods[$baseMethod])) {
                     static::$bootedTraitMethods[$baseMethod][] = lcfirst($method);

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

src/Urls/UrlRecord.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
             ->first();
 
         if (!$record) {
-            throw new UrlRecordNotFound('No url record found by slug [' . $slug . '] for locale [' . $locale . '].');
+            throw new UrlRecordNotFound('No url record found by slug ['.$slug.'] for locale ['.$locale.'].');
         }
 
         return $record;
@@ -56,7 +56,7 @@ 
             ->first();
 
         if (!$record) {
-            throw new UrlRecordNotFound('No url record found for model [' . $model->getMorphClass() . '@' . $model->id . '] for locale [' . $locale . '].');
+            throw new UrlRecordNotFound('No url record found for model ['.$model->getMorphClass().'@'.$model->id.'] for locale ['.$locale.'].');
         }
 
         return $record;
@@ -150,7 +150,7 @@ 
         }
 
         if ($ignoredModel) {
-            $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
+            $builder->whereNotIn('id', function($query) use ($ignoredModel) {
                 $query->select('id')
                     ->from('chief_urls')
                     ->where('model_type', '=', $ignoredModel->getMorphClass())

src/Urls/MemoizedUrlRecord.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
             ->first();
 
         if (!$record) {
-            throw new UrlRecordNotFound('No url record found for model [' . $model->getMorphClass() . '@' . $model->id . '] for locale [' . $locale . '].');
+            throw new UrlRecordNotFound('No url record found for model ['.$model->getMorphClass().'@'.$model->id.'] for locale ['.$locale.'].');
         }
 
         return $record;
@@ -44,7 +44,7 @@ 
 
     public static function getByModel(Model $model)
     {
-        return chiefMemoize('url-records-get-by-model', function ($model) {
+        return chiefMemoize('url-records-get-by-model', function($model) {
             return parent::getByModel($model);
         }, [$model]);
     }

src/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
                     $targetModel = Morphables::instance($targetUrlRecord->model_type)->find($targetUrlRecord->model_id);
 
                     if (!$targetModel) {
-                        throw new \DomainException('Corrupt target model for this url request. Model by reference [' . $targetUrlRecord->model_type . '@' . $targetUrlRecord->model_id . '] has probably been archived or deleted.');
+                        throw new \DomainException('Corrupt target model for this url request. Model by reference ['.$targetUrlRecord->model_type.'@'.$targetUrlRecord->model_id.'] has probably been archived or deleted.');
                     }
 
                     return static::createRedirect($targetModel->url($locale));
@@ -47,14 +47,14 @@ 
             }
 
             if (!$model) {
-                throw new \DomainException('Corrupt target model for this url request. Model by reference [' . $urlRecord->model_type . '@' . $urlRecord->model_id . '] has probably been archived or deleted.');
+                throw new \DomainException('Corrupt target model for this url request. Model by reference ['.$urlRecord->model_type.'@'.$urlRecord->model_id.'] has probably been archived or deleted.');
             }
 
             if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
                 if (!PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request [' . $slug . '] but it is not published.');
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -65,7 +65,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request [' . $slug . '] for locale [' . $locale . '].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)