Как настроить фильтр и сортировку по связанным таблицам?

Question

[Радость моя]

Привет друзья!

У меня есть модель User и UserSearch(для gridview). в User есть связь с таблицей балансов Balance:

/**
     * Внешняя связь с таблицей балансов
     */
    public function getBalance()
    {
        return $this->hasMany(Balance::className(), ['user_id' => 'id'])->sum('summ');
    }

т.е. в гридвью я вывожу текущую сумму баланса юзера. Вывод работает, но как мне сделать чтобы работал и фильтр для поиска и сортировка по балансу? В UserSearch сделал так:

// Если в фильтре был передан баланс
            if($this->balance) {
                $query->joinWith(['balance' => function ($q) {
                    $q->where('sum(balance.summ) = ' . $this->balance);
                }]);
            }

и получаю бяку:

common\models\User has no relation named "balance".

в User и UserSearch public $balance объявлены, в function rules добавлены.

Answer 1

[Максим Федоров]

Как сделать фильтрацию и сортировку по связанным данным довольно подробно описано в документации
что касается конкретно Вашей ошибки - то тут ничего удивительного у Вас есть свойство $balance и реляция с таким же именем, которая так же доступна как свойство. В результате у Вас получается два свойства с одним именем - одно объявленное Вами, второе - "магическое" на основе реляции.
Выход: или переименуйте реляцию или свое свойство

Comments

[Радость моя]

Максим, спасибо за подсказку. А нужно ли мне вообще свойство balance в UserSearch, если в самой табл нет такого столбца, а есть только связь с табл балансов?

[Максим Федоров]

Радость моя, если нужно фильтровать по балансу - тогда нужно какое-то свойство которое будет содержать условие для фильтрации

[Радость моя]

Максим Федоров, значит public $balance нужно оставить в UserSearch а связь назвать например
getFkBalance() тогда что мне писать в
$query->joinWith(['balance' ?

[Максим Федоров]

Радость моя, новое название связи

[Радость моя]

Максим Федоров, поменял название связи на fkbalance, теперь так же ругается только на это имя.
Вот весь UserSearch: https://gist.github.com/tihoho/fd4b6bd1e76c37a2a95...
А связь объявляю так:

public function getFkbalance()
    {
        return $this->hasMany(Balance::className(), ['user_id' => 'id'])->sum('summ');
    }

я снова что-то забыл?

[Максим Федоров]

Радость моя, ну и нафига вы объявили свойство fkbalance? я же в ответе Вам четко написал что у Вас два свойства с одинаковым именем

[Радость моя]

Максим Федоров, ну так я его и обозвал fkbalance чтобы было 2 balance и fkbalance. Значит я не понял ответа :(

Answer 2

[Дмитрий]

Доброе утро.
Покажите, именно, как Вы фильтруете в поисковой модели.
Если настроить нужно по разным моделям фильтрацию, то Вам надо ещё создать псевдонимы таблиц.
Покажу на своём примере.
Мне требовалось в маршрутах сделать поиск по городам.
В модели "маршруты" создал публичное свойство "$city_id;". Так же есть связующая таблица, где маршруты связаны с городами. В самом запросе добавил связь и joinWith().

$query = Routes::find()
            ->from(['r' => Routes::tableName()])
            ->with(['city', 'routesCities', 'price'])
            ->joinWith([
                'city' => function(ActiveQuery $query){
                    $query->from(['rc1' => Cities::tableName()]);
                }
            ])
            ->joinWith([
                'routesCities' => function(ActiveQuery $query){
                    $query->from(['rc2' => RoutesCities::tableName()]);
                }
            ])

В этом запросе я делаю псевдоним для таблицы маршрутов, для таблицы city и связующей таблицы.
Соответственно получаю три псевдонима "r" - маршруты, "rc1" - города, "rc2" - связующая таблица, хранится привязка города к маршруту.
И тогда в фильтрации пишу следующее:

$query->andFilterWhere([
            'r.id' => $this->id,
            /*****/
            'rc1.id' => $this->city_id, // фильтрую по id города.
        ]);

И в GridView делаю саму колонку фильтра, вывожу через запятую города, через которые проходит маршрут.

[
                'attribute' => 'city_id',
                'label' => "Города",
                'filter' => Cities::find()->select(['title', 'id'])->indexBy('id')->orderBy(['title' => SORT_ASC])->column(),
                'value' => function(Routes $routes){
                     return implode(',', ArrayHelper::map($routes->city, 'id', 'title'));
                }
            ],

Answer 3

[Максим Тимофеев]

в User и UserSearch public $balance объявлены

вот тут и проблема, так как Вы объявили в UserSearch public $balance то и получили в $this->balance строку или число, а не связь как ожидается. Так что просто переименуйте или связь или переменную публичную. Что бы они по разному назывались.

Comments

[Радость моя]

Максим, к сожалению я запутался какие переменные должны быть объявлены и в каких моделях. Подскажите пожалуйста по пунктам (ниже) что сделать из списка ниже (что убрать что добавить и что переименовать):

Что сейчас (могу полностью методы скинуть, но там можно запутаться) :
1. модель User имеет в rules среди прочего ['balance', 'number'], и не имеет никаких public $balance.
2. так же эта модель (User) имеет объявленную связь для подсчета стоимости балансов:

public function getBalance()
    {
        return $this->hasMany(Balance::className(), ['user_id' => 'id'])->sum('summ');
    }

3. модель UserSearch наследуется от User и имеет в rules [['balance'], 'number'] и не имеет никаких public $balance.
4. в методе UserSearch::search я попытался подтянуть возможность фильтрации по балансам так:

$query->andFilterWhere(['like', 'email', $this->email])
                ->andFilterWhere(['like', 'fio', $this->fio])
                ->andFilterWhere(['like', 'phone_mobile', $this->phone_mobile]);
            
            $query->joinWith(['balance' => function($query) { $query->from(['balance' => Balance::className()]); }]);

5. в gridview columns пытаюсь вывести так:

[
            'attribute' => 'balance',
            'value' => function($model) {
                return ( $model->balance ?: '0.00' ) . ' р.';
            },
            'label' => 'Баланс',
        ]

6. получаю ошибку common\models\User has no relation named "balance".

если убираю $query->joinWith(['... то gridview отображается.

Очень нужна помощь.

[Максим Тимофеев]

модель User имеет в rules среди прочего ['balance', 'number'], и не имеет никаких

То есть это поле в таблице юзер?
Тогда зачем Вы сделали public function getBalance() переименуйте. Сделайте допустим public function getBalanceRalation()

Вообще правильнее будет если Вы опишите структуру этих двух таблиц и код моделей. Так как есть у меня подозрения, что Вы там нахимичели что-то не очень правильное.

[Радость моя]

Максим Тимофеев,
это User:
https://gist.github.com/tihoho/5066c3c77807e9e528a...
это UserSearch:
https://gist.github.com/tihoho/470563e9e2de02dfed7...
структуры:
https://gist.github.com/tihoho/2929403a46e5612abf7...

[Максим Тимофеев]

С user нет поля balance, зачем там ['balance', 'number']? Убираем.
В UserSearch 'balance' переносим из ['balance', 'number'] в

[['id', 'status', 'created_at', 'updated_at', 'phone_internal', 'position_id', 'depart_id'], 'integer'],

функцию со связью public function getBalance() вы уже переименовали, ок.
вот это

$query->joinWith(['balance' => function($query) { $query->from(['balance' => Balance::className()]); }]);

не там где нужно и можно проще:
$query->joinWith('balancexx');
поставить между query и дата провайдером, короче на 52 строку
Ну и потом если нужно что-то фильтровать, допустим так

$query->andFilterWhere([
                'balance.summ' => $this->balance, //обратите внимание, что тут balance слева это имя таблицы

Ну и не стоило делать summ типом float. Лучше сделать integer. А в модели Balance добавить события beforeSave и afterFind где делить и умножать на 100. Тогда можно будет норм работать с выборкой диапазона. Если надо отфильтровать данные от одной суммы до другой.

[Радость моя]

Максим Тимофеев, спасибо за развернутый ответ. сделал как вы сказали:
User:
https://gist.github.com/tihoho/7e638df7f5b6def9819...
UserSearch:
https://gist.github.com/tihoho/1da2550bb4349f3b0e8...
получаю исключение:
common\models\User has no relation named "balancexx".

[Максим Тимофеев]

->sum('summ')
что у Вас там делает? Должна же связь возвращаться, а не результат запроса, правильно так:

public function getBalancexx()
    {
        return $this->hasOne(Balance::className(), ['user_id' => 'id']);
    }

[Радость моя]

Максим Тимофеев, а агрегацию мне тогда где нужно делать?

[Радость моя]

Максим Тимофеев, и тогда наверно надо делать hasMany т.к. в табл с балансами там куча записей которые надо будет где-то суммировать по полю summ, а не одна

[Максим Тимофеев]

Радость моя, Если сделаете hasMany тогда надо будет делать группировку по id. Иначе у Вас будет много одинаковых user c подходящими balance
Вообще есть отличная статья про serachModel с различными способами и ситуациями:
www.yiiframework.com/wiki/780
крайне рекомендую прочитать 5 раз прежде чем двигаться дальше

[Радость моя]

Максим Тимофеев, в примерах этой вики нет ниодного варианта с агрегацией, к сожалению.

[Максим Тимофеев]

Радость моя, Вот еще:
www.yiiframework.com/wiki/851/yii2-gridview-sortin...