Laravelで画像とテキストをフォームから更新できない。

こんにちは。Laravel初心者の大学生です。

現在Laravelで投稿の編集フォームを作っています。
編集フォームのテキストエリアにはDBに登録されているテキストが表示されるようにし、画像は新しい画像が選択された場合のみ更新できるようにしたいです。
自分でコードを組んでみたのですが、画像を選択していないと

SQLSTATE[HY000]: General error: 1364 Field 'image' doesn't have a default value (SQL: insert into tests (user_id, name, updated_at, created_at) values (1, あああ, 2020-07-19 06:59:31, 2020-07-19 06:59:31))

というエラーが表示され、また、画像を選択した場合は編集されるのではなく、DBに新しくデータが追加されてしまいます。

以下がコードになるのですが、どこで間違えてしまっているのかご教示願いたいです。

web.php

php
1Route::get('/tests/edit/{id}', 'TestController@edit'); //編集
2    Route::patch('/tests', 'TestController@update'); //更新

TestController.php

php
1use Illuminate\Support\Facades\Storage;
2
3public function edit(Request $request)
4    { 
5        $test = Test::findOrFail($request->id);
6        return view('test.edit', ['test' => $test]);
7    }
8
9public function update(Request $request, Test $test)
10    {
11        $test->user_id = $request->user()->id;
12        $test->name = $request->name;
13        
14        if($request->hasFile('image')) {
15            Storage::delete('public/image/' . $test->image); //元の画像を削除
16            $path = $request->file('image')->store('public/image'); //画像のパス
17            $test->image = basename($path);
18            $test->save();
19        }
20
21        $test->save();
22
23        return redirect('/tests')->with('flash_message', '編集が完了しました');
24    }

edit.blade.php

php
1@extends('layouts.app')
2
3@section('content')
4<div>
5    <form action="/tests" method="post" enctype="multipart/form-data">
6        {{ csrf_field() }}
7        {{ method_field('patch') }}
8            料理名：<input type="text" name="name" value='{{ $test->name }}'><br>
9            画像： <input type="file" name="image"><br>
10            <input type='submit' value='更新'>
11    </form>
12</div>
13@endsection

よろしくお願いいたします。

SanQ

2020/07/19 07:48

DBの対象テーブルの「image」カラムの設定はどうなっていますか？nullableですか？

toyop

2020/07/19 08:07

コメントありがとうございます。投稿する時点で画像は必須にしてあるので、nullableにはなっていないです。

行動規範の内容に同意します

回答1件

このコードはここに問題があると思います。
このコードはDBに既に存在するレコードを上書きするのではなく、新たにレコードを作成するコードになっていること。

すでに存在するDBのレコードの値を上書きする際は、Modelクラスにあるsave()を使用するのではなく、update()を使用します。
update()についてはこちら
Laravel Database: Query Builder
日本語版
データベース：利用開始 7.x Laravel - ReaDouble

あと、SQLSTATE[HY000]: General error: 1364 Field 'image' doesn't have a default valueの言わんとすることは、現在存在しているテーブルのレコードのimageカラムは値が入ってなくてはいけないのに、何も入れてくれていないという規約違反をしているからエラーだよ、ということです。
今回の例で言うと、save()を使って新たにレコードを作成するのにも関わらずimageが空だからエラーを出しています。このエラーは上記の問題を解決させ、もし写真の変更がなければそのレコードのimageは何もしない、というコードを書けば表示されなくなると思います。

投稿2020/07/19 08:49

編集2020/07/19 10:36

SanQ

総合スコア92