thinktomorrow / chief

src/Urls/Application/SaveUrlSlugs.php

Spacing +7 added lines, -7 removed lines

@@ -57,7 +57,7 @@ 
     private function saveRecord(string $locale, ?string $slug)
     {
         // Existing ones for this locale?
-        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function ($record) use ($locale) {
+        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 !$record->isRedirect()
@@ -66,7 +66,7 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
 
@@ -82,7 +82,7 @@ 
         }
 
         // Only replace the existing records that differ from the current passed slugs
-        $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
+        $nonRedirectsWithSameLocale->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug != $slug) {
                 $existingRecord->replaceAndRedirect(['slug' => $slug]);
             }
@@ -127,12 +127,12 @@ 
      */
     private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
     {
-        $existingRecords->filter(function ($record) use ($locale) {
+        $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 $record->isRedirect()
             );
-        })->each(function ($existingRecord) use ($slug) {
+        })->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug == $slug) {
                 $existingRecord->delete();
             }
@@ -147,11 +147,11 @@ 
 
         // The old homepage url should be removed since this is no longer in effect.
         // In case of any redirect to this old homepage, the last used redirect is now back in effect.
-        $existingRecords->reject(function ($existingRecord) {
+        $existingRecords->reject(function($existingRecord) {
             return (
                 $existingRecord->model_type == $this->model->getMorphClass() &&
                 $existingRecord->model_id == $this->model->id);
-        })->each(function ($existingRecord) {
+        })->each(function($existingRecord) {
 
             // TODO: if there is a redirect to this page, we'll take this one as the new url
             $existingRecord->delete();

src/Urls/ProvidesUrl/BaseUrlSegment.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      */
     public static function prepend(ProvidesUrl $model, string $slug, $locale): string
     {
-        $slugWithBaseSegment = $model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $model->baseUrlSegment($locale).'/'.$slug;
         $slugWithBaseSegment = trim($slugWithBaseSegment, '/');
 
         // If slug with base segment is empty string, it means that the passed slug was probably a "/" character.

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/Fields/Types/ImageField.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $images = [];
         $locale = $locale ?? app()->getLocale();
 
-        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function ($asset) use ($locale) {
+        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function($asset) use ($locale) {
             return $asset->pivot->locale == $locale;
         })->sortBy('pivot.order');
 

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