fulltext_search::prepareIndexes() - Code Metrics - Inspection of "Merge pull request #4238 from albertlast/SearchFas..." - albertlast/SMF2.1 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — release-2.1 ( 6f6d35...abeae7 )

by Mathias

created 2017-08-23 04:36 UTC

fulltext_search::prepareIndexes() C

↳ Parent: fulltext_search

Complexity

Conditions	11
Paths	28

Size

Total Lines	32
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	11
eloc	15
nc	28
nop	4
dl	0
loc	32
rs	5.2653
c	0
b	0
f	0

How to fix Complexity

<?php

/**
 * Simple Machines Forum (SMF)
 *
 * @package SMF
 * @author Simple Machines http://www.simplemachines.org
 * @copyright 2017 Simple Machines and individual contributors
 * @license http://www.simplemachines.org/about/smf/license.php BSD
 *
 * @version 2.1 Beta 4
 */

if (!defined('SMF'))
	die('No direct access...');

/**
 * Class fulltext_search
 * Used for fulltext index searching
 */
class fulltext_search extends search_api
{
	/**
	 * @var array Which words are banned
	 */
	protected $bannedWords = array();

	/**
	 * @var int The minimum word length
	 */
	protected $min_word_length = 4;

	/**
	 * @var array Which databases support this method?
	 */
	protected $supported_databases = array('mysql');

	/**
	 * The constructor function
	 */
	public function __construct()
	{
		global $modSettings, $db_type;

		// Is this database supported?
		if (!in_array($db_type, $this->supported_databases))
		{
			$this->is_supported = false;
			return;
		}

		$this->bannedWords = empty($modSettings['search_banned_words']) ? array() : explode(',', $modSettings['search_banned_words']);
		$this->min_word_length = $this->_getMinWordLength();
	}

	/**
	 * {@inheritDoc}
	 */
	public function supportsMethod($methodName, $query_params = null)
	{
		switch ($methodName)
		{
			case 'searchSort':
			case 'prepareIndexes':
			case 'indexedWordQuery':
				return true;
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			// All other methods, too bad dunno you.
			default:
				return false;
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
		}
	}

	/**
	 * fulltext_search::_getMinWordLength()
	 *
	 * What is the minimum word length full text supports?
	 *
	 * @return int The minimum word length
	 */
	protected function _getMinWordLength()
	{
		global $smcFunc;

		// Try to determine the minimum number of letters for a fulltext search.
		$request = $smcFunc['db_search_query']('max_fulltext_length', '
			SHOW VARIABLES
			LIKE {string:fulltext_minimum_word_length}',
			array(
				'fulltext_minimum_word_length' => 'ft_min_word_len',
			)
		);
		if ($request !== false && $smcFunc['db_num_rows']($request) == 1)

		{
			list (, $min_word_length) = $smcFunc['db_fetch_row']($request);
			$smcFunc['db_free_result']($request);
		}
		// 4 is the MySQL default...
		else
			$min_word_length = 4;

		return $min_word_length;
	}

	/**
	 * {@inheritDoc}
	 */
	public function searchSort($a, $b)
	{
		global $excludedWords, $smcFunc;

		$x = $smcFunc['strlen']($a) - (in_array($a, $excludedWords) ? 1000 : 0);
		$y = $smcFunc['strlen']($b) - (in_array($b, $excludedWords) ? 1000 : 0);

		return $x < $y ? 1 : ($x > $y ? -1 : 0);
	}

	/**
	 * {@inheritDoc}
	 */
	public function prepareIndexes($word, array &$wordsSearch, array &$wordsExclude, $isExcluded)
	{
		global $modSettings, $smcFunc;

		$subwords = text2words($word, null, false);

		if (empty($modSettings['search_force_index']))
		{
			// A boolean capable search engine and not forced to only use an index, we may use a non indexed search
			// this is harder on the server so we are restrictive here
			if (count($subwords) > 1 && preg_match('~[.:@$]~', $word))
			{
				// using special characters that a full index would ignore and the remaining words are short which would also be ignored
				if (($smcFunc['strlen'](current($subwords)) < $this->min_word_length) && ($smcFunc['strlen'](next($subwords)) < $this->min_word_length))
				{
					$wordsSearch['words'][] = trim($word, "/*- ");
					$wordsSearch['complex_words'][] = count($subwords) === 1 ? $word : '"' . $word . '"';
				}
			}
			elseif ($smcFunc['strlen'](trim($word, "/*- ")) < $this->min_word_length)
			{
				// short words have feelings too
				$wordsSearch['words'][] = trim($word, "/*- ");
				$wordsSearch['complex_words'][] = count($subwords) === 1 ? $word : '"' . $word . '"';
			}
		}

		$fulltextWord = count($subwords) === 1 ? $word : '"' . $word . '"';
		$wordsSearch['indexed_words'][] = $fulltextWord;
		if ($isExcluded)
			$wordsExclude[] = $fulltextWord;
	}

	/**
	 * {@inheritDoc}
	 */
	public function indexedWordQuery(array $words, array $search_data)
	{
		global $modSettings, $smcFunc;

		$query_select = array(
			'id_msg' => 'm.id_msg',
		);
		$query_where = array();
		$query_params = $search_data['params'];

		if( $smcFunc['db_title'] == "PostgreSQL")
			$modSettings['search_simple_fulltext'] = true;

		if ($query_params['id_search'])
			$query_select['id_search'] = '{int:id_search}';

		$count = 0;
		if (empty($modSettings['search_simple_fulltext']))
			foreach ($words['words'] as $regularWord)

			{
				$query_where[] = 'm.body' . (in_array($regularWord, $query_params['excluded_words']) ? ' NOT' : '') . (empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:complex_body_' . $count . '}';
				$query_params['complex_body_' . $count++] = empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? '%' . strtr($regularWord, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?|{}()])/'), array('[$1]'), $regularWord), '\\\'') . '[[:>:]]';
			}

		if ($query_params['user_query'])
			$query_where[] = '{raw:user_query}';
		if ($query_params['board_query'])
			$query_where[] = 'm.id_board {raw:board_query}';

		if ($query_params['topic'])
			$query_where[] = 'm.id_topic = {int:topic}';
		if ($query_params['min_msg_id'])
			$query_where[] = 'm.id_msg >= {int:min_msg_id}';
		if ($query_params['max_msg_id'])
			$query_where[] = 'm.id_msg <= {int:max_msg_id}';

		$count = 0;
		if (!empty($query_params['excluded_phrases']) && empty($modSettings['search_force_index']))

			foreach ($query_params['excluded_phrases'] as $phrase)
			{
				$query_where[] = 'subject NOT ' . (empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:exclude_subject_phrase_' . $count . '}';
				$query_params['exclude_subject_phrase_' . $count++] = empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? '%' . strtr($phrase, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?|{}()])/'), array('[$1]'), $phrase), '\\\'') . '[[:>:]]';
			}
		$count = 0;
		if (!empty($query_params['excluded_subject_words']) && empty($modSettings['search_force_index']))

			foreach ($query_params['excluded_subject_words'] as $excludedWord)
			{
				$query_where[] = 'subject NOT ' . (empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:exclude_subject_words_' . $count . '}';
				$query_params['exclude_subject_words_' . $count++] = empty($modSettings['search_match_words']) || $search_data['no_regexp'] ? '%' . strtr($excludedWord, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?|{}()])/'), array('[$1]'), $excludedWord), '\\\'') . '[[:>:]]';
			}

		if (!empty($modSettings['search_simple_fulltext']))
		{
			if($smcFunc['db_title'] == "PostgreSQL")

			{
				$language_ftx = $smcFunc['db_search_language']();

				$query_where[] = 'to_tsvector({string:language_ftx},body) @@ to_tsquery({string:language_ftx},{string:body_match})';
				$query_params['language_ftx'] = $language_ftx;
			}
			else
				$query_where[] = 'MATCH (body) AGAINST ({string:body_match})';
			$query_params['body_match'] = implode(' ', array_diff($words['indexed_words'], $query_params['excluded_index_words']));
		}
		else
		{
			$query_params['boolean_match'] = '';

			// remove any indexed words that are used in the complex body search terms
			$words['indexed_words'] = array_diff($words['indexed_words'], $words['complex_words']);

			if($smcFunc['db_title'] == "PostgreSQL"){
				$row = 0;
				foreach ($words['indexed_words'] as $fulltextWord) {
					$query_params['boolean_match'] .= ($row <> 0 ? '&' : '');
					$query_params['boolean_match'] .= (in_array($fulltextWord, $query_params['excluded_index_words']) ? '!' : '') . $fulltextWord . ' ';
					$row++;
				}
			}
			else
				foreach ($words['indexed_words'] as $fulltextWord)
					$query_params['boolean_match'] .= (in_array($fulltextWord, $query_params['excluded_index_words']) ? '-' : '+') . $fulltextWord . ' ';

			$query_params['boolean_match'] = substr($query_params['boolean_match'], 0, -1);

			// if we have bool terms to search, add them in
			if ($query_params['boolean_match']) {

				if($smcFunc['db_title'] == "PostgreSQL")
				{
					$language_ftx = $smcFunc['db_search_language']();

					$query_where[] = 'to_tsvector({string:language_ftx},body) @@ to_tsquery({string:language_ftx},{string:boolean_match})';
					$query_params['language_ftx'] = $language_ftx;
				}
				else
					$query_where[] = 'MATCH (body) AGAINST ({string:boolean_match} IN BOOLEAN MODE)';
			}

		}

		$ignoreRequest = $smcFunc['db_search_query']('insert_into_log_messages_fulltext', ($smcFunc['db_support_ignore'] ? ( '
			INSERT IGNORE INTO {db_prefix}' . $search_data['insert_into'] . '
				(' . implode(', ', array_keys($query_select)) . ')') : '') . '
			SELECT ' . implode(', ', $query_select) . '
			FROM {db_prefix}messages AS m
			WHERE ' . implode('
				AND ', $query_where) . (empty($search_data['max_results']) ? '' : '
			LIMIT ' . ($search_data['max_results'] - $search_data['indexed_results'])),
			$query_params
		);

		return $ignoreRequest;
	}
}

?>


1		<?php
2
3		/**
4		* Simple Machines Forum (SMF)
5		*
6		* @package SMF
7		* @author Simple Machines http://www.simplemachines.org
8		* @copyright 2017 Simple Machines and individual contributors
9		* @license http://www.simplemachines.org/about/smf/license.php BSD
10		*
11		* @version 2.1 Beta 4
12		*/
13
14		if (!defined('SMF'))
15		die('No direct access...');
16
17		/**
18		* Class fulltext_search
19		* Used for fulltext index searching
20		*/
21		class fulltext_search extends search_api
22		{
23		/**
24		* @var array Which words are banned
25		*/
26		protected $bannedWords = array();
27
28		/**
29		* @var int The minimum word length
30		*/
31		protected $min_word_length = 4;
32
33		/**
34		* @var array Which databases support this method?
35		*/
36		protected $supported_databases = array('mysql');
37
38		/**
39		* The constructor function
40		*/
41		public function __construct()
42		{
43		global $modSettings, $db_type;
44
45		// Is this database supported?
46		if (!in_array($db_type, $this->supported_databases))
47		{
48		$this->is_supported = false;
49		return;
50		}
51
52		$this->bannedWords = empty($modSettings['search_banned_words']) ? array() : explode(',', $modSettings['search_banned_words']);
53		$this->min_word_length = $this->_getMinWordLength();
54		}
55
56		/**
57		* {@inheritDoc}
58		*/
59		public function supportsMethod($methodName, $query_params = null)
60		{
61		switch ($methodName)
62		{
63		case 'searchSort':
64		case 'prepareIndexes':
65		case 'indexedWordQuery':
66		return true;
67		break;
		0 ignored issues – show Unused Code introduced 2016-12-27 02:48 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
68
69		// All other methods, too bad dunno you.
70		default:
71		return false;
72		break;
		0 ignored issues – show Unused Code introduced 2016-12-27 02:48 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
73		}
74		}
75
76		/**
77		* fulltext_search::_getMinWordLength()
78		*
79		* What is the minimum word length full text supports?
80		*
81		* @return int The minimum word length
82		*/
83		protected function _getMinWordLength()
84		{
85		global $smcFunc;
86
87		// Try to determine the minimum number of letters for a fulltext search.
88		$request = $smcFunc['db_search_query']('max_fulltext_length', '
89		SHOW VARIABLES
90		LIKE {string:fulltext_minimum_word_length}',
91		array(
92		'fulltext_minimum_word_length' => 'ft_min_word_len',
93		)
94		);
95	View Code Duplication	if ($request !== false && $smcFunc['db_num_rows']($request) == 1)
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
96		{
97		list (, $min_word_length) = $smcFunc['db_fetch_row']($request);
98		$smcFunc['db_free_result']($request);
99		}
100		// 4 is the MySQL default...
101		else
102		$min_word_length = 4;
103
104		return $min_word_length;
105		}
106
107		/**
108		* {@inheritDoc}
109		*/
110		public function searchSort($a, $b)
111		{
112		global $excludedWords, $smcFunc;
113
114		$x = $smcFunc['strlen']($a) - (in_array($a, $excludedWords) ? 1000 : 0);
115		$y = $smcFunc['strlen']($b) - (in_array($b, $excludedWords) ? 1000 : 0);
116
117		return $x < $y ? 1 : ($x > $y ? -1 : 0);
118		}
119
120		/**
121		* {@inheritDoc}
122		*/
123		public function prepareIndexes($word, array &$wordsSearch, array &$wordsExclude, $isExcluded)
124		{
125		global $modSettings, $smcFunc;
126
127		$subwords = text2words($word, null, false);
128
129		if (empty($modSettings['search_force_index']))
130		{
131		// A boolean capable search engine and not forced to only use an index, we may use a non indexed search
132		// this is harder on the server so we are restrictive here
133		if (count($subwords) > 1 && preg_match('~[.:@$]~', $word))
134		{
135		// using special characters that a full index would ignore and the remaining words are short which would also be ignored
136		if (($smcFunc['strlen'](current($subwords)) < $this->min_word_length) && ($smcFunc['strlen'](next($subwords)) < $this->min_word_length))
137		{
138		$wordsSearch['words'][] = trim($word, "/*- ");
139		$wordsSearch['complex_words'][] = count($subwords) === 1 ? $word : '"' . $word . '"';
140		}
141		}
142		elseif ($smcFunc['strlen'](trim($word, "/*- ")) < $this->min_word_length)
143		{
144		// short words have feelings too
145		$wordsSearch['words'][] = trim($word, "/*- ");
146		$wordsSearch['complex_words'][] = count($subwords) === 1 ? $word : '"' . $word . '"';
147		}
148		}
149
150		$fulltextWord = count($subwords) === 1 ? $word : '"' . $word . '"';
151		$wordsSearch['indexed_words'][] = $fulltextWord;
152		if ($isExcluded)
153		$wordsExclude[] = $fulltextWord;
154		}
155
156		/**
157		* {@inheritDoc}
158		*/
159		public function indexedWordQuery(array $words, array $search_data)
160		{
161		global $modSettings, $smcFunc;
162
163		$query_select = array(
164		'id_msg' => 'm.id_msg',
165		);
166		$query_where = array();
167		$query_params = $search_data['params'];
168
169		if( $smcFunc['db_title'] == "PostgreSQL")
170		$modSettings['search_simple_fulltext'] = true;
171
172		if ($query_params['id_search'])
173		$query_select['id_search'] = '{int:id_search}';
174
175		$count = 0;
176		if (empty($modSettings['search_simple_fulltext']))
177	View Code Duplication	foreach ($words['words'] as $regularWord)
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
178		{
179		$query_where[] = 'm.body' . (in_array($regularWord, $query_params['excluded_words']) ? ' NOT' : '') . (empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:complex_body_' . $count . '}';
180		$query_params['complex_body_' . $count++] = empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? '%' . strtr($regularWord, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?\|{}()])/'), array('[$1]'), $regularWord), '\\\'') . '[[:>:]]';
181		}
182
183		if ($query_params['user_query'])
184		$query_where[] = '{raw:user_query}';
185		if ($query_params['board_query'])
186		$query_where[] = 'm.id_board {raw:board_query}';
187
188		if ($query_params['topic'])
189		$query_where[] = 'm.id_topic = {int:topic}';
190		if ($query_params['min_msg_id'])
191		$query_where[] = 'm.id_msg >= {int:min_msg_id}';
192		if ($query_params['max_msg_id'])
193		$query_where[] = 'm.id_msg <= {int:max_msg_id}';
194
195		$count = 0;
196	View Code Duplication	if (!empty($query_params['excluded_phrases']) && empty($modSettings['search_force_index']))
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
197		foreach ($query_params['excluded_phrases'] as $phrase)
198		{
199		$query_where[] = 'subject NOT ' . (empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:exclude_subject_phrase_' . $count . '}';
200		$query_params['exclude_subject_phrase_' . $count++] = empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? '%' . strtr($phrase, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?\|{}()])/'), array('[$1]'), $phrase), '\\\'') . '[[:>:]]';
201		}
202		$count = 0;
203	View Code Duplication	if (!empty($query_params['excluded_subject_words']) && empty($modSettings['search_force_index']))
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
204		foreach ($query_params['excluded_subject_words'] as $excludedWord)
205		{
206		$query_where[] = 'subject NOT ' . (empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? ' LIKE ' : 'RLIKE') . '{string:exclude_subject_words_' . $count . '}';
207		$query_params['exclude_subject_words_' . $count++] = empty($modSettings['search_match_words']) \|\| $search_data['no_regexp'] ? '%' . strtr($excludedWord, array('_' => '\\_', '%' => '\\%')) . '%' : '[[:<:]]' . addcslashes(preg_replace(array('/([\[\]$.+*?\|{}()])/'), array('[$1]'), $excludedWord), '\\\'') . '[[:>:]]';
208		}
209
210		if (!empty($modSettings['search_simple_fulltext']))
211		{
212	View Code Duplication	if($smcFunc['db_title'] == "PostgreSQL")
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
213		{
214		$language_ftx = $smcFunc['db_search_language']();
215
216		$query_where[] = 'to_tsvector({string:language_ftx},body) @@ to_tsquery({string:language_ftx},{string:body_match})';
217		$query_params['language_ftx'] = $language_ftx;
218		}
219		else
220		$query_where[] = 'MATCH (body) AGAINST ({string:body_match})';
221		$query_params['body_match'] = implode(' ', array_diff($words['indexed_words'], $query_params['excluded_index_words']));
222		}
223		else
224		{
225		$query_params['boolean_match'] = '';
226
227		// remove any indexed words that are used in the complex body search terms
228		$words['indexed_words'] = array_diff($words['indexed_words'], $words['complex_words']);
229
230		if($smcFunc['db_title'] == "PostgreSQL"){
231		$row = 0;
232		foreach ($words['indexed_words'] as $fulltextWord) {
233		$query_params['boolean_match'] .= ($row <> 0 ? '&' : '');
234		$query_params['boolean_match'] .= (in_array($fulltextWord, $query_params['excluded_index_words']) ? '!' : '') . $fulltextWord . ' ';
235		$row++;
236		}
237		}
238		else
239		foreach ($words['indexed_words'] as $fulltextWord)
240		$query_params['boolean_match'] .= (in_array($fulltextWord, $query_params['excluded_index_words']) ? '-' : '+') . $fulltextWord . ' ';
241
242		$query_params['boolean_match'] = substr($query_params['boolean_match'], 0, -1);
243
244		// if we have bool terms to search, add them in
245	View Code Duplication	if ($query_params['boolean_match']) {
		0 ignored issues – show Duplication introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
246		if($smcFunc['db_title'] == "PostgreSQL")
247		{
248		$language_ftx = $smcFunc['db_search_language']();
249
250		$query_where[] = 'to_tsvector({string:language_ftx},body) @@ to_tsquery({string:language_ftx},{string:boolean_match})';
251		$query_params['language_ftx'] = $language_ftx;
252		}
253		else
254		$query_where[] = 'MATCH (body) AGAINST ({string:boolean_match} IN BOOLEAN MODE)';
255		}
256
257		}
258
259		$ignoreRequest = $smcFunc['db_search_query']('insert_into_log_messages_fulltext', ($smcFunc['db_support_ignore'] ? ( '
260		INSERT IGNORE INTO {db_prefix}' . $search_data['insert_into'] . '
261		(' . implode(', ', array_keys($query_select)) . ')') : '') . '
262		SELECT ' . implode(', ', $query_select) . '
263		FROM {db_prefix}messages AS m
264		WHERE ' . implode('
265		AND ', $query_where) . (empty($search_data['max_results']) ? '' : '
266		LIMIT ' . ($search_data['max_results'] - $search_data['indexed_results'])),
267		$query_params
268		);
269
270		return $ignoreRequest;
271		}
272		}
273
274		?>
		0 ignored issues – show Best Practice introduced 2016-12-11 19:54 UTC by Report Bug Copy Issue Report It is not recommended to use PHP's closing tag `?>` in files other than templates. Using a closing tag in PHP files that only contain PHP code is not recommended as you might accidentally add whitespace after the closing tag which would then be output by PHP. This can cause severe problems, for example headers cannot be sent anymore. A simple precaution is to leave off the closing tag as it is not required, and it also has no negative effects whatsoever. Loading history...

albertlast / SMF2.1

Push — release-2.1 ( 6f6d35...abeae7 )

fulltext_search::prepareIndexes() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like