search

yii2mod / yii2-comments

Comments module for Yii2
MIT License
158 stars 63 forks source link

orderBy #61

Closed mitrm closed 7 years ago

mitrm commented 7 years ago

Check with the children, the other will not work https://github.com/vova07/yii2-start-comments-module/blob/master/models/Comment.php#L230

ihorchepurnyi commented 7 years ago

Can you describe your issue?

mitrm commented 7 years ago

Без сортировки ->orderBy(['parent_id' => 'ASC', 'created_at' => 'ASC']) часть комментариев удаляется. Сами попробуйте, добавьте десять комментариев, часть из них в ответы, не по порядку.

ihorchepurnyi commented 7 years ago

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

mitrm commented 7 years ago

Я добавлял примерно так: Сначала два обычных, потом дочерний к первому, потом обычный, потом дочерние и обычные уже не отображались.

ihorchepurnyi commented 7 years ago

А какая у вас версия php, и если можете скиньте сюда конфиг вашего виджета

mitrm commented 7 years ago

<?php echo \yii2mod\comments\widgets\Comment::widget([ 'model' => $challenge, 'formId' => 'form_'.$challenge::tableName().$challenge->id.Yii::$app->language, 'pjaxContainerId' => 'pjax_'.$challenge::tableName().$challenge->id, 'dataProviderConfig' => [ 'pagination' => [ 'pageSize' => false, ], ], ]); ?>

версия php 5,6 и 7

mitrm commented 7 years ago

Вот в таком порядке http://c2n.me/3JGZkld

ihorchepurnyi commented 7 years ago

Большое спасибо, попробую

ihorchepurnyi commented 7 years ago

Добавил, вот

ihorchepurnyi commented 7 years ago

Все равно не могу отловить баг, добавлял в таком же порядке как на скрине

ihorchepurnyi commented 7 years ago

yii 2.0.11, php7.1

ihorchepurnyi commented 7 years ago

Проверил на mysql 5.7, 5.6. У вас какая версия mysql?

mitrm commented 7 years ago

mysql 5,5 локальная версия

ihorchepurnyi commented 7 years ago

Проверил и на 5.5

mitrm commented 7 years ago

Именно в этом порядке не отображает шестой комментарий http://c2n.me/3JInSyi

ihorchepurnyi commented 7 years ago

Да, я именно в таком порядке добавлял комментарии

mitrm commented 7 years ago

Не в этом суть. Ты сделаешь сортировку или нет? У меня та не работает и еще у кого может не работать. Мне по сути все равно, функционал текущий меня устраивает, я могу склонить репозиторий с сортировкой в новый пакет

mitrm commented 7 years ago

Вот тут сообщение есть http://c2n.me/3JIsdQ8 а тут его нет http://c2n.me/3JIsm4J

Если добавить сортировку сообщения появляются http://c2n.me/3JIsu04

Почему так понятия не имею, но факт есть факт

ihorchepurnyi commented 7 years ago

Понял, сейчас сделаем

ihorchepurnyi commented 7 years ago

На всякий случай спрошу, у них статус active? :)

В любом случае, сортировку сейчас добавлю, просто интересно из за чего это происходит

mitrm commented 7 years ago

да, все в одно время добавлены, одинаковые сообщения. После метода buildTree эти сообщения куда то деваются

ihorchepurnyi commented 7 years ago

Запушил https://github.com/yii2mod/yii2-comments/commit/2aada349fe44f3e0d075580806c4781c4b606c93

Релиз создавать пока не буду, обнови пожалуйста у себя пакет на версию dev-master

ihorchepurnyi commented 7 years ago

Спасибо за issue

mitrm commented 7 years ago

спасибо