API: Fix a bug to make list=allfileusages functional again
authorSorawee Porncharoenwase <nullzero.free@gmail.com>
Thu, 6 Feb 2014 06:36:21 +0000 (13:36 +0700)
committerAnomie <bjorsch@wikimedia.org>
Thu, 6 Feb 2014 15:41:53 +0000 (15:41 +0000)
Querying list=allfileusages with affrom, afto, or afunique
will always get afinvalidtitle because the code passes the namespace
wrongly due to gerrit Iaa5a71ec. This patch fixes the problem.

Change-Id: I010dc3a03be74f49902fd446e7dbc306c16b869c

includes/api/ApiQueryAllLinks.php

index 13f766e..bccc25f 100644 (file)
@@ -149,14 +149,14 @@ class ApiQueryAllLinks extends ApiQueryGeneratorBase {
 
                // 'continue' always overrides 'from'
                $from = ( $continue || $params['from'] === null ? null :
-                       $this->titlePartToKey( $params['from'], $params['namespace'] ) );
+                       $this->titlePartToKey( $params['from'], $namespace ) );
                $to = ( $params['to'] === null ? null :
-                       $this->titlePartToKey( $params['to'], $params['namespace'] ) );
+                       $this->titlePartToKey( $params['to'], $namespace ) );
                $this->addWhereRange( $pfx . $fieldTitle, 'newer', $from, $to );
 
                if ( isset( $params['prefix'] ) ) {
                        $this->addWhere( $pfx . $fieldTitle . $db->buildLike( $this->titlePartToKey(
-                               $params['prefix'], $params['namespace'] ), $db->anyString() ) );
+                               $params['prefix'], $namespace ), $db->anyString() ) );
                }
 
                $this->addFields( array( 'pl_title' => $pfx . $fieldTitle ) );