Laravel with как использовать where по данным из промежуточной таблицы?

Question

[Роман]

Имеются 4 таблицы.

account_service (pivot table):
user_id (Belongs To Many)
service_id (Belongs To Many)
status

services:
id
tariff_id (One to One)
platform_id (One to One)

platforms:
id
name

tariffs:
id
name

Делаю запрос с подгрузкой данных моделей через модель User:

$services = $this->with('services.tariff','services.platform')->first()->services;

Могу обратится к $services[0]->pivot->status;

Можно ли в запросе делать выборку по условию смежной таблице, где к примеру статус равен 1, что бы не делать перебор в цикле.

Пробовал так:

$services = $this->with('services.tariff','services.platform')->whereHas('services', function($q) {
            $q->where('status', '=', 1);
})->first()->services;

но без успешно...

Answer 1

[jazzus]

В контроллере

$users = User::with(['services' => function($query) {
                  $query->with('tariff','platform')
           ->wherePivot('status',true);
    }])->get();

В модели добавить к отношениям ->withPivot('status')

Comments

[Роман]

Благодарю! До вашего ответа использовал такой велосипед:

$services = $this->with('services.tariff','services.platform')->first()->services()->wherePivot('status',1)->get();

[jazzus]

RomanTrakhtenberg, лучше так

public function getActiveServicesAttribute()
{
  return $this->services()
              ->with('tariff','platform')
              ->wherePivot('status',true)
              ->get();
}

обращаться к геттеру откуда угодно, наименования сохранить
$activeServices = $user->ActiveServices;
Подойдет для одного юзера, но в коллекциях with и withCount, чтобы не было левых запросов к бд

[Alex Wells]

jazzus, нет, не лучше. Вообще нет. Аксесоры сделаны для представления ИМЕЮЩИХСЯ данных и ТОЛЬКО для этого.

На такой код. Потом не разгребешь.

[Роман]

А как ещё добавить группировку по platform_id?

[jazzus]

Alex Wells, лучше я имел ввиду сам запрос. А это для разнообразия знаний) т.к. обычные методы автор делать уже умеет. А так понятно оно не рефакторится и лучше билдер вернуть без with только с active, чтобы можно было где хочешь использовать

[jazzus]

RomanTrakhtenberg,
$query->groupBy('platform_id')
здесь есть все запросы и примеры

[Роман]

jazzus, Документацию видел, куда это вставить в вашем примере?

[Alex Wells]

jazzus, ага, а потом с таким разнообразием знаний от каждого из прогеров в проекте сиди и разбирайся. Особенно весело становится когда в аксесорах вызывают какой-то репозиторий или сервис, производящий трипятсот запросов и манипуляций, и все это дружно один за другим вызывается в трансформере, куда скормили коллекцию. Welcome to hell.

[Роман]

Alex Wells, обычно, когда критикуют, предлагают свой вариант решения.
jazzus, разобрался с группировкой, благодарю ещё раз.

[jazzus]

Alex Wells, может у автора это единственный запрос активных сервисов и он не будет это рефакторить и развивать? Какой-нибудь лендинг про телепузиков. Ладно согласен херню написал

[jazzus]

RomanTrakhtenberg, Алекс полностью прав, в модели не нужно получать никакие коллекции и т.д. Т.к. единственный вариант зарефакторить, это снести и писать заново. Нужно получать билдеры и с ними уже работать в классах, чтобы можно было добавлять параметры к билдеру в разных методах перед получением коллекции и не дублировать код

// в модели user
public function getActiveServices()
{
  return $this->services()->wherePivot('status',true);
}
// в контроллере/классе
$services =  $user->getActiveServices()
                  ->with('tariff','platform')
                  ->get();

[Роман]

jazzus, масштабировать конечно придётся, просто опыта правильной архитектуры и работы с фремворком мало. Поэтому думал всё убрать в модели (что бы не плодить жирных контроллеров), в них же дописать нужные методы. А с классами не совсем понимаю как быть. Если мне нужен класс для работы с пользователем, тогда я называю его опять User, но в нём уже работаю с моделью User и создаю нужные мне методы? В идеале пытаюсь придти к получению группы моделей одним методом (например при авторизации пользователя, загрузка полного дерева отношений к остальным моделям. Пользователь, его сервисы, тарифы, связки и т.д. Что бы был один запрос к БД, далее вызывать по цепочке $user->getService()->getName()...)

[jazzus]

RomanTrakhtenberg, как лично я делаю (не факт что правильно т.к. книжек не читал и программированию учился на тостере). В моделях получаю билдеры (т.е. конкретные запросы к бд), мелочи всякие, булеан проверки типа isActive() – потом очень наглядно везде такие запросы выглядят типа if $user->isActive() и т.д.. Логику в моделях вообще не пишу, только запросы. В контроллерах проверяю доступ, вызываю классы, в которых логика, пакую полученные данные в переменные и отдаю их на фронт. В классах вся логика, автоматизация. Плюс активно использую всякие классы, которые придумал Ларавел, типа ивентов. Называть классы можно как угодно, чтобы было понятно. Например класс для фильтрации я называю Filter. Трейты использую для общего кода в основном для моделей. Т.к. проверку isActive() можно для многих моделей делать, прописываешь в одном трейте и потом метод доступен для всех моделей где подключил. В итоге все распределяется по файлам. Но еще раз – я специально этой темой не интересовался – все по ходу пьесы так сложилось по причине личного удобства.