thinktomorrow / chief

src/Site/Sitemap/SitemapXml.php

Spacing +4 added lines, -4 removed lines

@@ -76,11 +76,11 @@ 
         $models = UrlRecord::allOnlineModels($locale);
 
         $this->urls = $models
-            ->reject(function (ProvidesUrl $model) use ($locale) {
+            ->reject(function(ProvidesUrl $model) use ($locale) {
                 // In case the url is not found or present for given locale.
                 return !$model->url($locale);
             })
-            ->map(function (ProvidesUrl $model) use ($locale, $alternateLocales) {
+            ->map(function(ProvidesUrl $model) use ($locale, $alternateLocales) {
                 $url = $model->url($locale);
 
                 $alternateUrls = [];
@@ -101,12 +101,12 @@ 
     {
         $pool = new Pool($this->httpClient, $this->crawlableUrlGenerator(), [
             'concurrency' => 5,
-            'fulfilled' => function (Response $response, $index) {
+            'fulfilled' => function(Response $response, $index) {
                 if ($response->getStatusCode() !== \Symfony\Component\HttpFoundation\Response::HTTP_OK) {
                     unset($this->urls[$index]);
                 }
             },
-            'rejected' => function (RequestException $reason, $index) {
+            'rejected' => function(RequestException $reason, $index) {
                 unset($this->urls[$index]);
             },
         ]);

src/Site/Sitemap/SitemapFiles.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     {
         $files = $this->filesystem->files($directory);
 
-        return collect($files)->filter(function ($file) {
+        return collect($files)->filter(function($file) {
             return (Str::startsWith($file->getFileName(), 'sitemap-') && Str::endsWith($file->getFileName(), '.xml'));
         });
     }

src/Site/Urls/Controllers/RemoveRedirectController.php

Spacing +3 added lines, -3 removed lines

@@ -18,11 +18,11 @@
         $urlRecord = UrlRecord::find($id);
 
         if (!$urlRecord) {
-            return response()->json(['No url record found by id ' . $id], 500);
+            return response()->json(['No url record found by id '.$id], 500);
         }
 
-        if (! $urlRecord->isRedirect()) {
-            return response()->json(['Url with id ' . $id . ' is not a redirect'], 500);
+        if (!$urlRecord->isRedirect()) {
+            return response()->json(['Url with id '.$id.' is not a redirect'], 500);
         }
 
         $this->pointChildRedirectsToParent($urlRecord, $urlRecord->redirect_id);

src/Site/Urls/Controllers/LinksController.php

Spacing +4 added lines, -4 removed lines

@@ -20,19 +20,19 @@
     public function update(Request $request)
     {
         /** @var ProvidesUrl|Model $model */
-        $model = (new ModelReference($request->modelClass, (int) $request->modelId))->instance();
+        $model = (new ModelReference($request->modelClass, (int)$request->modelId))->instance();
 
         $this->validate($request, ['links' => [
-            'array', 'min:1', new UniqueUrlSlugRule($model, $model),], [], ['links.*' => 'taalspecifieke link']
+            'array', 'min:1', new UniqueUrlSlugRule($model, $model), ], [], ['links.*' => 'taalspecifieke link']
         ]);
 
         (new SaveUrlSlugs($model))->handle($request->input('links', []));
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($model)->reject(function ($record) {
+        UrlRecord::getByModel($model)->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
 

src/Site/Urls/Controllers/CheckLinkController.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     public function check(Request $request)
     {
         /** @var ProvidesUrl|Model $model */
-        $model = (new ModelReference($request->modelClass, (int) $request->modelId))->instance();
+        $model = (new ModelReference($request->modelClass, (int)$request->modelId))->instance();
 
         // Trim slashes if any
         $slug = ($request->slug !== '/') ? trim($request->slug, '/') : $request->slug;
@@ -40,7 +40,7 @@ 
         if ($urlRecord->isRedirect()) {
             return 'Deze link bestaat reeds als redirect. Deze redirect zal bijgevolg worden verwijderd.';
         }
-        return 'Deze link bestaat reeds. Kies een andere of <a target="_blank" href="' . $this->editRouteOfOtherModel($urlRecord) . '">pas de andere pagina aan</a>.';
+        return 'Deze link bestaat reeds. Kies een andere of <a target="_blank" href="'.$this->editRouteOfOtherModel($urlRecord).'">pas de andere pagina aan</a>.';
     }
 
     private function editRouteOfOtherModel(UrlRecord $urlRecord): string

src/Site/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/Site/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/Site/Urls/Field/UrlSlugFields.php

Spacing +9 added lines, -9 removed lines

@@ -18,14 +18,14 @@ 
 
     public static function redirectsFromModel(ProvidesUrl $model)
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return !$record->isRedirect();
         })->sortByDesc('created_at');
 
         $fields = new static([]);
 
         foreach ($records as $record) {
-            $key = 'redirects-' . $record->locale . '-' . $record->slug;
+            $key = 'redirects-'.$record->locale.'-'.$record->slug;
             $fields[$key] = UrlSlugField::make($key)
                 ->setUrlRecord($record)
                 ->setFullUrl($model->resolveUrl($record->locale, $record->slug));
@@ -60,10 +60,10 @@ 
         $fields = new static([]);
 
         foreach ($locales as $locale) {
-            $fields['url-slugs.' . $locale] = UrlSlugField::make('url-slugs.' . $locale)
+            $fields['url-slugs.'.$locale] = UrlSlugField::make('url-slugs.'.$locale)
                 ->setBaseUrlSegment($model->baseUrlSegment($locale))
-                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)) . '/')
-                ->name('url-slugs[' . $locale . ']')
+                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)).'/')
+                ->name('url-slugs['.$locale.']')
                 ->label($locale);
         }
 
@@ -76,19 +76,19 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = UrlRecord::getByModel($model)->reject(function ($record) {
+        $records = UrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
         foreach ($records as $record) {
-            if (!isset($fields['url-slugs.' . $record->locale])) {
+            if (!isset($fields['url-slugs.'.$record->locale])) {
                 continue;
             }
 
-            $fields['url-slugs.' . $record->locale]
+            $fields['url-slugs.'.$record->locale]
                 ->setUrlRecord($record)
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) . '/');
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
         }
     }
 }

src/Site/Urls/Field/Url.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Site\Urls\Field;
 
@@ -19,9 +19,9 @@ 
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($model)->reject(function ($record) {
+        UrlRecord::getByModel($model)->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
     }