Продолжаю исследовать плагин комментирования Quip и находить там баги.
Теперь не работала система уведомлений (notify).
Как выглядело:
Ставим в любой теме галочку «Подписаться на новые сообщения в этой теме». Потом заходим в любую другую тему и опа, а галочка и там стоит чудесным образом. Т.е. подписавшись на любую тему вы автоматически видите себя подписанным на все темы.
Как я вылечил:
файл \core\components\quip\controllers\web\ThreadReply.php
строка примерно 215, заменяем
/** @var quipCommentNotify $notify */
$notify = $this->modx->getObject('quipCommentNotify',array(
'email' => $this->modx->user->Profile->get('email'),
'thread' => $this->thread,
));
на
/** @var quipCommentNotify $notify */
$notify = $this->modx->getObject('quipCommentNotify',array(
'email' => $this->modx->user->Profile->get('email'),
'thread' => $this->thread->get('name'),
));
Есть еще аналогичный кусочек кода ниже в этом файле, думаю с ним надо проделать тоже.
P.S. Возможно, что где то есть еще подобное… если замечу — дополню пост. Что самое интересное, гугление не выявило подобных проблем у пользователей
