Страница 33 из 38
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 21:07
rxu
Тогда какое-то другое решение нужно, видимо.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 21:15
nissin
Посмотрел ещё раз код повнимательней, теперь понял, исключение прописано для progress_bar.
Да, нужно по другому проверку делать. Пул действительно лучше отозвать.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 21:49
rxu
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 22:16
nissin
Вроде нормально, про progress_bar только забыл.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 22:21
rxu
nissin писал(а): про progress_bar только забыл
Почему? Там в ссылках редиректа тоже хэш есть.
Отправлено спустя 18 секунд:
Условие добавил.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 22:29
nissin
В принципе можно и прогресс бару хэш добавить.
И ещё часто вызывается generate_link_hash, хотя вроде функция не сильно тяжелая.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 22:33
rxu
Да сам по себе прогресс бар ничего ведь не делает, зачем его проверять? Хэш на редиректе имеется, его должно хватить.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 18.10.2016 22:36
nissin
Ну в принципе да, согласен.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 21.10.2016 8:40
nissin
Re: Вышел phpBB 3.1.10 [обсуждаем]
Ох уж эти проверки формы, вставили везде, где можно, но без должной проверки.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 24.10.2016 20:43
rxu
nissin писал(а): поэтому можно и так заменить
Склоняюсь к тому, чтобы сделать PR только в ветку 3.2+ в виде
Код: Выделить всё
$context_vars = $this->context->get_data_ref()['.'][0];
а с 3.1 уже не заморачиваться
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 24.10.2016 20:45
nissin
В этом случае он данные копировать будет каждый раз, а нужно что-бы вложенный массив по ссылке пошёл.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 24.10.2016 20:47
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()
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 24.10.2016 20:53
nissin
А $this->context->get_data_ref()['.'][0] выдаст массив.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 08.11.2016 15:52
KimIV
В шаблоне styles\prosilver\template\ucp_main_subscribed.html
В строке 33 {L_LAST_POST} {L_POST_BY_AUTHOR} {forumrow.LAST_POST_AUTHOR_FULL} « <a href="{topicrow.U_LAST_POST}">{forumrow.LAST_POST_TIME}</a>
Вместо topicrow.U_LAST_POST должно быть forumrow.U_LAST_POST
Данный косяк приводит к тому, что в мобильной версии не работает ссылка на последнее сообщение.
Re: Сообщаем об ошибках phpBB 3.1.x разработчикам
Добавлено: 08.11.2016 16:55
LavIgor