[MDV] Threads From Node In Thread 1.0.1

[recruty]

recruty добавил(а) новый ресурс:

[MDV] Threads From Node In Thread - Добавляет вывод тем из раздела, в котором создана тема.

Данный плагин добавляет слева список тем из раздела, в котором создана тема.

Плагин добавляет в навбар кнопку для открытия(активируется при наведении)
Кнопка ведёт на раздел, в котором создана тема.

Посмотреть вложение 136854


При активации этой кнопки, появляется блок слева, в котором выводятся все темы из раздела.

В данном случае, выводятся все темы из раздела "Мусор"

Посмотреть вложение 136855



Совместимость только со Стандартным стилём...

Узнать больше об этом ресурсе...

 

[Bams]

Дефолтная тема, не работает, xf2.2

 

[recruty]

Дефолтная тема, не работает, xf2.2

Покажите проблему.

 

[Bams]

Покажите проблему.

как ?) нет ошибок, кнопки тоже нет ) Настроек у плагина ведь нет, я установил и все

 

[ZloyDed]

Проверил у себя всё работает, xf 2.2.8 patch 1, там ли ты кнопку ищешь? Она только в темах выводится, и посмотри все ли модификации применились

 

[Bams]

Проверил у себя всё работает, xf 2.2.8 patch 1, там ли ты кнопку ищешь? Она только в темах выводится, и посмотри все ли модификации применились

В теме смотрю ) модификаторы применились, там их 2

 

[recruty]

В теме смотрю ) модификаторы применились, там их 2

Дайте, пожалуйста, ссылку на форум.

 

[Bams]

Видимо конфликт с чем-то, потому-что на чистом форуме все работает. Стоит

[cXF] Logo in Navigation​

возможно из-за этого

 

[ZloyDed]

Видимо конфликт с чем-то

Отключи все плагины кроме этого

 

[recruty]

Видимо конфликт с чем-то

Вы уверены что модификации применились?

Видимо конфликт с чем-то, потому-что на чистом форуме все работает. Стоит

[cXF] Logo in Navigation​

возможно из-за этого

Скорее всего. Мой плагин цепляется к p-nav-smallLogo, а этот плагин, я уверен, тоже редактирует это.

 

[Bams]

Вы уверены что модификации применились?

Посмотреть вложение 136860

Да

Скорее всего. Мой плагин цепляется к p-nav-smallLogo, а этот плагин, я уверен, тоже редактирует это.

Видимо так и есть

 

[recruty]

recruty обновил(а) ресурс [MDV] Threads From Node In Thread новой записью:

Исправление ошибок

Исправление ошибок

Узнать больше об этом обновлении...

 

[Atikin]

MDV\ThreadsFromNodeInThread\XF\Pub\Controller\Thread:

    public function actionIndex(ParameterBag $params)
    {
        $threads_node = parent::actionIndex($params);

        $finder = \XF::finder('XF:Thread');
        $node_id = $threads_node->getParam('thread')->node_id;
        $threads = $finder->with('Forum', true)->where('node_id', $node_id)->order('post_date', 'DESC')->fetch();
        $threads_node->setParam('threads', $threads);

        return $threads_node;
    }

Что насчет проверок прав на просмотр темы перед тем, как передать её в шаблон? В данном случае пользователи будут видеть в списке абсолютно все темы из раздела, даже удаленные и на модерации. В том же XF\Pub\Controller\Forum::actionForum можно было посмотреть, как все фильтруется.
Да пагинация не помешала бы, ибо будет в разделе 100500 тем, так они все и будут подгружаться каждый раз.
Кроме того,XF\Pub\Controller\Thread::actionIndex может вернуть не XF\Mvc\Reply\View, а XF\Mvc\Reply\Error (или вообще что угодно, если какой-либо плагин добавит такое), поэтому вызов setParam будет вызывать ошибку.

К модификации mdv_tfnit_add_node_threads_in_thread_view тоже есть вопрос. Зачем там подключается jQuery? В XF он есть из коробки.

В Less CSS зачем-то прописываются взятые с потолка значения для отступов, размеров, цветов и прочего, хотя для этого существуют настройки стиля.