Э
А пользователей сколько?
Один единственный, тестовая база
Size: a a a
2020 October 14
SK
trialUsers() - это связь?
A
trialUsers() возвращает коллекцию?
Э
Да, связь
SK
я к тому, что может скобки убрать, чтобы в each попали юзеры, а не querybuilder?
SK
да и зачем там each() ? может тут нужен where?
A
все эти проблемы от незнания и неумения юзать SQL
A
этот запрос - явный претендент на реализацию простым SQL запросом...
DK
У тебя trialUsers сначала получает всех пользователей, а потом ищет в них триальных
DK
А должен фильтровать триальных на уровне базы, а не на уровне Collection
A
+, тут нужно с запросом работать а не с коллекцией
SK
$this->trialUsers()->whereHas('payments', function($q) {$q->where('price', 1440);})->get()
может предполагалось вот это?
может предполагалось вот это?
ПЛ
return $this->trialUsers()->whereHas('payments', function $q {
$q->where('price',1400)
})->get();
$q->where('price',1400)
})->get();
DK
Плюс ты ещё подтягиваешь все их платежи
АШ
return $this->trialUsers()->whereHas('payments', function $q {
$q->where('price',1400)
})->get();
$q->where('price',1400)
})->get();
или если версия 7.4 то можно так
$this->trialUsers()->whereHas('payments', fn ($q) => $q->where('price', 1440))->get();АШ
А лучше в scope вывести еще для большей красоты)
ПЛ
смысла в скоупе нет, если условие не переиспользуемое
Э
или если версия 7.4 то можно так
$this->trialUsers()->whereHas('payments', fn ($q) => $q->where('price', 1440))->get();Это не сработает, ибо trialUsers - коллекция, а метод whereHas есть только у модели
АШ
смысла в скоупе нет, если условие не переиспользуемое
я думаю price не просто так
АШ
Да, связь
Так связь или коллекция? @theeldarka