Laravel Eloquent delete（）不起作用

Question

Documentation says:

$user = User::find($user_id);

$user->delete();

This doesnt work, however ProductColor::find($color_id) working. $color->delete() Doest return anything, DELETE FROM query doesn't even execute (as seen in debug bar).

But I can delete record with:

ProductColor::destroy($color_id);

Must be something I overlooked earlier, I'm new to Laravel.

I'm using store also, and it working as expected

public function store(Request $request)
    {
        $color = new ProductColor();
        $color->name = $request->color_name;
        $color->order = $request->color_order;
        $saved = $color->save();

        if ($saved) {
            return back()->with('message:success', 'Ok');
        } else {
            return back()->with('message:error', 'Error');
        }

    }

To sum up

This WORKS

public function destroy($color_id)
    {
        $deleted = ProductColor::destroy($color_id);

        if ($deleted) {
            return back()->with('message:success', 'Deleted');
        } else {
            return back()->with('message:error', 'Error');
        }
    }

This NOT

public function destroy($color_id)
{
  $color = ProductColor::find($color_id);

  $color->delete();
}

My Model

<?php

namespace Modules\Shop\Entities;

use Illuminate\Database\Eloquent\Model;
use Modules\Languages\Entities\Language;

class ProductColor extends Model
{
    protected $fillable = [];
    protected $table = 'product_colors';
}

Answer 1

Sorry, I've figured out the problem. My mistake to post this question.

What I tried to do:

$color = new ProductColor();
$color->find($color_id);
$color->delete();

Should be:

$color = ProductColor::find( $color_id );
$color->delete();

My problem was that I was scared about the IDE complaining about using non-static method 'find'

Answer 2

Your code is fine - the docs show exactly what you are doing.

If there is no error, and the color is not deleted as expected, then $color_id is not being passed as expected. Try using findOrFail , or add some other check that you found the expected model.

Answer 3

如果您添加了软删除选项，请检查您的模型类，在这种情况下，您的记录不会从数据库中删除。

Answer 4

 DB::table('pykcodes')
                ->where('user_id', $user_id)
                ->delete();

This works directly without using the model. Just make sure you call use DB; from the top.

Answer 5

My problem was, the name of the property inside the uri on the routes file was incorrect:

Route::delete('/foo/{fo3o}', [DollyController::class, 'delete']);
                  -----^

I don't know why the controller method was working:

public function delete(Foo $foo)
{
    $foo->delete();
    return ['message' => 'success'];
}

After correcting it, the problem was solve and the delete method started working again:

Route::delete('/foo/{foo}', [DollyController::class, 'delete']);
                ------^

Answer 6

You have to add SoftDeletes in User Model:

...

use Illuminate\Database\Eloquent\SoftDeletes;

class User extends Authenticatable {

use SoftDeletes;

protected $dates = ['deleted_at'];

...