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

[rxu]

https://github.com/phpbb/phpbb/pull/5013/files

[Sheer]

Если выбран поисковый механизм MySQL Fulltext и
1) поисковые индексы отсутствуют, то при попытке поиска форум падает с фатальной онибкой

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

Общая ошибка
SQL ERROR [ mysqli ]

Can't find FULLTEXT index matching the column list [1191]

SQL

SELECT SQL_CALC_FOUND_ROWS p.post_id FROM phpbb_posts p WHERE MATCH (p.post_subject, p.post_text) AGAINST ('+правильно ' IN BOOLEAN MODE) AND (p.post_visibility = 1 OR p.forum_id IN (1, 2)) ORDER BY p.post_time DESC LIMIT 250

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 999
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 193
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/mysql_base.php
LINE: 45
CALL: phpbb\db\driver\mysqli->sql_query()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 270
CALL: phpbb\db\driver\mysql_base->_sql_query_limit()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 321
CALL: phpbb\db\driver\driver->sql_query_limit()

FILE: [ROOT]/phpbb/search/fulltext_mysql.php
LINE: 594
CALL: phpbb\db\driver\factory->sql_query_limit()

FILE: [ROOT]/search.php
LINE: 593
CALL: phpbb\search\fulltext_mysql->keyword_search()

Предлагаемое решение
Открыть ./phpbb/search/fulltext_mysql.php
Найти

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

		$result = $this->db->sql_query_limit($sql, $this->config['search_block_size'], $start);

Перед вставить

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

		$this->db->sql_return_on_error(true); // Fix bug with SQL error if empty index

Найти

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

		if (!$result_count)

Заменить на

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

		if (!$result_count && sizeof($id_ary))

2)
Если поисковые индексы созданы, то при попытке осуществить поиск Только в текстах сообщений, форум падает с фатальной ошибкой

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

Общая ошибка
SQL ERROR [ mysqli ]

Can't find FULLTEXT index matching the column list [1191]

SQL

SELECT SQL_CALC_FOUND_ROWS p.post_id FROM phpbb_posts p WHERE MATCH (p.post_text) AGAINST ('+правильно ' IN BOOLEAN MODE) AND (p.post_visibility = 1 OR p.forum_id IN (1, 2)) ORDER BY p.post_time DESC LIMIT 250

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 999
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 193
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/mysql_base.php
LINE: 45
CALL: phpbb\db\driver\mysqli->sql_query()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 270
CALL: phpbb\db\driver\mysql_base->_sql_query_limit()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 321
CALL: phpbb\db\driver\driver->sql_query_limit()

FILE: [ROOT]/phpbb/search/fulltext_mysql.php
LINE: 594
CALL: phpbb\db\driver\factory->sql_query_limit()

FILE: [ROOT]/search.php
LINE: 593
CALL: phpbb\search\fulltext_mysql->keyword_search()

Предлагаемое решение
Найти

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

		if (sizeof($alter_list))
		{
			foreach ($alter_list as $alter)
			{
				$this->db->sql_query('ALTER TABLE ' . POSTS_TABLE . ' ' . implode(', ', $alter));
			}
		}

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

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

		if (!isset($this->stats['post_text']))
		{
			$this->db->sql_query('ALTER TABLE ' . POSTS_TABLE . ' ADD FULLTEXT post_text (post_text)');
		}

Найти

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

		if (isset($this->stats['post_content']))
		{
			$alter[] = 'DROP INDEX post_content';
		}

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

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

		if (isset($this->stats['post_text']))
		{
			$alter[] = 'DROP INDEX post_text';
		}

Найти

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

		return isset($this->stats['post_subject']) && isset($this->stats['post_content']);

Заменить на

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

		return isset($this->stats['post_subject']) && isset($this->stats['post_content']) && isset($this->stats['post_text']);

Найти

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

				if ($row['Key_name'] == 'post_subject')
				{
					$this->stats['post_subject'] = $row;
				}

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

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

				else if ($row['Key_name'] == 'post_text')
				{
					$this->stats['post_text'] = $row;
				}

[rxu]

Sheer, для второго уже и тикет есть https://tracker.phpbb.com/browse/PHPBB3-15224
Для инфы, версия мускла и тип таблиц БД?

[Sheer]

уже и тикет

Не нашел. На оффе что-то было невразумительное про 3.1

Для инфы

Верно, забыл.
Сервер базы данных: MySQL(i) 5.7.16, тип таблиц InnoDB
Версия phpBB: 3.2.1 (чистая установка)
Тип базы данных: mysqli
Версия php: 7.1.0

Отправлено спустя 2 минуты 4 секунды:
А вообще там с очисткой кеша результатов поиска тоже не все нормально, ну я так думаю.

[rxu]

с очисткой кеша результатов поиска тоже не все нормально

Что, где, как?

[Sheer]

Пример: Есть сообщение, содержащее некоторое слово, например abracadabra, если его задать как аргумент поиска, то оно попадет в кеш результатов и таблицу search_results. Если это слово тут же удалить из сообщения, или удалить это сообщение, то слово будет удалено из индекса, но если его снова задать как аргумент поиска, то оно будет найдено, хотя нигде в БД его нет, оно останется только в search_results и не будет удалено из кеша. Это будет происходить до тех пор, пока не будет очищен кеш
Результат вышеописанных действий.

Скриншот сделанный 2017-10-29 в 15.11.26-fullpage.png

[Sheer]

В ACP -- Персонализация -- Стили при переходе по ссылке Информация в списке установленных стилей в случае, если отсутствует папка с файлами стиля, получаем ошибку, например

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

[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 2650: file(./../styles/AllanStyle-SUBSILVER/style.cfg): failed to open stream: No such file or directory
[phpBB Debug] PHP Warning: in file [ROOT]/includes/functions.php on line 2653: Invalid argument supplied for foreach()

Решение:
Открыть includes/acp/acp_styles.php
Найти

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

		$style['_shown'] = true;

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

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

		$exists = file_exists(htmlspecialchars($this->phpbb_root_path . $this->styles_path_absolute . '/' . $style['style_path']) . '/style.cfg');
 		if (!$exists)
 		{
 			$style['_note'] = $this->user->lang['UNAVIALABLE'];
 		}

Найти

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

		if ($style['style_id'])
		{
			// Style is installed

			// Details
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=details&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['DETAILS']
			);

			// Activate/Deactive
			$action_name = ($style['style_active'] ? 'de' : '') . 'activate';

			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=' . $action_name . '&hash=' . generate_link_hash($action_name) . '&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['STYLE_' . ($style['style_active'] ? 'DE' : '') . 'ACTIVATE']
			);

/*			// Export
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=export&hash=' . generate_link_hash('export') . '&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['EXPORT']
			); */

			// Uninstall
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=uninstall&hash=' . generate_link_hash('uninstall') . '&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['STYLE_UNINSTALL']
			);

			// Preview
			$actions[] = array(
				'U_ACTION'	=> append_sid($this->phpbb_root_path . 'index.' . $this->php_ext, 'style=' . $style['style_id']),
				'L_ACTION'	=> $this->user->lang['PREVIEW']
			);
		}

Заменить на

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

		if ($style['style_id'])
		{
			// Style is installed

			// Details
			if ($exists)
			{
				$actions[] = array(
					'U_ACTION'	=> $this->u_action . '&action=details&id=' . $style['style_id'],
					'L_ACTION'	=> $this->user->lang['DETAILS']
				);

				// Activate/Deactive
				$action_name = ($style['style_active'] ? 'de' : '') . 'activate';

				$actions[] = array(
					'U_ACTION'	=> $this->u_action . '&action=' . $action_name . '&hash=' . generate_link_hash($action_name) . '&id=' . $style['style_id'],
					'L_ACTION'	=> $this->user->lang['STYLE_' . ($style['style_active'] ? 'DE' : '') . 'ACTIVATE']
				);

				// Preview
				$actions[] = array(
					'U_ACTION'	=> append_sid($this->phpbb_root_path . 'index.' . $this->php_ext, 'style=' . $style['style_id']),
					'L_ACTION'	=> $this->user->lang['PREVIEW']
				);
			}

/*			// Export
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=export&hash=' . generate_link_hash('export') . '&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['EXPORT']
			); */

			// Uninstall
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&action=uninstall&hash=' . generate_link_hash('uninstall') . '&id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['STYLE_UNINSTALL']
			);
		}

В файлы локализации acp/styles.php добавить

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

	'UNAVIALABLE'				=> 'Отсутствуют файлы стиля',

Результат

Скриншот сделанный 2017-10-29 в 22.53.14-fullpage.png

[rxu]

Если выбран поисковый механизм MySQL Fulltext

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

Отправлено спустя 29 секунд:

при переходе по ссылке Информация в списке установленных стилей в случае, если отсутствует папка с файлами стиля

Т.е. это в случае, если у юзера не всё хорошо с логикой? А надо это исправлять в коде?

[Sheer]

Вот вроде у мня с логикой как-то вроде. Да вот снес как-то папку со стилем не задумываясь и забыл. А потом дай думаю гляну, что за стиль такой, а тут на вот те

[rxu]

Понятно. Давай оставим код как есть, в данном конкретном случае это не баг кода

[Sheer]

Как бе не бак. Но вот отсутствует защита от дурака. А должна быть. Ибо дурней среди админов не считано.

[Татьяна5]

Если программа не обрабатывает ошибку, то это баг

[Sheer]

[rxu]

Если программа не обрабатывает ошибку, то это баг

Если удалить еще несколько файлов ядра, тоже будет ошибка. Это не баг.

[Татьяна5]

Файлы стилей и файлы расширений частью ядра не являются. Но отсутствие файлов расширений обрабатывается, а отсутствие файлов стиля почему-то нет