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

Сообщение **rxu** » 24.10.2017 16:37

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

Сообщение **Sheer** » 29.10.2017 1:28

Если выбран поисковый механизм 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** » 29.10.2017 7:44

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

Сообщение **Sheer** » 29.10.2017 14:53

rxu писал(а): 29.10.2017 7:44уже и тикет

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

rxu писал(а): 29.10.2017 7:44Для инфы

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

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

Сообщение **rxu** » 29.10.2017 15:38

Sheer писал(а): 29.10.2017 14:55с очисткой кеша результатов поиска тоже не все нормально

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

Сообщение **Sheer** » 29.10.2017 16:12

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

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

Сообщение **Sheer** » 29.10.2017 23:53

В 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 . '&amp;action=details&amp;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 . '&amp;action=' . $action_name . '&amp;hash=' . generate_link_hash($action_name) . '&amp;id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['STYLE_' . ($style['style_active'] ? 'DE' : '') . 'ACTIVATE']
			);

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

			// Uninstall
			$actions[] = array(
				'U_ACTION'	=> $this->u_action . '&amp;action=uninstall&amp;hash=' . generate_link_hash('uninstall') . '&amp;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 . '&amp;action=details&amp;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 . '&amp;action=' . $action_name . '&amp;hash=' . generate_link_hash($action_name) . '&amp;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 . '&amp;action=export&amp;hash=' . generate_link_hash('export') . '&amp;id=' . $style['style_id'],
				'L_ACTION'	=> $this->user->lang['EXPORT']
			); */

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

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

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

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

Результат

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

Сообщение **rxu** » 04.11.2017 15:16

Sheer писал(а): 29.10.2017 1:28Если выбран поисковый механизм MySQL Fulltext

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

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

Sheer писал(а): 29.10.2017 23:53при переходе по ссылке Информация в списке установленных стилей в случае, если отсутствует папка с файлами стиля

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

Сообщение **Sheer** » 04.11.2017 15:49

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

Сообщение **rxu** » 04.11.2017 15:52

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

Сообщение **Sheer** » 04.11.2017 15:55

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

Сообщение **Татьяна5** » 04.11.2017 16:05

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

Сообщение **Sheer** » 04.11.2017 16:10

Сообщение **rxu** » 04.11.2017 18:25

Татьяна5 писал(а): 04.11.2017 16:05Если программа не обрабатывает ошибку, то это баг

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

Сообщение **Татьяна5** » 04.11.2017 18:29

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

phpBB Guru - Официальная русская поддержка форума phpBB

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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