diff options
author | Pierre Schmitz <pierre@archlinux.de> | 2013-12-08 09:55:49 +0100 |
---|---|---|
committer | Pierre Schmitz <pierre@archlinux.de> | 2013-12-08 09:55:49 +0100 |
commit | 4ac9fa081a7c045f6a9f1cfc529d82423f485b2e (patch) | |
tree | af68743f2f4a47d13f2b0eb05f5c4aaf86d8ea37 /includes/api/ApiQuerySearch.php | |
parent | af4da56f1ad4d3ef7b06557bae365da2ea27a897 (diff) |
Update to MediaWiki 1.22.0
Diffstat (limited to 'includes/api/ApiQuerySearch.php')
-rw-r--r-- | includes/api/ApiQuerySearch.php | 39 |
1 files changed, 36 insertions, 3 deletions
diff --git a/includes/api/ApiQuerySearch.php b/includes/api/ApiQuerySearch.php index 86183391..36b55979 100644 --- a/includes/api/ApiQuerySearch.php +++ b/includes/api/ApiQuerySearch.php @@ -31,6 +31,14 @@ */ class ApiQuerySearch extends ApiQueryGeneratorBase { + /** + * When $wgSearchType is null, $wgSearchAlternatives[0] is null. Null isn't + * a valid option for an array for PARAM_TYPE, so we'll use a fake name + * that can't possibly be a class name and describes what the null behavior + * does + */ + const BACKEND_NULL_PARAM = 'database-backed'; + public function __construct( $query, $moduleName ) { parent::__construct( $query, $moduleName, 'sr' ); } @@ -59,7 +67,8 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { $prop = array_flip( $params['prop'] ); // Create search engine instance and set options - $search = SearchEngine::create(); + $search = isset( $params['backend'] ) && $params['backend'] != self::BACKEND_NULL_PARAM ? + SearchEngine::create( $params['backend'] ) : SearchEngine::create(); $search->setLimitOffset( $limit + 1, $params['offset'] ); $search->setNamespaces( $params['namespace'] ); $search->showRedirects = $params['redirects']; @@ -93,6 +102,8 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { } if ( is_null( $matches ) ) { $this->dieUsage( "{$what} search is disabled", "search-{$what}-disabled" ); + } elseif ( $matches instanceof Status && !$matches->isGood() ) { + $this->dieUsage( $matches->getWikiText(), 'search-error' ); } $apiResult = $this->getResult(); @@ -199,7 +210,9 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { } public function getAllowedParams() { - return array( + global $wgSearchType; + + $params = array( 'search' => array( ApiBase::PARAM_TYPE => 'string', ApiBase::PARAM_REQUIRED => true @@ -252,10 +265,23 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { ApiBase::PARAM_MAX2 => ApiBase::LIMIT_SML2 ) ); + + $alternatives = SearchEngine::getSearchTypes(); + if ( count( $alternatives ) > 1 ) { + if ( $alternatives[0] === null ) { + $alternatives[0] = self::BACKEND_NULL_PARAM; + } + $params['backend'] = array( + ApiBase::PARAM_DFLT => $wgSearchType, + ApiBase::PARAM_TYPE => $alternatives, + ); + } + + return $params; } public function getParamDescription() { - return array( + $descriptions = array( 'search' => 'Search for all page titles (or content) that has this value', 'namespace' => 'The namespace(s) to enumerate', 'what' => 'Search inside the text or titles', @@ -278,6 +304,12 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { 'offset' => 'Use this value to continue paging (return by query)', 'limit' => 'How many total pages to return' ); + + if ( count( SearchEngine::getSearchTypes() ) > 1 ) { + $descriptions['backend'] = 'Which search backend to use, if not the default'; + } + + return $descriptions; } public function getResultProperties() { @@ -345,6 +377,7 @@ class ApiQuerySearch extends ApiQueryGeneratorBase { return array_merge( parent::getPossibleErrors(), array( array( 'code' => 'search-text-disabled', 'info' => 'text search is disabled' ), array( 'code' => 'search-title-disabled', 'info' => 'title search is disabled' ), + array( 'code' => 'search-error', 'info' => 'search error has occurred' ), ) ); } |