SaveメソッドでDBに既に存在するモデルは更新されません

Question

私のDB内のレコードを正しく取得し、textfieldに値を出力します。しかし、私のデータベースを更新したいとき、それは動作しません。それは

save()方法はまた、既にデータベースに を存在するモデルを更新するために使用することができる文書で述べています。モデルを更新するには、モデルを取得する必要があります。

だから、私は2つのブレードのテンプレートショーと編集を持っています。最初のブレードは、データベースに値が存在することを示します（GET REQUEST）。 2番目のブレードは、ユーザーがデータを更新するための編集フォームを示します。 （POST REQUEST）。

show.blade.php

<form class = "form-vertical" role = "form" method = "post" action = "{{ route ('account.show', $result->id) }}"> 

<h3>View Employee</h3> 
<hr> 

<div class = "form-group"> 

    <label for = "email" class = "control-label">Email Address</label> 
    <input type = "text" name = "email" class = "form-control" placeholder = "{{ $result->email }}" readonly> 

</div> 

<div class = "form-group"> 

    <label for = "username" class = "control-label">Username</label> 
    <input type = "text" name = "username" class = "form-control" placeholder = "{{ $result->username }}" readonly> 

</div> 

<input type = "hidden" name = "_token" value = "{{ Session::token() }}"> 

</form> 

edit.blade.php

<form class = "form-vertical" role = "form" method = "post" action = "{{ route ('account.edit', $result->id) }}"> 

<h3>Edit Employee</h3> 
<hr> 

<div class = "form-group"> 

    <label for = "email" class = "control-label">Email Address</label> 
    <input type = "text" name = "email" class = "form-control" value = "{{ $result->email }}"> 

</div> 

<div class = "form-group"> 

    <label for = "username" class = "control-label">Username</label> 
    <input type = "text" name = "username" class = "form-control" value = "{{ $result->username }}"> 

</div> 

<div class = "form-group"> 

    <button type = "submit" class = "btn btn-success">Save</button> 

</div> 

<input type = "hidden" name = "_token" value = "{{ Session::token() }}"> 

</form> 

コントローラー：

//SHOW 
public function showEmployee($id) 
{ 
    $result = User::find($id); 

             //key  //value 
    return view ('account.show')->with('result', $result); 
} 

//EDIT 
public function editEmployee($id) 
{ 

    $result = User::find($id); 

    return view ('account.edit')->with('result', $result); 
} 

//UPDATE 
public function updateEmployee(Request $request, $id) 
{ 
    $this->validate($request, [ 
          //This will be unique in users table 
     'email' => 'required|unique:users,email,'.$id.'|email|max:255', 
     'username' => 'required|unique:users,username,'.$id.'|alpha|max:20', 
    ]); 

    $email = $request['email']; 
    $username = $request['username']; 

    $user = User::findOrFail($id); 
    $user->email = $email; //Accessing properties of Model 
    $user->username = $username; 

    $user->save(); 

    return view ('account.show'); 
} 

ルート：

//SHOW 
Route::get('/show/{id}', 
[ 
    'uses' => '\App\Http\Controllers\AccountController@showEmployee', 
    'as' => 'account.show', 
]); 

//EDIT 
Route::get('/edit/{id}', 
[ 
    'uses' => '\App\Http\Controllers\AccountController@editEmployee', 
    'as' => 'account.edit', 
]); 


//UPDATE 
Route::post('/edit/{id}', 
[ 
    'uses' => '\App\Http\Controllers\AccountController@updateEmployee', 
    'as' => 'account.edit', 
]); 

Answer 1

あなたはここに新しいUserを作成しています。

既存のデータを更新するには、まず編集するユーザーデータを読み込む必要があります。あなたは、両方の場合はまた、あなたの検証がここに失敗します（$idと仮定すると、ここで、そのユーザーのIDです）

$user = User::findOrFail($id); 

に

$user = new User(); 

を変更することにより、

これを行うと、その問題を解決することができます両方のエントリの一意性をチェックしているため、現在のユーザーにはすでにそのユーザー名または電子メールが割り当てられているため、ユーザー名と電子メールは変更されません。

は、この問題を解決するには、の「指定したidを無視する独自のルールを強制」を参照してください

'email' => 'required|unique:users,email,'.$id.'|email|max:255', 
    'username' => 'required|unique:users,username,'.$id.'|alpha|max:20', 

、検証のためにこのようなものを通じてチェックから現在のユーザーを除外する必要がありますここのドキュメント - Validation - unique rule

Answer 2

既存のモデルを取得し、それを更新する必要がある代わりに、あなたのupdateEmployee方法が提供さ$idパラメータを使用していない、とあなただけの新しいユーザーインスタンスを作成し、それを保存します。

$user = User::find($id);// retrieve the existing user 
$user->email = $email; 
$user->username = $username; 

$user->save();// update the record in DB