thinktomorrow / chief

src/Modules/ModuleManager.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
          * Therefore the index of these modules is at the modules tab of this page model.
          */
         if ($verb == 'index' && $this->model->isPageSpecific()) {
-            return app(Managers::class)->findByModel($this->model->page)->route('edit') . '#eigen-modules';
+            return app(Managers::class)->findByModel($this->model->page)->route('edit').'#eigen-modules';
         }
 
         $routes = [
@@ -153,7 +153,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/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/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/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/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/Relations/Relation.php

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) {
+        return $relations->map(function(Relation $relation) {
             $parent = $relation->parent;
             $parent->relation = $relation;
 
@@ -67,14 +67,14 @@ 
             ->orderBy('sort', 'ASC')
             ->get();
 
-        return $relations->map(function (Relation $relation) use ($parent_type, $parent_id) {
+        return $relations->map(function(Relation $relation) use ($parent_type, $parent_id) {
 
             // It could be that the child itself is soft-deleted, if this is the case, we will ignore it and move on.
             if (!$child = $relation->child) {
                 if (!$relation->child()->withTrashed()->first()) {
 //                if ((!method_exists($childInstance, 'trashed')) || ! $childInstance->onlyTrashed()->find($relation->child_id)) {
                     // If we cannot retrieve it then he collection type is possibly off, this is a database inconsistency and should be addressed
-                    throw new \DomainException('Corrupt relation reference. Related child [' . $relation->child_type . '@' . $relation->child_id . '] could not be retrieved for parent [' . $parent_type . '@' . $parent_id . ']. Make sure the morph key can resolve to a valid class.');
+                    throw new \DomainException('Corrupt relation reference. Related child ['.$relation->child_type.'@'.$relation->child_id.'] could not be retrieved for parent ['.$parent_type.'@'.$parent_id.']. Make sure the morph key can resolve to a valid class.');
                 }
 
                 return null;
@@ -86,7 +86,7 @@ 
         })
 
             // In case of soft-deleted entries, this will be null and should be ignored. We make sure that keys are reset in case of removed child
-            ->reject(function ($child) {
+            ->reject(function($child) {
                 return is_null($child);
             })
             ->values();
@@ -103,10 +103,10 @@ 
 
     public static function deleteRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                 ->where('parent_id', $id);
-        })->orWhere(function ($query) use ($type, $id) {
+        })->orWhere(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();
@@ -118,7 +118,7 @@ 
 
     public static function deleteAllChildRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('parent_type', $type)
                 ->where('parent_id', $id);
         })->get();
@@ -130,7 +130,7 @@ 
 
     public static function deleteAllParentRelationsOf($type, $id)
     {
-        $relations = static::where(function ($query) use ($type, $id) {
+        $relations = static::where(function($query) use ($type, $id) {
             return $query->where('child_type', $type)
                 ->where('child_id', $id);
         })->get();