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

[rxu]

forum_action=&

Это почему так?

[Sheer]

Кто ж их знает
ну гуру mcp.php?&f=47&i=main&action=&mode=forum_view&sd=d&sk=r&st=0&start=100

[nissin]

В общем я скачал 3.0RC1, там искомый код уже присутствует, значит идёт из 2.1 development ветки (которая в последующем стала 3.0 вместо 2.2).
Обсуждение разработки 2.1, как и сам репозиторий найти не могу. Вынесли похоже.

[Sheer]

rxu, Закомменть тут. Посмотрим, что будет.

[rxu]

Посмотрим, что будет.

Смотрим.

Отправлено спустя 14 минут 9 секунд:
Понятно. Заходим в тему, переходим в модераторский раздел, меняем сортировку (например, делаем время размещения по убыванию), теперь слева в меню кликаем "Просмотреть форум", получаем ту же сортировку по убыванию (по времени последнего сообщения).

[Sheer]

Нифига не понял. Без правок (код не закомментирован) - зашел в тему - сортировка по возрастанию. Посмотреть форум - сортировка по убыванию.

Отправлено спустя 4 минуты 19 секунд:
Закомментировал. Зашел в тему - сортировка по Автор. Посмотреть форум - сортировка по Время размещения

Отправлено спустя 27 секунд:
Не вижу косяков.

[rxu]

М-да. При модерировании форума по убыванию - сортировка по умолчанию. Значит, всё верно.

Отправлено спустя 2 минуты 19 секунд:
Sheer, не забудь, что там должно выполняться условие

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

$merge_select = ($action == 'merge_select' || $action == 'merge_topic' || $action == 'merge_topics') ? true : false;

Т.е. речь идет только о слиянии тем, видимо. При остальных операциях всё должно сортироваться нормально.

[Sheer]

Я не забыл. Ща посмотрю что при слиянии/переносе.

Отправлено спустя 6 минут 44 секунды:
Зашел в тему. Сортировка по Автор - Отметил сообщение - Выбрал Перенести сообщение - Вверху

Сообщение
Вы должны выбрать тему для объединения сообщений.

Ткнул Посмотреть форум - Сортировка Время сообщения, Вернулся назад, ткнул Выбрать тему - Сортировка Время сообщения
Все корректно работает.

[nissin]

Смотрел код, в общем Sheer прав.
Какое-то непонятное наследие, непонятно какого бага (и был ли он вообще).
Параметры сортировки никак не попадают из topic_view в forum_view.

[Sheer]

Может он и был ранее, ибо

Affects Version/s: 3.0.RC7
Fix Version/s: 3.0.0

Матерь божья!

[rxu]

Интересно, а на 3.0 баг воспроизводится?

[Sheer]

Самое интересное, что за столько лет никто не обратил внимание на невозможность смены сортировки при переносе/слиянии. Вчера на одном форуме пытались слить две темы - запарились искать тему-приемник, ибо была очень старая, но вот если бы можно было отсортировать по названию, нашли бы мигом.

Отправлено спустя 52 секунды:

на 3.0 баг воспроизводится

Ща проверю.

Отправлено спустя 3 минуты 45 секунд:
Если закомментить

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

unset($_POST['sk'], $_POST['sd'], $_REQUEST['sk'], $_REQUEST['sd']); 

никакого бага нет. Все точно так же, как и тут Сообщаем об ошибках phpBB 3.1.x разработчикам

[LavIgor]

Искал. Хотел найти. Нашёл.
https://github.com/phpbb/phpbb/commit/a ... 78a3daR314
Код был добавлен Людовиком в 2003 году, в предыдущем commit искомой строчки нет.

[rxu]

OK, https://tracker.phpbb.com/browse/PHPBB3-14106

[nissin]

Смотрю вот этот commit, получается этот код был изначально:
Если посмотреть mcp_topic.html в этом commit, то видно, что раньше выбор темы делался через отправку POST
<input class="btnlite" type="submit" name="action[merge_select]" value="{L_SELECT_TOPIC}" /></td>
и параметры сортировки действительно попадали в forum_view, чуть позже эту кнопку заменили ссылкой.