Почему не работает whereHas?

Question

[Илья]

Есть таблица с товарами и таблица для отношений товары-подарки. Подарками являются те же товары:

Schema::create('products', function (Blueprint $table) {
    $table->increments('id');
    $table->string('name');
    $table->boolean('is_gift')->default(false);
});

Schema::create('products_gifts', function (Blueprint $table) {
    $table->increments('id');
    $table->integer('product_id');
    $table->integer('gift_id');
});

Модель:

class Product extends Model
{
    protected $table = 'products';

    public function gifts()
    {
        return $this->belongsToMany(static::class, 'products_gifts', 'product_id', 'gift_id');
    }
}

Таблица products:


Таблица products_gifts:


Получим товары, которые не являются подарками (+ подарки в отношениях):

$products = Product::with('gifts')->where('is_gift', '=', false)->get()->toArray();

Всё ок, возвращаются три товара, с подарками, как и задумывалось.

Теперь нужно получить только те товары, у которых одним из подарков является "Первый товар-подарок", его id=4.

Что я делаю:

$products = Product::with('gifts')
    ->where('is_gift', '=', false)
    ->whereHas('gifts', function (Builder $query) {
        $query->where('id', '=', 4);
    })
    ->get();

Получается вот такой sql:

select *
from `products`
where `is_gift` = 0
    and exists(select *
        from `products` as `laravel_reserved_0`
        inner join `products_gifts` on `laravel_reserved_0`.`id` = `products_gifts`.`gift_id`
        where `products`.`id` = `products_gifts`.`product_id`
        and `id` = 4)

И вот такая ошибка:

SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'id' in where clause is ambiguous (SQL: select * from `products` where `is_gift` = 0 and exists (select * from `products` as `laravel_reserved_0` inner join `products_gifts` on `laravel_reserved_0`.`id` = `products_gifts`.`gift_id` where `products`.`id` = `products_gifts`.`product_id` and `id` = 4))

Если добавить 'laravel_reserved_0.' то работает:

$products = Product::with('gifts')
    ->where('is_gift', '=', false)
    ->whereHas('gifts', function (Builder $query) {
        $query->where('laravel_reserved_0.id', '=', 4);
    })
    ->get();

Но что-то мне подсказывает, что это решение так себе.

Я так понимаю мешает поле id в таблице products_gifts? К сожалению, структуру таблиц менять нельзя.

Что можно сделать?

Comments

[Модератор]

New_Horizons дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

[Alex Wells]

Думаю по нормальному не выйдет.

Answer 1

[Илья]

$products = Product::with('gifts')
	->where('is_gift', '=', false)
	->whereHas('gifts', function (Builder $query) {
		$table = $query->getModel()->getTable();
		$query->where("{$table}.id", '=', 4);
	})
	->get();

Comments

[vism]

сам то понял, что глупость сделал?

[Илья]

vism, нет, не понял, потрудитесь объяснить

[vism]

Илья, Извиняюсь, проверил.
внутри whereHas $query->getModel()->getTable() возвращает laravel_reserved_0
Идеально! :)

Answer 2

[Валерий]

select *
from `products`
where `is_gift` = 0
  and exists(select *
             from `products` as `laravel_reserved_0`
                    inner join `products_gifts` on `laravel_reserved_0`.`id` = `products_gifts`.`gift_id`
             where `products`.`id` = `products_gifts`.`product_id`
             and `laravel_reserved_0`.`id` = 4)

Последняя строка в запросе.

Comments

[Илья]

Да, спасибо, я понял что в этом проблема) Вопрос как её лучше решить средствами Laravel

Answer 3

[vism]

Вместо whereHas, использовать whereExist, и этот код преобразованый в query builder

select * from `products` as `laravel_reserved_0` inner join `products_gifts` on `laravel_reserved_0`.`id` = `products_gifts`.`gift_id` where `products`.`id` = `products_gifts`.`product_id` and `laravel_reserved_0`.`id` = 4

Answer 4

[krioz]

belongsToMany(static::class
а не нужно ли здесь указывать класс гифтов, а не себя же?

Comments

[Илья]

Не думаю, модель относится как-бы сама к себе. Подарки являются товарами.