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

[LavIgor]

staiki, поправит автор расширения.
Остаётся ждать.

[southklad]

Админ не ограничен в размере загружаемого файла.

Перечитываю тему, а меня как админа ограничивает, странно

Отправлено спустя 9 минут 57 секунд:
Прочитал всю тему, так и не нашел почему файлы с русскими именами большим весом не идут на форум?

[Sumanai]

Перечитываю тему, а меня как админа ограничивает, странно

Отключите javascript, и тогда максимальный размер файла будет ограничен конфигурацией PHP, но узнаете вы это только после загрузки.

Отправлено спустя 13 минут 29 секунд:
А, вот, вспомнил, уже писал. Тут это ошибкой не посчитали.
Так что исправил в phpBBex 2, а тут пусть считают это поведение правильным.

[Sumanai]

Вот теперь ошибка. Если в качестве причины удаления сообщения указать длинное слово без пробелов, то в модераторской будет немного странный вид:

[LavIgor]

в модераторской будет немного странный вид

Т.е. немного выпирает таблица?
А как иначе? Куда бы ни впихнуть длиннющий текст, разметка будет испорчена.
Пути решения:
1. Скрывать лишнее (тогда на мобильных не всё можно будет прочитать).
2. Разрешить разрывы слов.
3. Оставить широкие таблицы.

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

[Sumanai]

Куда бы ни впихнуть длиннющий текст, разметка будет испорчена.

Вообще- то нет, в 3.1 в большинстве мест это не работает- word-wrap: break-word;

Разрешить разрывы слов.

Уже. Только таблицы как всегда работают по своему. Нужно выставлять table-layout: fixed;, а после с любовью вручную выставлять ширину всех колонок.

[Sumanai]

В общем это снова я. При вот этих настройках в конфигах:

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

@define('PHPBB_INSTALLED', true);
@define('PHPBB_DISPLAY_LOAD_TIME', true);
@define('DEBUG', true);
// @define('DEBUG_CONTAINER', true);

Во всплывающей подсказке, появляющуюся по наведению на время генерации страницы в футере, типа Time: 0.095s, время SQL time: всегда равно нулю. Проблемный код присутствует во всех драйверах БД \phpbb\db\driver\, поэтому самому репортить и коммитить лень.
Вот первая ошибка:

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

			// EXPLAIN only in extra debug mode
			if (defined('DEBUG'))
			{
				$this->sql_report('start', $query);
			}
			else if (defined('PHPBB_DISPLAY_LOAD_TIME'))
			{
				$this->curtime = microtime(true);
			}

else if тут быть не должно, нужен просто if.
Вот вторая

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

				if (defined('DEBUG'))
				{
					$this->sql_report('stop', $query);
				}
				else if (defined('PHPBB_DISPLAY_LOAD_TIME'))
				{
					$this->sql_time += microtime(true) - $this->curtime;
				}

Тоже самое.
Жаль, что PHP не поддерживает что нибудь типа шаблонов или примесей, а то куча дублирующегося кода даже в этих, недавно переписанных драйверах БД.

[rxu]

куча дублирующегося кода даже в этих, недавно переписанных драйверах БД.

Их особенно не переписывали, дублирование кода - известная для разработчиков проблема в phpBB.

[Sumanai]

Их особенно не переписывали

В принципе да, к примеру, _sql_transaction одинаков с точностью до байта
Хотя я вот не знаю, как там убрать дублирование без оверхеда на вызов функций и прочих трюков.

[rxu]

else if тут быть не должно, нужен просто if

Что-то здесь не то. if не должен быть, т.к. sql_report('start') инициализирует $this->curtime, делать это дважды нет смысла.

[Sumanai]

т.к. sql_report('start') инициализирует $this->curtime

Где? Не вижу кода.

[rxu]

Где?

https://github.com/phpbb/phpbb/blob/3.1 ... #L986-L987

[southklad]

Не знаю баг или нет, но в Управление вложениями вес картинок один, а по факту другой.

[Sumanai]

Точно, не заметил модификатор приватности, думал, эта функция переопределяется. Надо бы по заметнее именовать функции, чтобы различались не только _.
Ну значит ещё лучше, удаляем дубликат.

[rxu]

Ну значит ещё лучше, удаляем дубликат.

Так не получится, ибо дебаг может быть отключен, а отображение времени загрузки включено.