Too Few Arguments

I am getting this error but cannot figure out why

Too few arguments to function App\Http\Controllers\ProfilesController::update(), 1 passed and exactly 2 expected

public function update(Request $request, $id)
    {
        $this->validate($request, [
            'name' => 'required'
            
        ]);
        $user = User::find($id);
         // Handle File Upload
        
        // Update Post
        $user->name = $request->input('name');
         
        $user->save();
        return redirect('profile.profile')->with('success', 'Profile Updated');
    }

Sergiu17

6 years ago

Level 60

Your route does not have wild card

// wrong
Route::put('/profile',ProfilesController@update')

// correct 
Route::put('/profile/{id}', 'ProfilesController@update');

ahmeddabak

6 years ago

Level 47

Make sure that you have imported the right Request class with this use statement use Illuminate\Http\Request;.
if your are not using a resource controller, make sure that your route difinition in web.php expect a parameter Route::post('/profile/{id}','ProfilesController@update');

artisticre

6 years ago

Level 5

I did this and got Missing required parameters for [Route: update] [URI: profile/{id}].

Route::put('/profile/{id}', 'ProfilesController@update')->name('update');

Sergiu17

6 years ago

Best Answer

Level 60

@artisticre now when you send the request, make sure you include the ID

<form action="/profile/{{ $user-> id }}"

// instead of
$user = User::find($id);
// use
$user = auth()->user()

artisticre

6 years ago

Level 5

I am using that Request and using using Resource Controller

Sergiu17

6 years ago

Level 60

@ahmeddabak if you used $user = auth()->user(), then you could remove $id from your update method

Route::put('/profile', 'ProfilesController@update')

public function update(Request $request)
{
    $user = auth()->user();

    // ...
}

Please or to participate in this conversation.