Сообщаем об ошибках phpBB 3.1.x разработчикам

[Shredder]

На пограничных потому, что объявление в форуме are51 всего одно. Чем больше объявлений или важных тем в форуме, тем больше одинаковых тем на разных страницах)

[rxu]

Чем больше объявлений или важных тем в форуме, тем больше одинаковых тем на разных страницах)

Только на пограничных. На остальных не нахожу.

[Shredder]

Да, именно на пограничных. По логике именно так и должно быть.

[rxu]

По логике именно так и должно быть.

В коде сможете показать логику?

[Shredder]

Вряд ли.

[rxu]

Возможный фикс:

Код: Выделить всё

OPEN
viewforum.php

FIND
if ($start > $topics_count / 2)

REPLACE WITH
if ($start > $page_topics_count / 2)

BEFORE, ADD
$page_topics_count = $topics_count - sizeof($announcement_list) - sizeof($global_announce_forums);

[Shredder]

После применения фикса разбиение на страницы с виду становится нормальным, но количество тем в форуме (около "Отметить все темы как прочтённые") становится неправильным, и затрагивает сразу все форумы.

[rxu]

Тогда по-другому, исправил в посте.

[Shredder]

Так тоже не помогает.

Насколько вижу из кода, $announcement_list - это массив "всех объявлений просматриваемого форума плюс всех важных тем из всех форумов". Это следует из части запроса:

Код: Выделить всё

		'WHERE'		=> '(t.forum_id = ' . $forum_id . '
				AND t.topic_type = ' . POST_ANNOUNCE . ') OR
			(' . $db->sql_in_set('t.forum_id', $g_forum_ary) . '
				AND t.topic_type = ' . POST_GLOBAL . ')',

где:

Код: Выделить всё

	$g_forum_ary = $auth->acl_getf('f_read', true);
	$g_forum_ary = array_unique(array_keys($g_forum_ary));

- все форумы, которые пользователю разрешено читать: http://www.vvvas.ru/it/phpbb/3.0/docs/a ... l#acl_getf

А нам нужно исключить из количества тем, передаваемых в пагинацию, только те объявления и важные темы из просматриваемого форума, которые находятся в нём самом. Видимо, для этого придётся создавать отдельный блок кода со SQL-запросом и другими проверками.

Добавлено спустя 16 минут 52 секунды:
И ещё довольно странно выглядит ситуация, когда перед переопределением количества тем на страницу в форуме, целых два раза используется количество из общих настроек - $config['topics_per_page']:
Это строка 212:

Код: Выделить всё

// Is a forum specific topic count required?
if ($forum_data['forum_topics_per_page'])
{
	$config['topics_per_page'] = $forum_data['forum_topics_per_page'];
}

А $config['topics_per_page'] используется на строках 153 и 155
В 3.0.x это переопределение шло ДО всех использований $config['topics_per_page']

[hd321kbps]

Интересно то, что если ставишь картинку форума. В Firefox 33.0.2 не показывает, а вот в Opera 25 все хорошо!

Добавлено спустя 10 минут 30 секунд:
В Safari 5.1.7 тот, который на Windows, тоже косяки!

[Leff]

При попытке установки на локальный хостинг выдает ошибку

Warning: Unexpected character in input: '\' (ASCII=92) state=1 in Z:\home\localhost\www\spm\install\index.php on line 110

Warning: Unexpected character in input: '\' (ASCII=92) state=1 in Z:\home\localhost\www\spm\install\index.php on line 110

Parse error: syntax error, unexpected T_STRING in Z:\home\localhost\www\spm\install\index.php on line 110

[Татьяна5]

Leff, какая версия php на локалке? Подходит не ниже 5.3.3

[Anvar]

На php 5.3.28 тоже работает, пока не выявлено ошибок..
Скорей всего если проверку на совместимость не пройдёт, то установить не получится, ошибка будет в оформленном виде.

На одном хостинге 5.2.x, жду когда хостер перенесёт на новый сервер с необходимой версией что-бы обновить действующий форум.

[Leff]

Версия 5.3.13

[Sheer]

Leff, что используете в качестве localhost ? Денвер? Обновите.