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

[rxu]

Тогда какое-то другое решение нужно, видимо.

[nissin]

Посмотрел ещё раз код повнимательней, теперь понял, исключение прописано для progress_bar.
Да, нужно по другому проверку делать. Пул действительно лучше отозвать.

[rxu]

nissin, а если так попробовать https://github.com/phpbb/phpbb/pull/4489/files

[nissin]

Вроде нормально, про progress_bar только забыл.

[rxu]

про progress_bar только забыл

Почему? Там в ссылках редиректа тоже хэш есть.

Отправлено спустя 18 секунд:
Условие добавил.

[nissin]

В принципе можно и прогресс бару хэш добавить.
И ещё часто вызывается generate_link_hash, хотя вроде функция не сильно тяжелая.

[rxu]

Да сам по себе прогресс бар ничего ведь не делает, зачем его проверять? Хэш на редиректе имеется, его должно хватить.

[nissin]

Ну в принципе да, согласен.

[nissin]

Re: Вышел phpBB 3.1.10 [обсуждаем]
Ох уж эти проверки формы, вставили везде, где можно, но без должной проверки.

[rxu]

поэтому можно и так заменить

Склоняюсь к тому, чтобы сделать PR только в ветку 3.2+ в виде

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

$context_vars = $this->context->get_data_ref()['.'][0]; 

а с 3.1 уже не заморачиваться

[nissin]

В этом случае он данные копировать будет каждый раз, а нужно что-бы вложенный массив по ссылке пошёл.

[rxu]

Так get_data_ref() возвращает ссылку.

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

/**
    * Returns a reference to template data array.
    *
    * This function is public so that template renderer may invoke it.
    * Users should alter template variables via functions in \phpbb\template\template.
    *
    * Note: modifying returned array will affect data stored in the context.
    *
    * @return array template data
    */
    public function &get_data_ref() 

[nissin]

А $this->context->get_data_ref()['.'][0] выдаст массив.

[KimIV]

В шаблоне styles\prosilver\template\ucp_main_subscribed.html
В строке 33 {L_LAST_POST} {L_POST_BY_AUTHOR} {forumrow.LAST_POST_AUTHOR_FULL} &laquo; <a href="{topicrow.U_LAST_POST}">{forumrow.LAST_POST_TIME}</a>
Вместо topicrow.U_LAST_POST должно быть forumrow.U_LAST_POST

Данный косяк приводит к тому, что в мобильной версии не работает ссылка на последнее сообщение.

[LavIgor]

KimIV, https://github.com/phpbb/phpbb/pull/4506