フォームで画像を登録したところDB上でNULLになってしまう。それを画像が保存されるようにしたい

Question

実現したいこと

画像を追加してDB上に画像を表示させたい。

発生している問題・分からないこと

入力フォームから画像を選択して登録ボタンを押す。
DB上で確認するとNULLになっていて、エラーなどは発生していない。

エラーメッセージ

エラーメッセージは出ていない。

該当のソースコード

<form action="{{ route('curriculum_store') }}" method="POST" enctype="multipart/form-data">
        @csrf
        <div class="form-group">
            <label for="thumbnail_image">サムネイル画像:</label>
            <input type="file" name="thumbnail_image" id="thumbnail_image" accept="image/*">
        </div>
        <br>
        <label for="grade_id">学年:</label>
        <select name="grade_id" id="grade_id">
            @foreach ($grades as $grade)
                <option value="{{ $grade->id }}" {{ request('grade_id') == $grade->id ? 'selected' : '' }}>
                    {{ $grade->name }}
                </option>
            @endforeach
        </select>
        <br>
        <!-- aaaaaaaaaaaaaaaaaaa -->
        <label for="title">授業名:</label>
        <input type="text" id="title" name="title">
        <label for="video_url">動画URL:</label>
        <input type="text" id="video_url" name="video_url">
        <label for="description">授業概要:</label>
        <textarea id="description" name="description" rows="4" cols="50"></textarea>
        <label for="alway_delivery_flg">常時公開:</label>
        <input type="checkbox" id="alway_delivery_flg" name="alway_delivery_flg" value="1">
        <input type="submit" value="登録">
    </form>

public function CurriculumStore(CurriculumsRequest $request)
    {
        $grade = Curriculum::where('grade_id' , $request->input('grade_id'))->first();
        $model = new Curriculum();
        $model->storeCurriculum($grade , $request);

        return redirect()->route('curriculum_list');
    }

public function storeCurriculum($grade , $request)
    {
        //授業データを保存
        $curriculums = new Curriculum();
        $curriculums->title = $request->input('title');

        //ここにサムネイル画像を入れる
        // 商品画像を保存
        if ($request->hasFile('thumbnail_image')) {
            return back()->with('error', 'ファイルがアップロードされていません');
            $imagePath = $request->file('thumbnail_image')->store('images', 'public');
            $curriculums->thumbnail = $imagePath;
        }

        $curriculums->description = $request->input('description');
        $curriculums->video_url = $request->input('video_url');
        $curriculums->alway_delivery_flg = $request->boolean('alway_delivery_flg');
        $curriculums->grade_id = $request->input('grade_id');

        $curriculums->save();
        return $curriculums;
    }

public function rules()
    {
        return [
            'title' =>'required|max:255',
            'description'=>'max:2000',
            'thumbnail' => 'nullable|image|mimes:jpeg,png,jpg,gif|max:2048',
            'image' => 'nullable|image|mimes:jpeg,png,jpg,gif|max:2048',
        ];
    }
    public function messages()
    {
        return [
            'title.required' => '授業名は必須項目です。',
            'title.max' => '授業名は255文字以内で入力してください。',
            'description.max' => '授業概要は2000文字以内で入力してください。',
            'thumbnail.image' => '商品画像は画像ファイルを選択してください。',
            'thumbnail.mimes' => '商品画像はjpeg、png、jpg、gif形式の画像ファイルを選択してください。',
            'thumbnail.max' => '商品画像のサイズは2MB以下にしてください。',
        ];
    }

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

チャットGPTで調べた。
保存先は正しいか
送信ができているかの確認のコードを教えてもらい試したが、エラー

補足

イメージ画像がある位置
public\img\ash-edmonds-fsI-_MRsic0-unsplash.jpg

---

Comments

[sai]

modelの9行目の処理に入っていないと思うのですが、$requestの値をvar_dumpするとどうでしょうか。

[makiharajin_74]

コメントありがとうございます。
下記であっていますか？
9行目　if ($request->hasFile('thumbnail_image')) {
10行目　var_dump('thumbnail_image');　ー＞追加

[sai]

そこだと処理に入らないと出力されないので、modelの6行目あたりに入れてみてください。

Answer 1

アップロードファイルがあったら前の画面に戻って終了、となっています

if ($request->hasFile('thumbnail_image')) {
    return back()->with('error', 'ファイルがアップロードされていません');

データ保存のロジック自体見直したほうがよさそうにも思えます。
未使用の変数などもありますし、Curriculumモデルにcurriculumsテーブルの保存を行うだけであればメソッドを作る必要はないでしょう。

Comments

[makiharajin_74]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。

Answer 2

サムネイル画像は thumbnail_image パラメータで送信して保存、最終的に $curriculums->thumbnail に保存先のパスを登録する設計になっているかと思います。

ですが、 php(Request) ファイルの6行目、リクエスト時のパラメータのバリデーションの名前が違うので、こちらが原因かもしれません。

            'thumbnail' => 'nullable|image|mimes:jpeg,png,jpg,gif|max:2048',

thumbnail_image に変更することで解決できるかと思います。

Comments

[makiharajin_74]

回答ありがとうございます。試してみましたが解決できませんでした。
また、エラーなども出ない状態です。

'thumbnail_image' => 'nullable|image|mimes:jpeg,png,jpg,gif|max:2048',

この場合どの辺りに原因がありそうでしょうか？

[ytsuyuzaki]

それであれば
```
$request->hasFile()
```
での返却値がどのようになっているか、画像送信時にどの行で何が起こっているのかを一つずつ確認するのが近道です。
PHPのデバッガや `dd()` によってコントローラーに届いたデータ内容を確認したり、関数の返却値を確認してみてください。

画像以外の登録は上手く行っているようなので、ともすれば画像の有無を見ているif文から処理内容を確認する必要があります。

ちなみに、 `$request->hasFile()` 関数は php.ini の upload_max_filesize で設定されている値 (デフォルトでは2M )を超えるファイルサイズだと false を返却されてしまいます。