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

[Pazh]

выставлено вложенность цитирования 1 цитата, в полном ответе при цитировании сообщения, содержащего цитату из Обзора темы в поле ответа вставляется двойная цитата, т.е. цитата с двумя вложенными цитатами. При отправке или при предпросмотре лишние цитаты убираются, но это крайне не удобно. Аналогично это потом тянется и в расширение Быстрый ответ. Причина - в скрытый div с текстом сообщения, который используется js кодом цитирования движок вставляет ПОЛНЫй текст сообщений (без учета настройки вложенности цитирования - хотя это было бы логично и к тому же уменьшило бы вес отдаваемой страницы)
Тестируем расширение-fix:

pazh_quotedepth-1.0.0.zip

В идеале еще и bbcode ref оттуда удалять

[Sheer]

Не уменьшается счетчик сообщений при удалении сообщения, когда оно является единственным в теме, во всех остальных случаях счетчик изменяется корректно. total posts (всего сообщений) - проблема счетчика?
https://tracker.phpbb.com/browse/PHPBB3-16003

[Pazh]

в файле adm\style\permissions.js найти

https://github.com/phpbb/phpbb/pull/5070

Они прикалываются так? Элементарный косяк в настройках прав доступа к форуму, обнаруженный полтора года назад, а они запланировали его исправление в phpbb4 - бараны

[Pazh]

Ошибка в микроданных в файле styles\prosilver\template\navbar_header.html
в блоке хлебных крошек <li class="breadcrumbs" itemscope itemtype="http://schema.org/BreadcrumbList">...</li> нужно заменить все

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

<span itemprop="title">

на

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

<span itemprop="name">

В спецификации https://schema.org/BreadcrumbList нет свойства title

[rxu]

нужно заменить

Уже. будет в 3.2.6.
https://github.com/phpbb/phpbb/pull/5549/files

[Pazh]

Никто не замечал что при обновлении с 3.0 не срабатывает в миграции \phpbb\db\migration\data\v310\rename_too_long_indexes.php
удаление индекса

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

			'drop_keys' => array(
				$this->table_prefix . 'search_wordmatch' => array(
					'unq_mtch',
				),
			),

Хотя в логах миграция отработала, новый индекс создается, а старый не убивается - может кто проверить или обратить внимание при обновлении...

[Sheer]

Замечал. STK его убивает.

[Pazh]

Да, STK находит и убивает, но должно по процедуре обновления убиваться.

[Sheer]

Должно, но... А еще остается board_dst в таблице _config

[Pazh]

Sheer, да, тоже заметил. может где в коде миграции опечатка?

[Sheer]

Не знаю. Насчет board_dst в миграции просто нет sql-запроса на удаление
timezone.php --> public function update_timezones

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

		// Update board default timezone
		$sql = 'UPDATE ' . $this->table_prefix . "config
			SET config_value = '" . $this->convert_phpbb30_timezone($this->config['board_timezone'], $this->config['board_dst']) . "'
			WHERE config_name = 'board_timezone'";
		$this->sql_query($sql);

Вот на этом она и кончается. По идее далее должен следовать запрос на удаление строки из таблицы.
А насчет индекса... в миграции rename_too_long_indexes.php только функция update_schema(), в других (смотрел не все) есть еще revert_schema(). Может в этом причина.

[Pazh]

в миграции rename_too_long_indexes.php

в этой миграции удаление старого и создание нового индекса идет в одном массиве - новый индекс создается, значит в массив вход происходит при обновлении, но вот drop_key не срабатывает

[rxu]

новый индекс создается, а старый не убивается

Похоже, что ошибка в миграторе. Уникальные индексы не удаляются, даже если в миграции прописана такая инструкция.

Отправлено спустя 3 минуты 29 секунд:
Pazh, попробовать в файле \phpbb\db\tools\tools.php заменить строку

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

if (!$this->sql_index_exists($table, $index_name))

на

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

if (!$this->sql_index_exists($table, $index_name) && !$this->sql_unique_index_exists($table, $index_name))

должно помочь.

Отправлено спустя 23 минуты 52 секунды:
https://github.com/phpbb/phpbb/pull/5594/files

Отправлено спустя 21 минуту 4 секунды:

А еще остается board_dst в таблице _config

А нигде нет инструкции, что она должна быть удалена. И кстати, где в миграциях добавляется 'board_timezone' - тоже не нашел.

[Sheer]

А нигде нет инструкции, что она должна быть удалена

И что? В "чистой установке" board_dst отсутствует

где в миграциях добавляется 'board_timezone'

.
\phpbb\db\migration\data\v310\timezone.php
Найти

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

			WHERE config_name = 'board_timezone'";
		$this->sql_query($sql);

Добавить после

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

		$sql = 'DELETE FROM ' . $table_prefix . 'config WHERE config_name = \'board_dst\'';
		$this->sql_query($sql);

[rxu]

И что?

ну что - поэтому и остается.

Добавить после

Это можно и добавить, если только оно специально не оставлено для какой-то обратной совместимости.

board_timezone

Вопрос - откуда оно берется при конвертации старых версий? Инструкции по добавлению нигде нет!