如何在laravel eloquent中基于一对多关系中的单个列删除所有行？

Question

When deleting a particular movie , i want to delete all showtime related to it. I have 3 tables related to this query.

1. Showtime (which has mm relation with times table (pivot table showtime_time)
2. Times
3. Movies

What i tried is:

MovieController:

public function destroy($id)
{
    $movie = Movie::findOrFail($id);
    if (ShowTime::where('movie_id', '=', $id)->exists()) {
        $showtimes = ShowTime::whereIn('movie_id', '=', $id)->get();
        foreach ($showtimes as $showtime) {
            $showtime->times()->detach();
            $showtime->delete();
            $movie->delete();
        }
        return back();
    } else {
        $movie->delete();
    }
    return redirect()->route('movies.index');

}

Which gave me this error.

Argument 1 passed to Illuminate\\Database\\Query\\Builder::cleanBindings() must be of the type array, string given, called in C:\\Users\\lenovo\\cinetime_nepal_backend\\vendor\\laravel\\framework\\src\\Illuminate\\Database\\Query\\Builder.php on line 907

Model class

<?php

namespace Modules\Movie\Entities;
use Illuminate\Database\Eloquent\Model;
class Movie extends Model
{
    protected $guarded = ['id']; //fillable
    protected $hidden = ['created_at', 'updated_at'];

    public function showTimes()
    {
        return $this->hasMany(ShowTime::class, 'movie_id');
    }
}

This is how i am deleting showtime in showtimecontroller

ShowtimeController

public function destroy($id)
{
    $showtime = ShowTime::findOrFail($id);
    $showtime->times()->detach();
    $showtime->delete();
    return back();
}

I am new to laravel, i searched and tried many codes answered in this site but could not help myself. Help Please

Answer 1

If you're using MySQL you can add 'ON DELETE CASCADE' to your foreign key. This will automatically delete any records that reference the parent when the parent is deleted. just put it to your showtime migration

$table->foreign('movie_id')->references('id')->on('movies')->onDelete('cascade');

Or, if you don't want that approach, then you can overwrite your delete method in your movie model. Put this code in your movie model.

public function delete() {
    $this->showTimes()->times()->delete();
    $this->showTimes()->delete();
    parent::delete();
}

Then in your controller you just delete your movie and it's all fine.

public function destroy($id)
{
    $movie = Movie::findOrFail($id);
    $movie->delete();

    return redirect()->route('movies.index');

}