Страница 25 из 30

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

Добавлено: 04.03.2019 11:36
Pazh
выставлено вложенность цитирования 1 цитата, в полном ответе при цитировании сообщения, содержащего цитату из Обзора темы в поле ответа вставляется двойная цитата, т.е. цитата с двумя вложенными цитатами. При отправке или при предпросмотре лишние цитаты убираются, но это крайне не удобно. Аналогично это потом тянется и в расширение Быстрый ответ. Причина - в скрытый div с текстом сообщения, который используется js кодом цитирования движок вставляет ПОЛНЫй текст сообщений (без учета настройки вложенности цитирования - хотя это было бы логично и к тому же уменьшило бы вес отдаваемой страницы)
Тестируем расширение-fix:
pazh_quotedepth-1.0.0.zip
(8.42 КБ) 149 скачиваний
В идеале еще и bbcode ref оттуда удалять

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

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

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

Добавлено: 04.04.2019 0:07
Pazh
rxu писал(а): 02.01.2018 9:55
Pazh писал(а): 22.11.2017 12:23в файле adm\style\permissions.js найти
https://github.com/phpbb/phpbb/pull/5070
Они прикалываются так? :facepalm: :facepalm: :facepalm: Элементарный косяк в настройках прав доступа к форуму, обнаруженный полтора года назад, а они запланировали его исправление в phpbb4 - бараны

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

Добавлено: 24.04.2019 14:31
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

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

Добавлено: 24.04.2019 15:56
rxu
Pazh писал(а): 24.04.2019 14:31нужно заменить
Уже. будет в 3.2.6.
https://github.com/phpbb/phpbb/pull/5549/files

Re: Ошибки и исправления в версии 3.2.x

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

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

			'drop_keys' => array(
				$this->table_prefix . 'search_wordmatch' => array(
					'unq_mtch',
				),
			),
Хотя в логах миграция отработала, новый индекс создается, а старый не убивается - может кто проверить или обратить внимание при обновлении...

Re: Ошибки и исправления в версии 3.2.x

Добавлено: 17.05.2019 15:00
Sheer
Замечал. STK его убивает.

Re: Ошибки и исправления в версии 3.2.x

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

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

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

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

Добавлено: 17.05.2019 20:24
Pazh
Sheer, да, тоже заметил. может где в коде миграции опечатка?

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

Добавлено: 17.05.2019 20:44
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(). Может в этом причина.

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

Добавлено: 17.05.2019 20:49
Pazh
Sheer писал(а): 17.05.2019 20:44 в миграции rename_too_long_indexes.php
в этой миграции удаление старого и создание нового индекса идет в одном массиве - новый индекс создается, значит в массив вход происходит при обновлении, но вот drop_key не срабатывает

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

Добавлено: 17.05.2019 21:43
rxu
Pazh писал(а): 17.05.2019 14:50 новый индекс создается, а старый не убивается
Похоже, что ошибка в миграторе. Уникальные индексы не удаляются, даже если в миграции прописана такая инструкция.

Отправлено спустя 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 секунды:
Sheer писал(а): 17.05.2019 19:15 А еще остается board_dst в таблице _config
А нигде нет инструкции, что она должна быть удалена. И кстати, где в миграциях добавляется 'board_timezone' - тоже не нашел.

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

Добавлено: 17.05.2019 22:49
Sheer
rxu писал(а): 17.05.2019 22:31 А нигде нет инструкции, что она должна быть удалена
И что? В "чистой установке" board_dst отсутствует
rxu писал(а): 17.05.2019 22:31 где в миграциях добавляется '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);

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

Добавлено: 17.05.2019 22:54
rxu
Sheer писал(а): 17.05.2019 22:49И что?
ну что - поэтому и остается.
Sheer писал(а): 17.05.2019 22:49Добавить после
Это можно и добавить, если только оно специально не оставлено для какой-то обратной совместимости.
Sheer писал(а): 17.05.2019 22:49board_timezone
Вопрос - откуда оно берется при конвертации старых версий? Инструкции по добавлению нигде нет!