Laravel で値を登録する際にリストにない値を登録したい、その際に既存のリストとの重複をバリデーションしたい、ただしフォームは一つ

Question

実現したいこと

controller で update() をしても反映がされない。

前提

Laravel 初心者です
Laravel10 で入力欄を作っています。

1. Laravel で例えば趣味を入力する欄を select で作る
2. リストにないものは input:text で入力する
3. ただし text の値は select の値と重複しない
4. これを一つの入力フォームで実現したい
5. 最後にアクセスされた値をリストの最上段にもっていきたい（ updated_at を上げる？もしくは専用のカラム（以下の例では last_use ）を使う？）

これで 4. までは実現することができました。

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

$hobby -> update();
は実行されていて選択された趣味の id を $message として返して表示していますが、実際には変更がテーブルに反映されていません。

該当のソースコード

例えば index.blade.php は

<select name="hobby">
<option value="1">映画</option>
<option value="2">旅行</option>
<otpion value="3">筋トレ</option>
</select>
<input type="text" name="hobby_new">

として、hobby にない値を入力したい際に hobby_new に値を入力し submit したとします。
これを趣味用テーブル hobbies に登録し、次回以降は例えば <option value="4">ゲーム</option>と追加したいと考えています。

コントローラは以下のような感じです

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests\HobbyPostRequest;	// バリデーション用
use Illuminate\Http\RedirectResponse;
use Illuminate\View\View;
use App\Models\Hobby;

class HobbyController extends Controller
{
	public function index() : View
	{
		$hobbys = hobby::all()->sortByDesc('last_use');
		return view('hobbies/index', ['hobbies' => $hobbies]);
	}

	public function create(hobbyPostRequest $request, hobby $hobby) : RedirectResponse
	{
		$hobby = new hobby();
		$hobby -> last_use = date("Y-m-d H:i:s");
		if($request -> hobby_new){
			$hobby -> hobby = $request -> hobby_new;
			$hobby -> save();
			$message = $hobby -> hobby .'を追加しました';
		}else{
			$hobby -> id = $request -> id;
			$hobby -> update();
			$message = $hobby -> id .'を変更しました';
		}
	return redirect('hobbies')->with('message', $message);
}

試したこと

気になったので調べてみたところ $hobby -> update(); の戻り値は TRUE になっていました。
なぜ update() しても反映されないのかわかりません。
どなたかヒントだけでもお願いいたします。

Answer 1

Hobbyモデルに$fillableを定義してlast_useを追加してください。
そうすればupdate()で更新できるようになります。

• App\Models\Hobby

    protected $fillable = [
        'last_use',
    ];

Comments

[sanokoyuki]

niiyz 様、ご指導ありがとうございます
$fillable 、勉強になります
ただ、

class Hobby extends Model
{
use HasFactory;
protected $fillable = [
'last_use',
];
}
として念のためキャッシュをクリアしましたがだめでした

$fillable についてはこちらも参考にさせていただきました
https://qiita.com/monji586/items/58d91891caa51b514166

[sanokoyuki]

追加の情報です
気になったので tinker を使って操作してみました
$ sail artisan tinker
> $hobby = Hobby::find( 1 );
> $hobby -> last_use = date("Y-m-d H:i:s");
> $hobby -> update();
でやると更新ができてデータベースにも反映されました
ということは $fillable 以外にも何か問題があるのでしょうね・・・

[niiyz]

見直しましたが既存にアクセスするソースがおかしいですね。

```
//$hobby = new hobby();
// $hobby -> last_use = date("Y-m-d H:i:s");
// $hobby -> id = $request -> id;
// $hobby -> update();

// $fillableいらないパターン
$hobby = Hobby::find(1);
$hobby -> last_use = date("Y-m-d H:i:s");
$hobby ->save();

or

// $fillableいるパターン
$hobby = Hobby::find(1);
$hobby ->update(['last_use' = date("Y-m-d H:i:s")]);
```

> public function create(hobbyPostRequest $request, hobby $hobby)
ルーティングが見れてないので推測ですがメソッドインジェクションで既存のHobbyが$hobbyに展開されてくるようになっていても
常に$hobby = new hobby();で初期されています。
$hobby -> id = $request -> id;でupdate()しても裏でselect * from hobbies where id = ?とか実行されないと思います。

[sanokoyuki]

ありがとうございます。
結果的にですが、updated_at が last_use と同じ働きをしてくれるので、last_use カラムをなくせました。

HobbyController.php のクラス内を

class HobbyController extends Controller
{
public function index() : View
{
$hobbys = hobby::all()->sortByDesc('updated_at');
return view('hobbies/index', ['hobbies' => $hobbies]);
}

public function create(hobbyPostRequest $request, hobby $hobby) : RedirectResponse
{
if($request -> hobby_new){
$hobby = new hobby();
$hobby -> hobby = $request -> hobby_new;
$hobby -> save();
$message = $hobby -> hobby .'を追加しました';
}else{
$hobby = Hobby::find($request -> id);
$hobby -> update();
$message = $hobby -> id .'を変更しました';
}
return redirect('hobbies')->with('message', $message);
}

とすることで正常動作することを確認できました。
本当に助かりました。ありがとうございました。

[sanokoyuki]

すみません、嘘をついていました。
last_use をなくすと変更がないので updated_at が更新されません。

- $hobby -> update();
+ $hobby -> touch();

で、updated_at が更新されました。
今回は本当に勉強になりました。
ご回答いただきました niiyz 様、ありがとうございました。