Level 22
U have to delete the file first but u r deleting the database record first
1 like
May 30, 2017
17
Level 3
Storage:: delete() not working
Hi, I am using this methods to delete files
$request = request(['id', 'path']);
$id = $request['id'];
$path = $request['path'];
File::find($id)->delete();
Storage::delete(public_path($path));
I can check the $path correctness by using command line to test the past. But Storage::delete() does do work.
Could someone please help? And what should I do when a file's name is with spaces? How to return Storage errors so I know what's is happening?
Level 3
@nagavinod424 It is still not working
Level 22
I think the path specified is wrong
public_path gives the public folder path
$path should be absolute path from the public folder
then the file will be deleted surely.....
ex : $path = 'uploads\file_name'
Level 122
You are going to let anyone delete anything?
1 like
Level 1
@Snapey Hello ....What s Your means?its unsave?
Level 102
@hadihasanpur I can just call this endpoint again and again with different paths, until i have everything deleted. I could start with path = 'index.php' to bring down your site :)
2 likes
Level 3
I tried both options but it did not works. test with command line from public folder ok.
@Snapey: I will need do more for authorization. Should not pass the pass via request but I have no idea why the delete() is not working at all. I wish It can give some errors.
thanks
Level 50
@ThanhPv I think you over complicating things here. Your delete method should look like this:
Storage::disk('s3')->delete('$path' . $file_path);
The order in which you delete does not matter here since you declare you variables before you delete the record from the DB and you do not actually take them from database at all but from a request.
Hardcode it. See it it works and work backwards and you will find where the problem is. Storage should return true on success.
2 likes
Level 3
Wow php unlink($filename) works so perfectly.
Many thanks
1 like
Level 8
I have the same issue. I tried it in "php artisan tinker" with the exact same path and it works. But not via browser. I think this is permission issue.
Is there anyway to do it in laravel's way? Because I don't want to use unlink($filename).
Level 75
Is there anyway to do it in laravel's way?
Laravel uses symfony components which in turn users php unlink.
Level 12
@kwarnkham use $deleted=Storage::disk('public')->delete($media->filename); with disk('public') i solved the issue
11 likes
Level 1
thank you that solved it!
Level 1
I used this method and it worked File :: delete ('image /'. $ Avatar-> avatar); don't forget to Use File;
1 like
Level 4
// path to my image file
$filename = public_path() . '/images/' . $property->featured_image;
// then delete the file from the server with:
unlink( $filename );
Please or to participate in this conversation.