thinktomorrow / chief

src/Pages/Page.php

Spacing +7 added lines, -7 removed lines

@@ -101,7 +101,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.'.');
     }
 
     /**
@@ -141,9 +141,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 '';
@@ -154,7 +154,7 @@ 
         $classKey = get_class($this);
         if (property_exists($this, 'labelSingular')) {
             $labelSingular = $this->labelSingular;
-        } else {
+        }else {
             $labelSingular = Str::singular($classKey);
         }
 
@@ -194,7 +194,7 @@ 
             $locale = app()->getLocale();
         }
         try {
-            $memoizedKey = $this->getMorphClass() . '-' . $this->id . '-' . $locale;
+            $memoizedKey = $this->getMorphClass().'-'.$this->id.'-'.$locale;
 
             if (isset(static::$cachedUrls[$memoizedKey])) {
                 return static::$cachedUrls[$memoizedKey];
@@ -250,11 +250,11 @@ 
     public function statusAsLabel()
     {
         if ($this->isPublished()) {
-            return '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>';
+            return '<a href="'.$this->url().'" target="_blank"><em>online</em></a>';
         }
 
         if ($this->isDraft()) {
-            return '<a href="' . $this->url() . '" target="_blank" class="text-error"><em>offline</em></a>';
+            return '<a href="'.$this->url().'" target="_blank" class="text-error"><em>offline</em></a>';
         }
 
         if ($this->isArchived()) {

src/Modules/ModuleManager.php

Spacing +3 added lines, -3 removed lines

@@ -37,10 +37,10 @@ 
          */
         if ($verb == 'index' && $this->model->isPageSpecific()) {
             if (!$this->model->page) {
-                throw new \RuntimeException('Cannot retrieve parent for page specific module [type: ' . $this->registration->key() . ', id: ' . $this->existingModel()->id . ']');
+                throw new \RuntimeException('Cannot retrieve parent for page specific module [type: '.$this->registration->key().', id: '.$this->existingModel()->id.']');
             }
 
-            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 = [
@@ -160,7 +160,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/Management/Application/StoreManager.php

Spacing +2 added lines, -2 removed lines

@@ -40,9 +40,9 @@
         if ($request->filled('template')) {
             $this->applyTemplate->handle(
                 FlatReferenceFactory::fromString($request->input('template'))->className(),
-                (string) FlatReferenceFactory::fromString($request->input('template'))->id(),
+                (string)FlatReferenceFactory::fromString($request->input('template'))->id(),
                 get_class($manager->existingModel()),
-                (string) $manager->existingModel()->id
+                (string)$manager->existingModel()->id
             );
         }
 

src/Templates/ApplyTemplate.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
             return;
         }
 
-        $availableApplicatorsString = implode(array_map(function ($applicator) { return get_class($applicator);}, $this->applicators), ',');
+        $availableApplicatorsString = implode(array_map(function($applicator) { return get_class($applicator); }, $this->applicators), ',');
 
         throw new \RuntimeException("No proper template applicator found. $sourceClassName [$sourceId] cannot be applied as template. Available applicators: [$availableApplicatorsString]");
     }

src/Templates/PageTemplateApplicator.php

Spacing +2 added lines, -2 removed lines

@@ -53,13 +53,13 @@
 
             if ($child instanceof Module && $child->isPageSpecific()) {
                 $duplicatedChild = $child::create([
-                    'slug'           => $targetModel->title ? $targetModel->title . '-' . $child->slug : $child->slug . '-copy',
+                    'slug'           => $targetModel->title ? $targetModel->title.'-'.$child->slug : $child->slug.'-copy',
                     'owner_id'        => $targetModel->id,
                     'owner_type' => $targetModel->getMorphClass()
                 ]);
 
                 $this->moduleTemplateApplicator->handle($child, $duplicatedChild);
-            } else {
+            }else {
                 $duplicatedChild = $child;
             }
 

src/Urls/UrlHelper.php

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
 
     public static function modelsByType(array $types, Model $ignoredModel = null, bool $online = true)
     {
-        $models = chiefMemoize('all-online-models-' . implode('_', $types), function () use ($types, $online) {
+        $models = chiefMemoize('all-online-models-'.implode('_', $types), function() use ($types, $online) {
             $builder = UrlRecord::whereNull('redirect_id')
                 ->select('model_type', 'model_id')
                 ->groupBy('model_type', 'model_id');
@@ -98,11 +98,11 @@ 
                 $builder->whereIn('model_type', $types);
             }
 
-            return $builder->get()->mapToGroups(function ($record) {
+            return $builder->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
-            })->map(function ($record, $key) {
+            })->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->map->reject(function ($model) use ($online) {
+            })->map->reject(function($model) use ($online) {
                 if ($online) {
                     return is_null($model) || !$model->isPublished();
                 } // Invalid references to archived or removed models where url record still exists.
@@ -112,7 +112,7 @@ 
         });
 
         if ($ignoredModel) {
-            $models = $models->reject(function ($model) use ($ignoredModel) {
+            $models = $models->reject(function($model) use ($ignoredModel) {
                 return (get_class($model) === get_class($ignoredModel) && $model->id === $ignoredModel->id);
             });
         }

src/Common/helpers.php

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
         $manifestPath = '/chief-assets/back';
 
         // Manifest expects each entry to start with a leading slash - we make sure to deduplicate the manifest path.
-        $entry = str_replace($manifestPath, '', '/' . ltrim($filepath, '/'));
+        $entry = str_replace($manifestPath, '', '/'.ltrim($filepath, '/'));
 
         try {
             // Paths should be given relative to the manifestpath so make sure to remove the basepath
@@ -28,7 +28,7 @@ 
         } catch (\Exception $e) {
             \Illuminate\Support\Facades\Log::error($e);
 
-            return $manifestPath . $entry;
+            return $manifestPath.$entry;
         }
     }
 }
@@ -116,7 +116,7 @@ 
 
         $teaser = mb_substr($text, 0, $max, 'utf-8');
 
-        return strlen($text) <= $max ? $teaser : $teaser . $ending;
+        return strlen($text) <= $max ? $teaser : $teaser.$ending;
     }
 }
 
@@ -247,16 +247,16 @@ 
             'fragment',
         ], null), $parsed_url, $overrides);
 
-        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'] . '://' : null;
-        $port = $parsed_url['port'] ? ':' . $parsed_url['port'] : null;
-        $fragment = $parsed_url['fragment'] ? '#' . $parsed_url['fragment'] : null;
+        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'].'://' : null;
+        $port = $parsed_url['port'] ? ':'.$parsed_url['port'] : null;
+        $fragment = $parsed_url['fragment'] ? '#'.$parsed_url['fragment'] : null;
 
-        $baseurl = $scheme . $parsed_url['host'] . $port . $parsed_url['path'];
+        $baseurl = $scheme.$parsed_url['host'].$port.$parsed_url['path'];
         $current_query = [];
 
         $_query = explode('&', $parsed_url['query']);
 
-        array_map(function ($v) use (&$current_query) {
+        array_map(function($v) use (&$current_query) {
             if (!$v) {
                 return;
             }
@@ -274,7 +274,7 @@ 
 
         $query = urldecode(http_build_query(array_merge($current_query, $query_params)));
 
-        return $baseurl . '?' . $query . $fragment;
+        return $baseurl.'?'.$query.$fragment;
     }
 }
 

src/Mediagallery/MimetypeIcon.php

Spacing +3 added lines, -3 removed lines

@@ -29,12 +29,12 @@
 
     public function icon(): string
     {
-        if(isset($this->mapping[$this->mimetype])) {
+        if (isset($this->mapping[$this->mimetype])) {
             return $this->mapping[$this->mimetype];
         }
 
-        foreach($this->mapping as $mimetype => $icon) {
-            if(Str::contains($this->mimetype, $mimetype)) {
+        foreach ($this->mapping as $mimetype => $icon) {
+            if (Str::contains($this->mimetype, $mimetype)) {
                 return $icon;
             }
         }

src/Mediagallery/Application/RemovalAction.php

Spacing +6 added lines, -6 removed lines

@@ -22,24 +22,24 @@
     public function handle(Request $request)
     {
         // Strict protection enabled: we won't remove assets who are still being used...
-        $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId){
+        $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId) {
             return DB::table('asset_pivots')
                 ->where('asset_id', $assetId)
                 ->where('unused', 0)
                 ->exists();
         })->toArray();
 
-        if($assetIds)
+        if ($assetIds)
         {
-                foreach($assetIds as $k => $assetId) {
-                    try{
+                foreach ($assetIds as $k => $assetId) {
+                    try {
                         $this->deleteAsset->remove($assetId);
-                    } catch(FileNotAccessibleException $e) {
+                    } catch (FileNotAccessibleException $e) {
                         unset($assetIds[$k]); // So our count of removed assets is correct in the log.
                     }
                 }
 
-                Audit::activity()->log('removed ' . count($assetIds) . ' assets from the mediagallery.');
+                Audit::activity()->log('removed '.count($assetIds).' assets from the mediagallery.');
 
                 return true;
         }