Уважаемые пользователи!
Напоминаем, что с 7 ноября 2020 года phpBB Group прекратила выпуск обновлений для phpBB версии 3.2.
С учетом этого, рекомендуется обновить конференции до версии 3.3.
Сайт официальной русской поддержки phpBB Guru продолжит поддержку phpBB 3.2 до отдельного объявления.

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

Проблемы с установкой или работой phpBB 3.1.x? Получите помощь здесь!
С 1 июля 2018 года phpBB Group прекращает поддержку phpBB версии 3.1.
Сайт официальной русской поддержки phpBB Guru продолжит поддержку phpBB 3.1 до 1 января 2019 года.
Подробнее: Окончание поддержки phpBB 3.1.
Правила форума
Местная Конституция | Шаблон запроса | Документация (phpBB3) | Мини [FAQ] по phpBB3.1.x | FAQ-3 (phpbb3) | Как задавать вопросы | Как устанавливать расширения

Ваш вопрос может быть удален без объяснения причин, если на него есть ответы по приведённым ссылкам (а вы рискуете получить предупреждение ;) ).
Shredder
Former team member
Сообщения: 2203
Стаж: 14 лет 1 месяц
Благодарил (а): 114 раз
Поблагодарили: 126 раз

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

Сообщение Shredder »

На пограничных потому, что объявление в форуме are51 всего одно. Чем больше объявлений или важных тем в форуме, тем больше одинаковых тем на разных страницах)
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15898
Стаж: 16 лет 8 месяцев
Откуда: Красноярск
Благодарил (а): 464 раза
Поблагодарили: 1555 раз

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

Сообщение rxu »

Shredder писал(а):Чем больше объявлений или важных тем в форуме, тем больше одинаковых тем на разных страницах)
Только на пограничных. На остальных не нахожу.
Изображение
Shredder
Former team member
Сообщения: 2203
Стаж: 14 лет 1 месяц
Благодарил (а): 114 раз
Поблагодарили: 126 раз

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

Сообщение Shredder »

Да, именно на пограничных. По логике именно так и должно быть.
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15898
Стаж: 16 лет 8 месяцев
Откуда: Красноярск
Благодарил (а): 464 раза
Поблагодарили: 1555 раз

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

Сообщение rxu »

Shredder писал(а):По логике именно так и должно быть.
В коде сможете показать логику?
Изображение
Shredder
Former team member
Сообщения: 2203
Стаж: 14 лет 1 месяц
Благодарил (а): 114 раз
Поблагодарили: 126 раз

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

Сообщение Shredder »

Вряд ли.
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15898
Стаж: 16 лет 8 месяцев
Откуда: Красноярск
Благодарил (а): 464 раза
Поблагодарили: 1555 раз

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

Сообщение 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
Former team member
Сообщения: 2203
Стаж: 14 лет 1 месяц
Благодарил (а): 114 раз
Поблагодарили: 126 раз

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

Сообщение Shredder »

После применения фикса разбиение на страницы с виду становится нормальным, но количество тем в форуме (около "Отметить все темы как прочтённые") становится неправильным, и затрагивает сразу все форумы.
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 15898
Стаж: 16 лет 8 месяцев
Откуда: Красноярск
Благодарил (а): 464 раза
Поблагодарили: 1555 раз

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

Сообщение rxu »

Тогда по-другому, исправил в посте.
Изображение
Shredder
Former team member
Сообщения: 2203
Стаж: 14 лет 1 месяц
Благодарил (а): 114 раз
Поблагодарили: 126 раз

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

Сообщение 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
phpBB 2.0.2
Сообщения: 334
Стаж: 10 лет 9 месяцев
Откуда: Россия, Крым, Армянск
Благодарил (а): 160 раз
Поблагодарили: 347 раз

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

Сообщение hd321kbps »

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

Добавлено спустя 10 минут 30 секунд:
В Safari 5.1.7 тот, который на Windows, тоже косяки!
У вас нет необходимых прав для просмотра вложений в этом сообщении.
Leff
phpBB 1.2.0
Сообщения: 15
Стаж: 8 лет 5 месяцев

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

Сообщение 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
Поддержка
Поддержка
Сообщения: 11589
Стаж: 11 лет 5 месяцев
Благодарил (а): 142 раза
Поблагодарили: 2179 раз

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

Сообщение Татьяна5 »

Leff, какая версия php на локалке? Подходит не ниже 5.3.3
Anvar
Former team member
Сообщения: 1965
Стаж: 12 лет 10 месяцев
Благодарил (а): 57 раз
Поблагодарили: 620 раз

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

Сообщение Anvar »

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

На одном хостинге 5.2.x, жду когда хостер перенесёт на новый сервер с необходимой версией что-бы обновить действующий форум.
Leff
phpBB 1.2.0
Сообщения: 15
Стаж: 8 лет 5 месяцев

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

Сообщение Leff »

Версия 5.3.13
Аватара пользователя
Sheer
Former team member
Сообщения: 12113
Стаж: 15 лет 11 месяцев
Откуда: Калининград не Кенигсберг
Благодарил (а): 41 раз
Поблагодарили: 1715 раз

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

Сообщение Sheer »

Leff, что используете в качестве localhost ? Денвер? Обновите.
Изображение
Общие ошибки новичков (07.11.2005) & Как задавать вопросы
Мини FAQ
Если ничто другое не помогает, прочтите, наконец, инструкцию!
"Никакая инструкция не может перечислить всех обязанностей должностного лица, предусмотреть все отдельные случаи и дать вперёд соответствующие указания, а поэтому господа инженеры должны проявить инициативу и, руководствуясь знаниями своей специальности и пользой дела, принять все усилия для оправдания своего назначения".
Циркуляр Морского технического комитета №15 от 29.11.1910 г.

Вернуться в «Поддержка phpBB 3.1.x»