CODEWITHSUNDEEP
phplaravellaravel-5laravel-bladelaravel-form
Hima Hima
Hima Hima

Reputation: 123

Can't update user in laravel

I want some help with my code.

I try to update user data but it's not update anything.

User Name, Email, Posisson, Image.

Any help please.

My Route : I used URL because route didn't work.

Route::get('editusers/{id}','UsersController@update');

My Controller:

public function edit($id)
{
    $editusers=User::findOrFail($id);
    return view('admin.users.EditUser', compact('editusers'));
}

public function update(Request $request, $id)
{
   $this->validate($request, [
        'name' => 'required',
        'email' => 'required',
        'posission' => 'required',
    ]);
    $useredit = User::find($id);
    $useredit->name = $request->input('name');
    $useredit->email = $request->input('email');
    $useredit->posission = $request->input('posission');
    if($request->hasFile('file'))
    {
        $file = $request->file('file');
        $filename = time().'.'.$file->getClientOriginalExtension();
        Image::make($file)->resize(150, 150)->save(public_path('/admin/images/'.$filename));
        $useredit->UserImg = $filename;
    }

    $useredit->save();
    return redirect()->back();
}

HTML :

<form class=""  action="{{url('editusers',Auth::user()->id)}}"  role="form" enctype="multipart/form-data">
     {!! csrf_field() !!}
     <input type="hidden" name="_method" value="PUT">

     <label>Edit Your Profile :</label>
     <div class="form-group">
         <label>Name :</label> 
         <input class="form-control" value="{{$editusers->name}}" name="Name">
     </div>

      <div class="form-group">
         <label>Email :</label>
         <input class="form-control" value="{{$editusers->email}}" name="email">
      </div>
      <div class="form-group">
          <label>Posisson :</label>
          <input class="form-control" value="{{$editusers->posission}}" name="posission">
      </div>
      <div class="form-group">
          <label>Image :</label>
          <img src="{{ asset('admin') }}/images/{{$editusers->UserImg}}" alt="avatar" class="img-circle" style="max-height: 100px;">
          <input type="file" id="file" name="file"/> 
      </div>
  <input class="btn btn-success btn-mini deleteRecord" type="submit"   name="submit" value="Update">

What I expect is that it updates my database.

Upvotes: 2

Views: 2845

Answers (4)

Ayoub Aarab
Ayoub Aarab

Reputation: 67

i think you should use resources route to solve this:

Route::resource('editusers','UserController');
but first you need to run this command
php artisan make:controller UserController --resource

Upvotes: 0

Prafulla Kumar Sahu
Prafulla Kumar Sahu

Reputation: 9703

As your form has

<form class=""  action="{{url('editusers',Auth::user()->id)}}"  role="form" enctype="multipart/form-data" method="POST">
    {!! csrf_field() !!}
     <input type="hidden" name="_method" value="PUT">

so your route must have put(),

so it should be Route::put('editusers/{id}','UsersController@update');

also you can use @method('PUT') instead of <input type="hidden" name="_method" value="PUT"> and @csrf instead of {!! csrf_field() !!}

either change $useredit->name = $request->input('name'); to $useredit->name = $request->input('Name'); or in form

<input class="form-control" value="{{$editusers->name}}" name="Name"> to 

<input class="form-control" value="{{$editusers->name}}" name="name">

Upvotes: 2

Sok Chanty
Sok Chanty

Reputation: 1816

The problem because you put wrong method at your route. Change it 

// From 
Route::get('editusers/{id}', 'UsersController@update')
// To
Route::put('editusers/{id}', 'UsersController@update')

Anyways, you should change your route to be standard. It should be:

//To show data you should use: 
Route::get('users/edit/{id}', 'UsersController@show');

//To update user data. 
Route::put('users/edit', 'UsersController@update');

Upvotes: 0

Sethu
Sethu

Reputation: 1389

Your route is wrong Route::get('editusers/{id}','UsersController@update');

it supposed to be PUT

Route::put('editusers/{id}','UsersController@update');

Upvotes: 0

Related Questions