Laravel passing multiple parameter to controller

Question

I want to delete an image and return back to user page, so i try to pass 2 parameter to my controller (my userid and image_id).

But I get this error

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

My button code in blade

<button id="delete" class="btn btn-danger btn-sm" data-title="{{ $mcu_form_document['title'] }}"
        style="color: #fff; font-size: 1.2em;"
        href="{{ route('mcu-form-document.destroy', ['id'=>$id ,'img'=>$mcu_form_document['id']]) }}">
    Delete <i class="la la-trash" style="color: #fff; font-size: 1.2em;"></i>
</button>

<form action="{{ route('mcu-form-document.destroy', ['id'=>$id ,'img'=>$mcu_form_document['id']]) }}" method="post" id="deleteForm">
    {{ csrf_field() }}
    {{ method_field('DELETE') }}

    <input type="submit" value="" style="display:none;">
</form>

My controller

public function destroy($id,$img_id)
{
    $mcu_form_document = McuFormDocument::find($img_id);
    $mcu_form_document->delete();

    return redirect()
        ->route('mcu.resume.list', ['id' => $id])
        ->with('success', 'Document has been successfully deleted!');
}

my route

    Route::resource('mcu-resume', 'McuResumeController');

check your routes php artisan route:list that resource isn't going to make any route with more than 1 parameter — lagbox
– lagbox, Commented Aug 11, 2020 at 2:39
you will have to define a route yourself that takes 2 parameters ... but why do you need your 'user_id' passed? — lagbox
– lagbox, Commented Aug 11, 2020 at 2:46
you would either need to create this as a nested resource or create your own delete route that takes 2 parameters — lagbox
– lagbox, Commented Aug 11, 2020 at 3:00

Heru Handoko · Accepted Answer · 2020-08-20 06:34:53Z

1

my solution is using concatenation when passing the params in href

 href="{{ route('mcu-form-document.destroy', $id .'|'.$mcu_form_document['id']) }}"

and split it with preg_split in my controller

    public function destroy($params)
    {
    $str_arr = preg_split("/\|/", $params);
    $id = $str_arr[0];
    $img_id = $str_arr[1];
    // dd($id . "-" . $img_id);
    }

hope this will help others..thx for all the help

answered Aug 20, 2020 at 6:34

Heru Handoko

891 gold badge2 silver badges10 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Rabinson · Accepted Answer · 2020-08-11 05:04:44Z

0

Or you can also add a request parameter into your controller which enables you to pass multiple data and get the required value as below:

use Illuminate\Http\Request;

public function destroy(Request $request)
{
   $id = $request->get('id');
   $image_id = $request->get('img');
   $mcu_form_document = McuFormDocument::find($image_id);
   $mcu_form_document->delete();

   return redirect()
    ->route('mcu.resume.list', ['id' => $id])
    ->with('success', 'Document has been successfully deleted!');
 }


 //web.php
 Route::delete('mcu-resume-destroy', 'McuResumeController@destroy');

answered Aug 11, 2020 at 5:04

Rabinson

1012 silver badges7 bronze badges

2 Comments

Heru Handoko Over a year ago

got null for $id

Rabinson Over a year ago

can you try using action('McuFormDocumentController@destroy', [ 'id' => $id, 'img'=>$mcu_form_document['id'] ]) should have worked with route() too.

Collectives™ on Stack Overflow

Laravel passing multiple parameter to controller

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related