@@ -29,12 +29,12 @@ |
||
29 | 29 | |
30 | 30 | public function icon(): string |
31 | 31 | { |
32 | - if(isset($this->mapping[$this->mimetype])) { |
|
32 | + if (isset($this->mapping[$this->mimetype])) { |
|
33 | 33 | return $this->mapping[$this->mimetype]; |
34 | 34 | } |
35 | 35 | |
36 | - foreach($this->mapping as $mimetype => $icon) { |
|
37 | - if(Str::contains($this->mimetype, $mimetype)) { |
|
36 | + foreach ($this->mapping as $mimetype => $icon) { |
|
37 | + if (Str::contains($this->mimetype, $mimetype)) { |
|
38 | 38 | return $icon; |
39 | 39 | } |
40 | 40 | } |
@@ -22,24 +22,24 @@ |
||
22 | 22 | public function handle(Request $request) |
23 | 23 | { |
24 | 24 | // Strict protection enabled: we won't remove assets who are still being used... |
25 | - $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId){ |
|
25 | + $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId) { |
|
26 | 26 | return DB::table('asset_pivots') |
27 | 27 | ->where('asset_id', $assetId) |
28 | 28 | ->where('unused', 0) |
29 | 29 | ->exists(); |
30 | 30 | })->toArray(); |
31 | 31 | |
32 | - if($assetIds) |
|
32 | + if ($assetIds) |
|
33 | 33 | { |
34 | - foreach($assetIds as $k => $assetId) { |
|
35 | - try{ |
|
34 | + foreach ($assetIds as $k => $assetId) { |
|
35 | + try { |
|
36 | 36 | $this->deleteAsset->remove($assetId); |
37 | - } catch(FileNotAccessibleException $e) { |
|
37 | + } catch (FileNotAccessibleException $e) { |
|
38 | 38 | unset($assetIds[$k]); // So our count of removed assets is correct in the log. |
39 | 39 | } |
40 | 40 | } |
41 | 41 | |
42 | - Audit::activity()->log('removed ' . count($assetIds) . ' assets from the mediagallery.'); |
|
42 | + Audit::activity()->log('removed '.count($assetIds).' assets from the mediagallery.'); |
|
43 | 43 | |
44 | 44 | return true; |
45 | 45 | } |
@@ -17,7 +17,7 @@ discard block |
||
17 | 17 | |
18 | 18 | public function handle(string $filename, Request $request) |
19 | 19 | { |
20 | - $assets = Asset::whereIn('id', (array) $request->input('asset_ids', []))->get(); |
|
20 | + $assets = Asset::whereIn('id', (array)$request->input('asset_ids', []))->get(); |
|
21 | 21 | |
22 | 22 | // enable output of HTTP headers TODO: this should be moved to the controller instead... |
23 | 23 | $options = new Archive(); |
@@ -26,7 +26,7 @@ discard block |
||
26 | 26 | $zip = new ZipStream($filename, $options); |
27 | 27 | |
28 | 28 | $assets->each(function(Asset $asset) use($zip){ |
29 | - if(file_exists($asset->getFirstMediaPath())) { |
|
29 | + if (file_exists($asset->getFirstMediaPath())) { |
|
30 | 30 | $zip->addFileFromPath($asset->filename(), $asset->getFirstMediaPath()); |
31 | 31 | } |
32 | 32 | }); |
@@ -21,13 +21,13 @@ discard block |
||
21 | 21 | $assets = Asset::with('media')->orderBy('created_at', 'DESC')->select('assets.*'); |
22 | 22 | |
23 | 23 | if ($search) { |
24 | - $assets->whereHas('media', function (Builder $query) use ($search) { |
|
25 | - $query->where('file_name', 'LIKE', '%' . $search . '%'); |
|
24 | + $assets->whereHas('media', function(Builder $query) use ($search) { |
|
25 | + $query->where('file_name', 'LIKE', '%'.$search.'%'); |
|
26 | 26 | }); |
27 | 27 | } |
28 | 28 | |
29 | 29 | if ($unused) { |
30 | - $assets->whereNotExists(function ($query) { |
|
30 | + $assets->whereNotExists(function($query) { |
|
31 | 31 | $query->select(DB::raw(1)) |
32 | 32 | ->from('asset_pivots') |
33 | 33 | ->whereRaw('asset_pivots.asset_id = assets.id') |
@@ -35,7 +35,7 @@ discard block |
||
35 | 35 | }); |
36 | 36 | } |
37 | 37 | |
38 | - if($owner){ |
|
38 | + if ($owner) { |
|
39 | 39 | $owner = FlatReferenceFactory::fromString($owner)->instance(); |
40 | 40 | $modelAssets = collect(); |
41 | 41 | $modelAssets = $modelAssets->merge($owner->assets()); |
@@ -24,18 +24,18 @@ discard block |
||
24 | 24 | |
25 | 25 | public function bulk(Request $request) |
26 | 26 | { |
27 | - if($request->input('type') == 'download') { |
|
27 | + if ($request->input('type') == 'download') { |
|
28 | 28 | return $this->download($request); |
29 | 29 | } |
30 | 30 | |
31 | - if($request->input('type') == 'remove') { |
|
31 | + if ($request->input('type') == 'remove') { |
|
32 | 32 | return $this->remove($request); |
33 | 33 | } |
34 | 34 | } |
35 | 35 | |
36 | 36 | private function download(Request $request) |
37 | 37 | { |
38 | - $filename = Str::slug(config('app.name')) . '_assets_' . date('YmdHis') . '.zip'; |
|
38 | + $filename = Str::slug(config('app.name')).'_assets_'.date('YmdHis').'.zip'; |
|
39 | 39 | |
40 | 40 | // TODO: do the response here instead of the zip action itself |
41 | 41 | return $this->zipAction->handle($filename, $request); |
@@ -45,7 +45,7 @@ discard block |
||
45 | 45 | { |
46 | 46 | $result = $this->removalAction->handle($request); |
47 | 47 | |
48 | - if($result) { |
|
48 | + if ($result) { |
|
49 | 49 | return redirect()->back()->with('messages.success', 'De mediabestanden zijn verwijderd'); |
50 | 50 | } |
51 | 51 |