Chad has uploaded a new change for review. https://gerrit.wikimedia.org/r/204284
Change subject: Fix for I49ee1270: Tabs aren't actually a problem ...................................................................... Fix for I49ee1270: Tabs aren't actually a problem Ideographic spaces are what actually causes parse errors. Why would you want to search for an ideographic space? Bug: T75374 Change-Id: I8876abe3418283f4772a438d3ac7c11684cc6333 --- M includes/Searcher.php 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CirrusSearch refs/changes/84/204284/1 diff --git a/includes/Searcher.php b/includes/Searcher.php index c15c585..1da185b 100644 --- a/includes/Searcher.php +++ b/includes/Searcher.php @@ -350,7 +350,10 @@ $searcher = $this; $originalTerm = $term; $searchContainedSyntax = false; - $this->term = trim( str_replace( "\t", ' ', $term ) ); + // Trim leading/trailing whitespace and strip out weird whitespacey characters + $this->term = trim( + str_replace( array( " ", "\t" ), ' ', $term ) + ); $this->boostLinks = $wgCirrusSearchBoostLinks; $searchType = 'full_text'; // Handle title prefix notation -- To view, visit https://gerrit.wikimedia.org/r/204284 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I8876abe3418283f4772a438d3ac7c11684cc6333 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/CirrusSearch Gerrit-Branch: master Gerrit-Owner: Chad <ch...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits