Level 53
@rafaelmunoznl do you tried Storage::disk('public')->delete($request->avatar_location);?
1 like
Mar 3, 2020
9
Level 8
Delete files from storage
I am using the storage directory to uploa the avatars of my users.
The structure is like this: storage/app/public/avatars/file.jpg
I am able to upload avatars without problem, but when the user decides to change the avatar, I need of course, to delete the previous avatar. i have been trying to implement the method described here: https://laravel.com/docs/5.8/filesystem#deleting-files.
I did this:
<?php
namespace App\Models\Client\Traits\Method;
...
use Illuminate\Support\Facades\Storage; // <- Call the facade
...
trait ClientMethod
{
public static function getPicture($request, $size = false)
{
Storage::delete($request->avatar_location); //<- here I intend to delete the file in https://mydomain.com//storage/avatars/rasputin-1583231825.png
// more code here
}
}
The file exist in that location. I can download it. Everything looks like the documentation, but it does not get deleted.
Am I missing something?
Level 8
@sti3bas No before, but I tested it now.
It does not work
Level 53
@rafaelmunoznl What is the output of dd($request->avatar_location);?
Level 8
@sti3bas the output is:
https://mydomain.com/storage/avatars/avatar-1583231825.png
the file exist in that position. i can see and download it
Level 53
@rafaelmunoznl why are you storing the full url?
Level 8
@sti3bas In my data base I just store this string:
avata_location = 'avatars/misi1-1583236599.jpg'
The full URL is in the $request variable.
When I click on the submit button I get the $requestvariable like this:
#parameters: array:7 [▼
"_token" => "6pQ4rchK4KrrO2O85T1bse3ZC1FejA0lggWXNp1v"
"_method" => "PUT"
"avatar_type" => "gravatar"
"client_id" => "1"
"email" => "[email protected]"
"avatar_location" => "http://violeta-app.test/storage/avatars/misi1-1583236599.jpg"`// <- url
"company_id" => "2"
]
}
and that is what I want to delete when I do:
Storage::disk('public')->delete($request->avatar_location);
Should I not use the full url to delete whe I use Storage facade?
Level 53
@rafaelmunoznl you should pass the path (avatars/misi1-1583236599.jpg) from the database to delete method and not the full URL.
Level 102
@rafaelmunoznl imagine if you could delete an image on the internet using its url. Google images would be in huge trouble :)
2 likes
Level 8
@sti3bas Oh! Thanks. now it works. Thanks a lot
Please or to participate in this conversation.