thinktomorrow / chief

src/System/HealthMonitor/Checks/HomepageSetCheck.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     public function message(): string
     {
-        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="' . route('chief.back.settings.edit') . '" class="text-secondary-800 underline hover:text-white">Settings</a>';
+        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="'.route('chief.back.settings.edit').'" class="text-secondary-800 underline hover:text-white">Settings</a>';
     }
 
     public function notifiers(): array

src/System/HealthMonitor/Monitor.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
                 }
 
                 return;
-            } else {
+            }else {
                 foreach ($notifiers as $notifier) {
                     app($notifier)->onSuccess($checkInstance);
                 }

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/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;
         }

src/Mediagallery/Application/ZipAction.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 
     public function handle(string $filename, Request $request)
     {
-        $assets = Asset::whereIn('id', (array) $request->input('asset_ids', []))->get();
+        $assets = Asset::whereIn('id', (array)$request->input('asset_ids', []))->get();
 
         // enable output of HTTP headers TODO: this should be moved to the controller instead...
         $options = new Archive();
@@ -26,7 +26,7 @@ 
         $zip = new ZipStream($filename, $options);
 
         $assets->each(function(Asset $asset) use($zip){
-            if(file_exists($asset->getFirstMediaPath())) {
+            if (file_exists($asset->getFirstMediaPath())) {
                 $zip->addFileFromPath($asset->filename(), $asset->getFirstMediaPath());
             }
         });

src/Mediagallery/Http/MediagalleryController.php

Spacing +4 added lines, -4 removed lines

@@ -21,13 +21,13 @@ 
         $assets = Asset::with('media')->orderBy('created_at', 'DESC')->select('assets.*');
 
         if ($search) {
-            $assets->whereHas('media', function (Builder $query) use ($search) {
-                $query->where('file_name', 'LIKE', '%' . $search . '%');
+            $assets->whereHas('media', function(Builder $query) use ($search) {
+                $query->where('file_name', 'LIKE', '%'.$search.'%');
             });
         }
 
         if ($unused) {
-            $assets->whereNotExists(function ($query) {
+            $assets->whereNotExists(function($query) {
                 $query->select(DB::raw(1))
                     ->from('asset_pivots')
                     ->whereRaw('asset_pivots.asset_id = assets.id')
@@ -35,7 +35,7 @@ 
             });
         }
 
-        if($owner){
+        if ($owner) {
             $owner = FlatReferenceFactory::fromString($owner)->instance();
             $modelAssets = collect();
             $modelAssets = $modelAssets->merge($owner->assets());