在Laravel中以多对多关系显示要删除的数据的问题

Question

I have many to many relationship between users and products table and I set up a view (welcome.blade.php) that when I click on the product name, which is a link, it is supposed to redirect me to page where my delete form is so I can delete that particular product, but I get 404 not found page. I suspect that error is somewhere in my routes but I can't seem to find the problem. Also when I click on some product my url says project/destroy/1 which I think is good. Here is my code:

web.php:

Route::get('/home', 'HomeController@index')->name('home');
Route::post('/store', 'HomeController@store')->name('store');
Route::delete('/destroy/{$id}', 'HomeController@destroy')->name('destroy');

destroy.blade.php:

<div class="col-md-12">
    <form action="destroy/{{ $product->id }}" method="POST">
     @csrf
     @method('DELETE')
        <button type="submit" class="btn btn-danger">Delete</button>
    </form>
</div>

welcome.blade.php:

@if($products)
<table class="table">
    <thead>
        <th>#</th>
        <th>Product Name</th>
        <th>Owner Of The Product</th>
        <th>Created At</th>
    </thead>
    <tbody>
    @foreach ($products as $product)
        <tr>
            <td>{{ $product->id }}</td>
            <td>
                <a href="{{ route('destroy', $product->id) }}">{{ $product->files }}</a>
            </td>
            <td>
                @foreach ($product->users as $user) {{ $user->name }}
                @endforeach
            </td>
            <td>{{ date('M j, Y', strtotime($product->created_at)) }}</td>
        </tr>
     @endforeach
     </tbody>
     </table>
 @endif

HomeController.php:

<?php

namespace App\Http\Controllers;

use App\Product;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;

class HomeController extends Controller
{
    public function destroy(Product $product)
    {
        $product->users()->detach();
        $product->delete();
        return view('destroy');
    }
}

Answer 1

You defined the following route in your file.

Route::delete('/destroy/{$id}', 'HomeController@destroy')->name('destroy');

This creates a DELETE route. DELETE routes are meant to be accessed by APIs. Links in anchor tags use GET routes. Clicking on

<a href="{{ route('destroy' , $product->id) }}">{{ $product->files }}</a>

makes the router try and match GET /destroy/{$id} which is not defined, so it throws either an exception if debug is on or a 404 if not. You can see this in action by looking at the network tab of your browser's developer console.

Unless you add another GET route, you'll keep getting 404s.

Route::get('/destroy/{$id}', 'HomeController@destroy')->name('product.destroy-form');

will make the following link work.

<a href="{{ route('product.destroy-form' , $product->id) }}">{{ $product->files }}</a>

Also, in the destroy method you're returning the view without passing any variables but in destroy.blade.php you seem to be using $product . Don't forget to add it!

Answer 2

I think it is because to delete a product using a new page the first thing to do is navigate to that page which you are not doing. You are instead going directly to the delete function using this

Route::delete('/destroy/{$id}', 'HomeController@destroy')->name('destroy');

You need to define a route that lets you go to that page first which u can do by creating a route like this:

Route::get('/delete/{$id}', 'HomeController@delete')->name('delete');

You should create a new function delete in the HomeController that just returns the destroy.blade.php. Something like this.

$product= Product::find($id);
    return view('destroy')->with('product', $product);

where $id is the product id and Product is the model that you are using.

Answer 3

You are nearly there, you just have a step too many.

Good call on the form and using the delete method, this is absolutely what you are supposed to do

However where you are going wrong is in using the link to go to a separate page with the form on to delete. You are better using a little javascript so the link submits a hidden form from the link.

<a onclick="document.getElementById('delete-form-{{$product->id}}').submit();}">{{ $product->files }}</a>
<form id="delete-form-{{$product->id}}" action="{{ route('destroy', ['id' => $product->id]) }}" method="POST" style="display: none;">
    @csrf
    @method('delete')
</form>

You can do the method you have but you need an additional route::get request to a page that loads the form and then you can submit the form on that page to delete.