【laravel】プロフィール更新時に古い画像をStorage::deleteを用いて削除したい

Question

前提・実現したいこと

プロフィール更新時にStorageにある古い画像を削除し、新しく登録された画像を
プロフィール画像としてStorageに保存する、ということがやりたいことです。

DBへの登録と、新しい画像のStorageへの保存まではできているのですが、
旧画像が削除されず、ファイル内に溜まっている状況です。

発生している問題・エラーメッセージ

Controller

public function update(Request $request, User $user)
    {

        $data = $request->all();

        $validator = Validator::make($data,[
            'screen_name' => ['required','string','max:50',Rule::unique('users')->ignore($user->id)],
            'name' => ['required','string','max:225'],
            'profile_image' =>['file','image','mimes:jpeg,png,jpg','max:2048'],
            'email' =>['required','string','email','max:255',Rule::unique('users')->ignore($user->id)]
        ]);

        $validator->validate();
        $user->updateProfile($data);　//ここで画像の削除、保存を行う

        return redirect('show/'.$user->id);
    }

User.php

use Illuminate\Support\Facades\Storage;

class User extends Authenticatable
{
    public function updateProfile(Array $params){
　　　　　//新しいプロフィール画像があれば画像の削除保存をする
        if(isset($params['profile_image'])){
            $delItem = self::find($this->id);
            $deleteFileName = $delItem->profile_image;
            $file_private = $this->name;

            //すでにある画像のパスを指定して削除、新画像を保存
            Storage::disk('public')->delete("public/profile_image/${file_private}".'/'."${deleteFileName}");
            $file_name = $params['profile_image']->store("public/profile_image/${file_private}");

　　　　　　　//DB更新
            $this::where('id',$this->id)->update([
                'screen_name' => $params['screen_name'],
                'name' => $params['name'],
                'profile_image' =>basename($file_name),
                'email' => $params['email'],
            ]);
        }else{
　　　　　　　//新しいプロフィール画像がなければ、DB更新のみ
            $this::where('id',$this->id)->update([
                'screen_name' => $params['screen_name'],
                'name' => $params['name'],
                'email' => $params['email'],
            ]);
        }

        return;
    }
}

試したこと

ドキュメントを参照しましたが、文法的な間違いはないように思います。
シンボリックリンクは設定済みです。
https://readouble.com/laravel/8.x/ja/filesystem.html

TEST5(ユーザー名）のファイルに画像が複数ある状態です

補足情報（FW/ツールのバージョンなど）

laravel8
MySQL5.7.24

Answer 1

ファイルアップロードに使ってるstore()時の、【root】はstorage_path('app')です。

このコード

$params['profile_image']->store("public/profile_image/${file_private}");

の、ディレクトリパスは、
【root】 + 指定パス = > storage_path('app') + "public/profile_image/${file_private}"
ということになるので、

/storage/app/public/profile_image/${file_private}/に、$params['profile_image']が保存されているはず。。。

一方、Storage::disk('public')の、【root】はstorage_path('app/public')です。

このコード

Storage::disk('public')->delete("public/profile_image/${file_private}".'/'."${deleteFileName}");

の、ファイルパスは、
【root】 + 指定パス = > storage_path('app/public') + "public/profile_image/${file_private}/".'/'."${deleteFileName}"
ということになるので、

/storage/app/public/public/profile_image/${file_private}/${deleteFileName}
を、削除しようとしているはず。。

---

登場する【root】は、config/filesystems.phpの設定次第です。
もしこれが初期設定のままなのであれば、
上記の通り「保存パス」と「削除パス」で違うものを指定していることになりますので、
これが、削除できない原因である可能性が高そうです。

Comments

[asahiko123]

回答ありがとうございます。
ご指摘のとおり、パスを修正したところ上手く動くようになりました。
(filesystems.phpは初期設定)

下記ドキュメントには「ディスクの指定」の箇所で
「デフォルトでは、このアップロード済みファイルのstoreメソッドはデフォルトのディスクを使用します。」とあるんですが、この「デフォルトのディスク」が「filesystems.php」のlocalドライバのことで、ここで設定されたrootがstoreメソッドに参照されているという理解で正しかったでしょうか？
https://readouble.com/laravel/8.x/ja/filesystem.html

[mix-peach]

デフォルトのディスクも、filesystems.php　で設定されています。
↓こんなのが、先頭にいるはず。。

'default' => env('FILESYSTEM_DRIVER', 'local'),

こちらも初期設定から変えていなければ上記のようになっているので、
【デフォルトのディスク】は、localドライバとなります。

[asahiko123]

わかりました。ありがとうございます。