diff options
author | Pierre Schmitz <pierre@archlinux.de> | 2014-12-27 15:41:37 +0100 |
---|---|---|
committer | Pierre Schmitz <pierre@archlinux.de> | 2014-12-31 11:43:28 +0100 |
commit | c1f9b1f7b1b77776192048005dcc66dcf3df2bfb (patch) | |
tree | 2b38796e738dd74cb42ecd9bfd151803108386bc /includes/api/ApiQueryLangBacklinks.php | |
parent | b88ab0086858470dd1f644e64cb4e4f62bb2be9b (diff) |
Update to MediaWiki 1.24.1
Diffstat (limited to 'includes/api/ApiQueryLangBacklinks.php')
-rw-r--r-- | includes/api/ApiQueryLangBacklinks.php | 48 |
1 files changed, 16 insertions, 32 deletions
diff --git a/includes/api/ApiQueryLangBacklinks.php b/includes/api/ApiQueryLangBacklinks.php index 5bd451b6..34842c63 100644 --- a/includes/api/ApiQueryLangBacklinks.php +++ b/includes/api/ApiQueryLangBacklinks.php @@ -31,7 +31,7 @@ */ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { - public function __construct( $query, $moduleName ) { + public function __construct( ApiQuery $query, $moduleName ) { parent::__construct( $query, $moduleName, 'lbl' ); } @@ -44,7 +44,7 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { } /** - * @param $resultPageSet ApiPageSet + * @param ApiPageSet $resultPageSet * @return void */ public function run( $resultPageSet = null ) { @@ -92,14 +92,14 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { $this->addOption( 'ORDER BY', array( 'll_title' . $sort, 'll_from' . $sort - )); + ) ); } } else { $this->addOption( 'ORDER BY', array( 'll_lang' . $sort, 'll_title' . $sort, 'll_from' . $sort - )); + ) ); } $this->addOption( 'LIMIT', $params['limit'] + 1 ); @@ -111,10 +111,14 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { $count = 0; $result = $this->getResult(); foreach ( $res as $row ) { - if ( ++ $count > $params['limit'] ) { - // We've reached the one extra which shows that there are additional pages to be had. Stop here... - // Continue string preserved in case the redirect query doesn't pass the limit - $this->setContinueEnumParameter( 'continue', "{$row->ll_lang}|{$row->ll_title}|{$row->ll_from}" ); + if ( ++$count > $params['limit'] ) { + // We've reached the one extra which shows that there are + // additional pages to be had. Stop here... Continue string + // preserved in case the redirect query doesn't pass the limit. + $this->setContinueEnumParameter( + 'continue', + "{$row->ll_lang}|{$row->ll_title}|{$row->ll_from}" + ); break; } @@ -140,7 +144,10 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { $fit = $result->addValue( array( 'query', $this->getModuleName() ), null, $entry ); if ( !$fit ) { - $this->setContinueEnumParameter( 'continue', "{$row->ll_lang}|{$row->ll_title}|{$row->ll_from}" ); + $this->setContinueEnumParameter( + 'continue', + "{$row->ll_lang}|{$row->ll_title}|{$row->ll_from}" + ); break; } } @@ -202,23 +209,6 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { ); } - public function getResultProperties() { - return array( - '' => array( - 'pageid' => 'integer', - 'ns' => 'namespace', - 'title' => 'string', - 'redirect' => 'boolean' - ), - 'lllang' => array( - 'lllang' => 'string' - ), - 'lltitle' => array( - 'lltitle' => 'string' - ) - ); - } - public function getDescription() { return array( 'Find all pages that link to the given language link.', 'Can be used to find all links with a language code, or', @@ -228,12 +218,6 @@ class ApiQueryLangBacklinks extends ApiQueryGeneratorBase { ); } - public function getPossibleErrors() { - return array_merge( parent::getPossibleErrors(), array( - array( 'missingparam', 'lang' ), - ) ); - } - public function getExamples() { return array( 'api.php?action=query&list=langbacklinks&lbltitle=Test&lbllang=fr', |