Level 102
I would not assume this gives you the correct path
$path = ('avatars/'.$user->username);
where is the file placed?
Storage::disk('public')->delete('avatars/'.$user->username);
Like this perhaps ?
1 like
Sep 22, 2022
8
Level 4
Remove Old Picture or file uploaded using unlink
I have a file upload on my website and i am trying to removed the previous picture when i upload a new one.
I have this in my controller
$user = User::find(auth()->user()->id);
$path = ('avatars/'.$user->username);
if(file_exists($path)){
@unlink($path);
}
$user->update([
'avatar' => $this->avatar->store($path, 'wasabi'),
]);
How do i do this please?
Level 4
@Sinnbeck So i did this
$user = User::find(auth()->user()->id);
$path = ('avatars/'.$user->username);
$oldPath = ('https://s3.eu-west-1.wasabisys.com/sample-bucket/'.$user->avatar);
// dd($oldPath);
if(Storage::disk('wasabi')->exists($oldPath)){
Storage::disk('wasabi')->delete($oldPath);
}
$user->update([
'avatar' => $this->avatar->store($path, 'wasabi'),
]);
But still it doesn't work
Level 104
@monstajamss out of interest, what do you think auth()->user() gives you? An Eloquent query is unnecessary!
Level 4
@tykus so basically i am trying to get the old image from the database, check if it exists . Then remove old one and update it with new one
Level 104
@monstajamss sure, store it using the correct disk
Level 4
@tykus I am storing it inside the correct disk. it is just unlinking it
Level 122
If you use storage::disk you need to provide the path relative to the bucket specified by that disk, and not the whole URL..
if(Storage::disk('wasabi')->exists($user->avatar)){
Storage::disk('wasabi')->delete($user->avatar);
}
1 like
Level 4
@Snapey That worked, Cheers :)
Please or to participate in this conversation.