STEMMechanics/Website
1
0
Fork 0
Code Issues 4 Pull Requests 1 Actions Packages Projects Releases 3 Wiki Activity

media controller should not directly delete files

Browse Source
This commit is contained in:
James Collins
2023-05-20 17:56:48 +10:00
parent b2004e3483
commit 3ed2aadc34
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/Http/Controllers/Api/MediaController.php
View File

@@ -165,10 +165,6 @@ class MediaController extends ApiController
public function destroy(Media $medium) public function destroy(Media $medium)
{ {
if (MediaConductor::destroyable($medium) === true) { if (MediaConductor::destroyable($medium) === true) {
if (file_exists($medium->path()) === true) {
unlink($medium->path());
}
$medium->delete(); $medium->delete();
return $this->respondNoContent(); return $this->respondNoContent();
} }