Yii: Ошибка пагинации при together = true. Как исправить?

Question

[Тимур]

Бьюсь уже третьи сутки. Гуглил до тошноты. Будьте добры, направьте в нужно русло.


Ситуация: на сайте есть пользователи. Пользователи имеют определенные характеристики, которые хранятся в связанных таблицах. Нужно найти пользователей, соответствующих определенным характеристикам.


Проблема: отражена ниже на скриншотах

Модель Users

public function relations() {
    return array(
        'usersMotivations' => array(self::HAS_MANY, 'UsersMotivations', 'userID'),
    );
}

Действие контроллера

public function actionIndex() {
    $criteria = new CDbCriteria;
    $criteria->with = array('usersMotivations');
    $criteria->addInCondition('motivationID',array(1,2));
    $criteria->together = true;

    $dataProvider = new CActiveDataProvider('Users',array(
        'criteria'=>$criteria,
        'pagination'=>array('pageSize'=>10)
    ));

    $this->render('index', array( 'dataProvider' => $dataProvider));
}

Представление

$this->widget('zii.widgets.CListView', array(
    'dataProvider'=>$dataProvider,
    'itemView'=>'_view',
));

И вот здесь происходит самое интересное. При 'pagination'=>array('pageSize'=>10) страница с результатами выглядит так:



При 'pagination'=>array('pageSize'=>2) один из результатов дублируется на двух страницах



Теперь если убрать строку $criteria->addInCondition('motivationID',array(1,2)); то один из результатов дублируется, а другой — исчезает вообще



Опытным путем выяснил, что виной всему $criteria->together = true; и если его убрать, то с пагинацией становится все нормально, но тогда перестает работать условие поиска

$criteria->addInCondition('motivationID',array(1,2));

Как быть, подскажите пожалуйста?

Answer 1

[Darivush]

Попробуйте

$criteria->group='t.user_id';

где t.user_id PK модели Users.
Плюс т.к. у вас стоит задача не выбрать все мотивации у юзера можно сделать такой финт: добавить релейшен

'usersMotivation' => array(self::HAS_ONE, 'UsersMotivations', 'userID'),

И использовать тот же самый критерий

$criteria = new CDbCriteria;
$criteria->with = array('usersMotivation');
$criteria->addInCondition('motivationID',array(1,2));

если не ошибаюсь заработает даже без

$criteria->together = true;

Comments

[Darivush]

У вас проблема возникает из-за того что AR подгружает для выбираемых пользователей дополнительные мотивации, что в ответе sql выдает по 1 строке для каждой мотивации, сверху накладывается лимит от пейджера и получаются прыжки. попробуйте добавить еще мотивацию для пользователя ХАКЕРЕНОК и ваш пейджер начнет глючить еще больше

[Тимур]

Не подумайте ничего плохого, но помимо плюса в карму, я бы вас еще поцеловал :D (шутка)
Я 3 дня бился! А надо было то всего $criteria->group='t.id';

[Тимур]

Огромное-при-огромное СПАСИБО!!! И отдельное спасибо за объяснение причины такого поведения AR

[Darivush]

Одно но в таком решение в релейшене usersMotivations у вас не будет массива со всеми мотивациями пользователя и в случаи чего придется их подгружать сабзапросом

[Тимур]

В моем случае использовать HAS_ONE нельзя, т.к. у меня usersMotivations — ассоциативная таблица. Я убрал лишний код, чтоб не загружать лишней инфой. Релейшн у меня такой

'usersMotivations' => array(self::HAS_MANY, 'UsersMotivations', 'userID'),
'listMotivation' => array(self::HAS_MANY, 'ListMotivation', array('motivationID'=>'id'),'through'=>'usersMotivations'),

и список мотиваций пользователя выводится на странице результатов поиска. Мой случай $criteria->group='t.id';
Еще раз спасибо!

[Darivush]

Вы немного не поняли — это не изменение вашего релейшена, а дополнительный под конкретную задачу, чтобы sql не пыхтел над group_by.

[Тимур]

ага! Понял! Только в таком случае не

$criteria->addInCondition('motivationID',array(1,2));

а просто

$criteria->addCondition('motivationID = 1');

и да, работает без together = true;

Блин, гигантское вам СПАСИБИЩЕ!!! Этот пробел знаний в моей голове съедал кучу времени! Низкий поклон!

Answer 2

[enchikiben]

Это из-за связи один ко многим. Вам надо как то так:

public function actionIndex() {
    $criteria = new CDbCriteria;
$criteria->addCondition(" (SELECT COUNT(*) FROM `UsersMotivations` WHERE userID = t.ID AND `motivationID` IN (:motivationID)  ) > 0 ");
$criteria->params[":motivationID"] = implode(',',array(1,2));

    $dataProvider = new CActiveDataProvider('Users',array(
        'criteria'=>$criteria,
        'pagination'=>array('pageSize'=>10)
    ));

    $this->render('index', array( 'dataProvider' => $dataProvider));
}

Comments

[Тимур]

Пара вопросов:
1) Верно ли, что вот этот вложенный запрос «SELECT COUNT(*) FROM `UsersMotivations` WHERE userID = t.ID AND `motivationID` IN (:motivationID)» будет выполняться для каждой записи? По-моему это как-то не лучший вариант
2) Есть ли варианты, более близкие к AR?

[Тимур]

И еще вопрос: вы сказали, что 'то из-за связи один ко многим. А можно подробнее? Почему? Я не пойму что именно происходит. У меня задача не «сделать, чтоб работало», а понять и научитсья