LaravelでS3に保存したが画像が表示されない

Question

###前提・実現したいこと
LaravelでS3に画像を保存し、その画像を表示させたいのですが、表示されなくて困っています。
S3確認すると、task/に保存されており、中には10.jpg、11.jpgのようにid + .jpgの形式で保存されています。

###該当のソースコード
index.blade.php

<form id="expire-form" method="post" action="{{ route('post_stock') }}" enctype="multipart/form-data">
  @csrf
  <div class="upload-button">ファイルを選択する
    <input type="file" name="image" onchange="uv.style.display='inline-block'; uv.value = this.value;">
    <input type="text" id="uv" class="upload-value" disabled>
  </div>
  <div class="group">
    <label for="name">商品名</label>
    <input id="name" type="text" name="product">
  </div>
</form>

list.blade.php

@foreach($stock_cosmetics as $stock_cosmetic)
  <div class="stock-items">
     @if($stock_cosmetic->image === null)
       <img src="{{ asset('img/no-image.jpg') }}" alt="no-image">
     @else
       <img src="{{ Storage::disk('s3')->url("stock/{$stock_cosmetic->id}.jpg") }}">
     @endif
     <div class="stock-items-text">
        <p>{{ $stock_cosmetic->product  }}</p>    
     </div>
　　　<a href="{{ route('show_stock', ['id' => $stock_cosmetic->id]) }}">詳細/編集</a> 
  </div>
@endforeach

StockController

public function index(Request $request)
    {
        $user_id = Auth::id();
        $stocks = StockCosmetic::where('user_id', $user_id);
        $stocks->select('image', 'id', 'product',　'created_at')->orderBy('created_at', 'desc');
        $stock_cosmetics = $stocks->paginate(15);

        return view('stock_cosmetics.list_of_stock', ['user_id' => $user_id, 'stock_cosmetics' => $stock_cosmetics]);
    }


public function store(Stock $request)
    {
        $stock_cosmetic = new StockCosmetic();

        $stock_cosmetic->color = $request->input('product');
        $stock_cosmetic->user_id = Auth::id();
     
        $stock_cosmetic->save();

        // image
        $request->image->storePubliclyAs('/task', $stock_cosmetic->id . '.jpg', ['disk' => 's3']);

　　　　　return redirect('stock_cosmetics/list_of_stock/{user_id}/stock_cosmetics');
　　}

web.php

Route::get('/list_of_stock/{user_id}/stock_cosmetics/', 'StockCosmeticController@index')->name('list_of_stock');
Route::post('/post_stock', 'StockCosmeticController@store')->name('post_stock');

###補足情報（FW/ツールのバージョンなど）
-各バージョン
-Laravel 6.x
-PHP 7.4.9

恐らくsrcの指定方法が誤っているのではないかと思いますが、解決方法がわからない状態です。
どなたかご教授いただけるとありがたいです。

Answer 1

S3確認すると、task/に保存されており

と書かれていますので、以下のようにする必要があるのではないでしょうか。

  <img src="{{ Storage::disk('s3')->url("task/{$stock_cosmetic->id}.jpg") }}">

（追記）

また、StockControllerクラスのstoreメソッドでも、S3でのパスをモデルのimageプロパティに保存する必要がありそうです。

    $stock_cosmetic = new StockCosmetic();

    $stock_cosmetic->color = $request->input('product');
    $stock_cosmetic->user_id = Auth::id();

    $stock_cosmetic->save();

    if ($request->hasFile('image') && $request->file('image')->isValid()) {
        // ファイルが存在し、アップロードに問題がなかった場合
        $path = $request->image->storePubliclyAs('/task', $stock_cosmetic->id . '.jpg', ['disk' => 's3']);
        $stock_cosmetic->image = $path;
        $stock_cosmetic->save();
    }

参考: ファイル

Comments

[aki0201]

訂正したのですが、やはり表示されないようです。

[Lulucom]

訂正した状態で、HTMLソースを確認すると、そのimgタグのsrcのURLは正しそうですか？
また、ブラウザからそのURLに直接アクセスするとどのように表示されますか？

[aki0201]

ブラウザからURLを確認するために、@if($stock_cosmetic->image === null)の箇所を削除すると、画像が反映されたので、こちらに原因があるようです。
画像の投稿がなければ、ダミー画像のようなものを表示するようにしたいのですが、どのようにすればいいのでしょうか。

[aki0201]

controllerに追記のコードを追加したのですが、やはり画像が表示されないようでした。また、S３を確認したところ、今まで「10.jpg」のように登録されていたのですが、「.jpg」とidが入らなくなってしまったようです。

[Lulucom]

> idが入らなくなってしまった
確かにそうですね、回答を修正します。

[aki0201]

画像の保存と表示はできるようになりました！
ただ、何も画像をアップロードしなかった時に、Call to a member function isValid() on null
というエラーが出てしまうようです。

[aki0201]

無事、登録できました！
ありがとうございます！

[Lulucom]

よかったです。