From 7ff539b729489516edc0a66a27e2a01a8b46f6b4 Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 17 Feb 2014 11:39:58 -0800 Subject: [PATCH] Fix a bad call in ElasticSearch recovery code Summary: Ref T4446. Some discussion in IRC. Prior to hitting the 1.0.0 issue, we hit and resolved this issue; this is a leftover call from bringing ApplicationSearch to main search. Test Plan: User confirmed this fixes the issue. Reviewers: btrahan Reviewed By: btrahan CC: aran Maniphest Tasks: T4446 Differential Revision: https://secure.phabricator.com/D8260 --- .../search/engine/PhabricatorSearchEngineElastic.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/applications/search/engine/PhabricatorSearchEngineElastic.php b/src/applications/search/engine/PhabricatorSearchEngineElastic.php index 95b973ad28..120460a6c0 100644 --- a/src/applications/search/engine/PhabricatorSearchEngineElastic.php +++ b/src/applications/search/engine/PhabricatorSearchEngineElastic.php @@ -222,7 +222,8 @@ final class PhabricatorSearchEngineElastic extends PhabricatorSearchEngine { throw $ex; } $query = clone $query; - $query->setQuery( + $query->setParameter( + 'query', addcslashes( $query->getParameter('query'), '+-&|!(){}[]^"~*?:\\')); $response = $this->executeRequest($uri, $this->buildSpec($query));