Skip to content

(SIV34) Unread posts separator

Featured Replies

  • Author

1.0.1

* исправлена ошибка с разделением первых сообщений на странице

  • 2 months later...

Нашел баг - при создании новой темы разделитель отображается сверху самого первого поста темы. Причем, не везде, что странно, а только в некоторых разделах, зависимость не могу найти.

Edited by Strategius

При отключении PinPosts баг пропадает. В чем проблема там кроется?

Странно, где может быть конфликт в хуках?... Разделитель появляется сверху первого поста в ново-созданной теме, но только при включенном хуке Pinposts. Не подскажете как это пофиксить, пожалуйста? Неприятный баг.

  • Author

post_count в pinPosts получает не порядковой номер сообщения в теме а его ид, поэтому в хуке первое сообщение в теме может определиться некорректно.

post_count в pinPosts получает не порядковой номер сообщения в теме а его ид

А как бы это исправить? Нашел кстати очень старую версию хука для 3.1.4, там логика другая - закрепленному сообщению присваивается наименьший порядковый номер в зависимости от страницы, на которой находится сообщение, т.е.

 

$index++;
$post_data[$id]['post']['post_count'] = $index;

Не подскажете, как такое же реализовать в версии для 3.4?

 

Вот та версия для 3.1.4 -

  • Author

Вот та версия для 3.1.4

Это платное приложение.

Это платное приложение.

Это первая версия хука еще от другого автора, он ее выложил в свободном доступе 26.11.2009, на оф. ресурсе: http://lastdragon.ru/projects/ipb3-pinposts

 

Поковырял функцию присвоения ID, не получается... При целиковой замене блока post_count, первому закрепленному сообщению присваивается фиксированный ID 1, и если закрепить другое сообщение, будет два ID 1. Как это адаптировать для 3.4, не подскажите, пожалуйста?

Подскажите, а может решением проблемы бага с хуком (SIV34) Unread Posts Separator отсюда будет получение ID первого сообщения Sql запросом? Или это будет слишком нагружать сервер, если тема будет скажем на 300 страниц?

  • Author

Не будет, тем более если вы не можете исправить логику в хуке для исключения этого поста, с sql запросом вы точно ничего не сделаете. Эти хуки не совместимы.

Эти хуки не совместимы

:( А переделка логики для исключения поста сделает хуки совместимыми, ведь там проблема только в этом?

  • Author

А переделка логики для исключения поста сделает хуки совместимыми, ведь там проблема только в этом?

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

Вижу для вас это не будет проблемой

Это самая большая проблема, которая передо мной сейчас стоит..) Если Вам не сложно, не могли бы указать на проблемное место в хуке, где нужно копать? Насколько понял, не обязательно заморачиваться с запросом для post_count?

  • Author

					if( $topic['lastMarked'] <= $post['post']['post']['post_date'] )
					{

Пропустить сообщение если это закрепленное сообщение в теме.

если это закрепленное сообщение в теме

А для этого подойдет это условие?

 

<if test="pinned:|:$post['post']['_isPinned']"></if>

Create an account or sign in to comment

Account

Navigation

Search

Search

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.