thinktomorrow / chief

src/Admin/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/Admin/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/Admin/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 = ModelReference::fromString($owner)->instance();
             $modelAssets = collect();
             $modelAssets = $modelAssets->merge($owner->assets());

src/Admin/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/Admin/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/Admin/Nav/Nav.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
 
     public function untagged(): self
     {
-        $filteredItems = array_filter($this->items, function(NavItem $navItem){
+        $filteredItems = array_filter($this->items, function(NavItem $navItem) {
             return $navItem->isUntagged();
         });
 
@@ -41,7 +41,7 @@ 
     {
         $instance = new static();
 
-        foreach($navItems as $item) {
+        foreach ($navItems as $item) {
             $instance->add($item);
         }
 

src/Admin/Nav/NavItem.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 
     public function isTagged($tags): bool
     {
-        return count(array_intersect($this->tags, (array) $tags)) > 0;
+        return count(array_intersect($this->tags, (array)$tags)) > 0;
     }
 
     public function isUntagged(): bool
@@ -41,7 +41,7 @@ 
 
     public function render(): string
     {
-        $output = '<a class="' . (isActiveUrl($this->url()) ? 'active' : '') . '" href="' . $this->url() . '">';
+        $output = '<a class="'.(isActiveUrl($this->url()) ? 'active' : '').'" href="'.$this->url().'">';
         $output .= ucfirst($this->label());
         $output .= '</a>';
 

src/Admin/Settings/Application/ChangeHomepage.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     {
         foreach ($modelReferences as $locale => $modelReferenceString) {
             if (!$modelReferenceString) {
-                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale ' . $locale . ' is missing.');
+                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale '.$locale.' is missing.');
             }
         }
     }

src/Admin/Settings/SettingFieldsManager.php

Spacing +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
 
     public function editFields(): Fields
     {
-        return $this->fields()->map(function (Field $field) {
-            return $field->valueResolver(function ($model = null, $locale = null, $field) {
+        return $this->fields()->map(function(Field $field) {
+            return $field->valueResolver(function($model = null, $locale = null, $field) {
                 return $this->settings->get($field->getKey(), $locale);
             });
         });