thinktomorrow / chief

src/Modules/Module.php

Spacing +2 added lines, -2 removed lines

@@ -63,11 +63,11 @@
 
     public static function managedModelKey(): string
     {
-        if(isset(static::$managedModelKey)){
+        if (isset(static::$managedModelKey)) {
             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.'.');
     }
 
     /**

src/Pages/Page.php

Spacing +4 added lines, -4 removed lines

@@ -80,11 +80,11 @@ 
 
     public static function managedModelKey(): string
     {
-        if(isset(static::$managedModelKey)){
+        if (isset(static::$managedModelKey)) {
             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.'.');
     }
 
     /**
@@ -124,9 +124,9 @@ 
     public function flatReferenceLabel(): string
     {
         if ($this->exists) {
-            $status = ! $this->isPublished() ? ' [' . $this->statusAsPlainLabel().']' : null;
+            $status = !$this->isPublished() ? ' ['.$this->statusAsPlainLabel().']' : null;
 
-            return $this->title ? $this->title . $status : '';
+            return $this->title ? $this->title.$status : '';
         }
 
         return '';

src/Common/Helpers/Memoize.php

Spacing +2 added lines, -2 removed lines

@@ -35,8 +35,8 @@
 
     private function convertToCachableParameters(array $parameters)
     {
-        foreach($parameters as $key => $value) {
-            if($value instanceof Model) $parameters[$key] = get_class($value) . '@' . $value->id;
+        foreach ($parameters as $key => $value) {
+            if ($value instanceof Model) $parameters[$key] = get_class($value).'@'.$value->id;
         }
 
         return $parameters;

Braces +3 added lines, -1 removed lines

@@ -36,7 +36,9 @@
     private function convertToCachableParameters(array $parameters)
     {
         foreach($parameters as $key => $value) {
-            if($value instanceof Model) $parameters[$key] = get_class($value) . '@' . $value->id;
+            if($value instanceof Model) {
+                $parameters[$key] = get_class($value) . '@' . $value->id;
+            }
         }
 
         return $parameters;

src/Fields/Types/Field.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Fields\Types;
 
@@ -51,7 +51,7 @@ 
 
     public function name(string $name = null)
     {
-        if(!is_null($name)) {
+        if (!is_null($name)) {
             $this->values['name'] = $name;
 
             return $this;
@@ -86,7 +86,7 @@ 
     {
         $name = $this->name();
 
-        if(strpos($name, ':locale')) {
+        if (strpos($name, ':locale')) {
             return preg_replace('#(:locale)#', $locale, $name);
         }
 
@@ -175,7 +175,7 @@ 
         }
 
         if (!in_array($name, ['label', 'key', 'description', 'column', 'name', 'prepend', 'append'])) {
-            throw new \InvalidArgumentException('Cannot set value by ['. $name .'].');
+            throw new \InvalidArgumentException('Cannot set value by ['.$name.'].');
         }
 
         $this->values[$name] = $arguments[0];

src/Settings/Settings.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
         if (is_array($this->items[$key])) {
 
-            if(!$locale) $locale = app()->getLocale();
+            if (!$locale) $locale = app()->getLocale();
 
             if ($this->items[$key] == null || !isset($this->items[$key][$locale])) {
                 return $default;

Braces +3 added lines, -1 removed lines

@@ -22,7 +22,9 @@
 
         if (is_array($this->items[$key])) {
 
-            if(!$locale) $locale = app()->getLocale();
+            if(!$locale) {
+                $locale = app()->getLocale();
+            }
 
             if ($this->items[$key] == null || !isset($this->items[$key][$locale])) {
                 return $default;

src/Urls/UrlRecord.php

Indentation +5 added lines, -5 removed lines

@@ -65,8 +65,8 @@ 
     public static function getByModel(Model $model)
     {
         return static::where('model_type', $model->getMorphClass())
-                     ->where('model_id', $model->id)
-                     ->get();
+                        ->where('model_id', $model->id)
+                        ->get();
     }
 
     public static function findRecentRedirect(Model $model, string $locale): ?self
@@ -134,9 +134,9 @@ 
         if ($ignoredModel) {
             $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
                 $query->select('id')
-                      ->from('chief_urls')
-                      ->where('model_type', '=', $ignoredModel->getMorphClass())
-                      ->where('model_id', '=', $ignoredModel->id);
+                        ->from('chief_urls')
+                        ->where('model_type', '=', $ignoredModel->getMorphClass())
+                        ->where('model_id', '=', $ignoredModel->id);
             });
         }
 

Spacing +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
         return static::where('model_type', $model->getMorphClass())
             ->where('model_id', $model->id)
             ->where('locale', $locale)
-            ->where('redirect_id','<>', null)
+            ->where('redirect_id', '<>', null)
             ->orderBy('redirect_id', 'ASC')
             ->first();
     }
@@ -127,12 +127,12 @@ 
             $builder->where('locale', $locale);
         }
 
-        if (! $includeRedirects) {
+        if (!$includeRedirects) {
             $builder->whereNull('redirect_id');
         }
 
         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/Application/SaveUrlSlugs.php

Spacing +9 added lines, -9 removed lines

@@ -56,7 +56,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()
@@ -65,7 +65,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();
             });
 
@@ -81,7 +81,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]);
             }
@@ -126,12 +126,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();
             }
@@ -140,15 +140,15 @@ 
 
     private function deleteIdenticalRecords($existingRecords): void
     {
-        if($this->strict) return;
+        if ($this->strict) return;
 
         // 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();
@@ -162,7 +162,7 @@ 
      */
     private function prependBaseUrlSegment(string $slug, $locale): string
     {
-        $slugWithBaseSegment = $this->model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $this->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.

Braces +3 added lines, -1 removed lines

@@ -140,7 +140,9 @@
 
     private function deleteIdenticalRecords($existingRecords): void
     {
-        if($this->strict) return;
+        if($this->strict) {
+            return;
+        }
 
         // 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.

src/Urls/Application/RevertUrlSlug.php

Spacing +3 added lines, -3 removed lines

@@ -23,10 +23,10 @@ 
 
     public function handle(string $locale): void
     {
-        try{
+        try {
             $currentUrlRecord = UrlRecord::findByModel($this->model, $locale);
 
-            if($recentRedirect = UrlRecord::findRecentRedirect($this->model, $locale)){
+            if ($recentRedirect = UrlRecord::findRecentRedirect($this->model, $locale)) {
 
                 $recentRedirectSlug = $recentRedirect->slug;
                 $recentRedirect->delete();
@@ -34,7 +34,7 @@ 
                 $currentUrlRecord->replaceAndRedirect(['slug' => $recentRedirectSlug]);
             }
         }
-        catch(UrlRecordNotFound $e)
+        catch (UrlRecordNotFound $e)
         {
             // No url present so nothing to do here...
         }

Braces +1 added lines, -2 removed lines

@@ -33,8 +33,7 @@
 
                 $currentUrlRecord->replaceAndRedirect(['slug' => $recentRedirectSlug]);
             }
-        }
-        catch(UrlRecordNotFound $e)
+        } catch(UrlRecordNotFound $e)
         {
             // No url present so nothing to do here...
         }

src/Urls/UrlHelper.php

Spacing +4 added lines, -4 removed lines

@@ -15,11 +15,11 @@ 
      */
     public static function allOnlineModels(bool $onlySingles = false): array
     {
-        return chiefMemoize('all-online-models', function () use($onlySingles) {
+        return chiefMemoize('all-online-models', function() use($onlySingles) {
 
             $builder = UrlRecord::whereNull('redirect_id')->select('model_type', 'model_id')->groupBy('model_type', 'model_id');
 
-            if($onlySingles) {
+            if ($onlySingles) {
                 $builder->where('model_type', 'singles');
             }
 
@@ -28,9 +28,9 @@ 
             });
 
             // Get model for each of these records...
-            $models = $liveUrlRecords->map(function($record, $key){
+            $models = $liveUrlRecords->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->each->reject(function ($model) {
+            })->each->reject(function($model) {
                 // Invalid references to archived or removed models where url record still exists.
                 return is_null($model);
             })->flatten();