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

Reply via email to