Laravel - delete button

Question

Welcome ! I'm new to laravel and i have problem. I start my private project and i want to delete query from database using delete button ( simple todo app). When i click it it doesn't delete but only redirect me to id of this query. Dont know what to do. Can someone help me?

Regards tomczas

Destroy function in homecontroller

public function destroy($id)
{
 $todo = Todo::findOrFail($id);
 $todo->delete();
return back();

}

home blade

@foreach($todos as $todo)
          <ul class="todo-list">
            <li>
              <!-- drag handle -->
                  <span class="handle">
                    <i class="fa fa-ellipsis-v"></i>
                    <i class="fa fa-ellipsis-v"></i>
                  </span>
              <!-- checkbox -->
              <input type="checkbox" value="">
              <!-- todo text -->
              <span class="text">{{$todo->tytul}}</span>
              <!-- Emphasis label -->
              <small class="label label-danger"><i class="fa fa-clock-o"></i> {{$todo->czas}}</small>
              <!-- General tools such as edit or delete-->
              <div class="tools">
                <i class="fa fa-edit"></i>
                {{Form::open([ 'method'  => 'DELETE', 'route' => [ 'home.destroy', $todo ] ])}}
                                {{ Form::hidden('id', $todo->id) }}
                                {{Form::button('<i class="fa fa-trash-o"></i>', array('type' => 'submit', 'class' => ''))}}
                            {{ Form::close() }}
                <i class="fa fa-trash-o"></i>
              </div>
            </li>

          </ul>
          @endforeach

num8er · Accepted Answer · 2016-12-17 22:46:24Z

3

Fix the form:

{{Form::open(['method'  => 'DELETE', 'route' => ['home.destroy', $todo->id]])}}

and no need for:

{{ Form::hidden('id', $todo->id) }}

... laravel does all magick

p.s. make sure You've created resource route to make home controller act as REST controller.

extra, watch this video: https://www.youtube.com/watch?v=6pjPXOwKzJM

another way of doing delete function is that - You can create some route:

Route::get('home/{id}/delete', ['uses' => 'HomeController@destroy', 'as' => 'home.delete']);

and in this case no need for form:

<div class="tools">
  <i class="fa fa-edit"></i>
  <a href="{{route('home.delete', $todo->id)}}?{{time()}}">
    <i class="fa fa-trash-o"></i>
  </a>        
</div>

p.s. after reading Your routes I see also core problem:

Route::resource('home', 'HomeController@index');

should be (read this):

Route::resource('home', 'HomeController');

edited Dec 17, 2016 at 22:46

answered Dec 17, 2016 at 21:34

num8er

19.5k4 gold badges50 silver badges61 bronze badges

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

4 Comments

tomczas Over a year ago

I have this Route::resource('home', 'HomeController@index'); and it still redirects me to mypath/1 where 1 is ID task which i want to delete

num8er Over a year ago

@tomczas I've updated my answer. So I'm not depending on "magics" of frameworks and I'm defining my routes exacly, and using "another way of ding" example in my answer.

tomczas Over a year ago

NotFoundHttpException in RouteCollection.php line 161:

tomczas Over a year ago

Route::get('/', function () { return view('welcome'); }); Route::resource('home', 'HomeController@index'); Route::get('/home/{$id}/delete', ['use' => 'HomeController@destroy', 'as' => 'home.delete']); Route::get('logout', array('uses' => 'HomeController@doLogout')); Route::auth();

num8er · Accepted Answer · 2016-12-17 22:46:50Z

-1

Ok second idea is working great but little mistake in spelling:

should be:

Route::get('/home/{id}/delete', ['uses' => 'HomeController@destroy', 'as' => 'home.delete']);

so {id} not :id and uses not use Thank u very much for help !

edited Dec 17, 2016 at 22:46

num8er

19.5k4 gold badges50 silver badges61 bronze badges

answered Dec 17, 2016 at 22:40

tomczas

1791 gold badge4 silver badges20 bronze badges

1 Comment

num8er Over a year ago

I've fixed it in my answer (: I'm not using LV so much last year so forgot how to set the {id} and :id is from expressjs (;

Hamidreza Ghanbari · Accepted Answer · 2018-09-21 08:10:26Z

-1

in a new version of laravel 5.6 , laravel suggest to use code below to delete one row

<form method="post" action={{ route('yourControllerDeleteMethodName'), ['yourRowName'=> $yourRowName->slug] }}> <input type="hidden" value="DELETE"/> <button type="submit">click to delete</button> </form>

this work when you use slug to get row if you use id to get row just write $id instead of slug.

hope to work for you:)\

answered Sep 21, 2018 at 8:10

Hamidreza Ghanbari

2972 silver badges11 bronze badges

Collectives™ on Stack Overflow

Laravel - delete button

3 Answers 3

4 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related