thinktomorrow / chief

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

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());

src/Mediagallery/Http/BulkActionsController.php

Spacing +4 added lines, -4 removed lines

@@ -24,18 +24,18 @@ 
 
     public function bulk(Request $request)
     {
-        if($request->input('type') == 'download') {
+        if ($request->input('type') == 'download') {
             return $this->download($request);
         }
 
-        if($request->input('type') == 'remove') {
+        if ($request->input('type') == 'remove') {
             return $this->remove($request);
         }
     }
 
     private function download(Request $request)
     {
-        $filename = Str::slug(config('app.name')) . '_assets_' . date('YmdHis') . '.zip';
+        $filename = Str::slug(config('app.name')).'_assets_'.date('YmdHis').'.zip';
 
         // TODO: do the response here instead of the zip action itself
         return $this->zipAction->handle($filename, $request);
@@ -45,7 +45,7 @@ 
     {
         $result = $this->removalAction->handle($request);
 
-        if($result) {
+        if ($result) {
             return redirect()->back()->with('messages.success', 'De mediabestanden zijn verwijderd');
         }
 

src/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
                     $targetModel = Morphables::instance($targetUrlRecord->model_type)->find($targetUrlRecord->model_id);
 
                     if (!$targetModel) {
-                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $targetUrlRecord->model_type . '@' . $targetUrlRecord->model_id . '] has probably been archived or deleted.');
+                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$targetUrlRecord->model_type.'@'.$targetUrlRecord->model_id.'] has probably been archived or deleted.');
                     }
 
                     return static::createRedirect($targetModel->url($locale));
@@ -49,14 +49,14 @@ 
             }
 
             if (!$model) {
-                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $urlRecord->model_type . '@' . $urlRecord->model_id . '] has probably been archived or deleted.');
+                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$urlRecord->model_type.'@'.$urlRecord->model_id.'] has probably been archived or deleted.');
             }
 
             if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
                 if (!PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request [' . $slug . '] but it is not published.');
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -67,7 +67,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request [' . $slug . '] for locale [' . $locale . '].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)