From: Brad Jorsch <bjorsch@wikimedia.org>
Date: Wed, 21 Oct 2015 14:23:37 +0000 (-0400)
Subject: ApiQueryAllRevisions: Actually use 'start' and 'end'
X-Git-Tag: 1.31.0-rc.0~9284^2
X-Git-Url: http://git.cyclocoop.org/%24self?a=commitdiff_plain;h=dde8b480700b25615f4ceb0f50620b6e5f07ef25;p=lhc%2Fweb%2Fwiklou.git

ApiQueryAllRevisions: Actually use 'start' and 'end'

No idea how that got missed.

Bug: T116177
Change-Id: I69cb4d6ce12fe39ab5683f7791d00ee16958981d
---

diff --git a/includes/api/ApiQueryAllRevisions.php b/includes/api/ApiQueryAllRevisions.php
index e853cdc3e3..da895e2e42 100644
--- a/includes/api/ApiQueryAllRevisions.php
+++ b/includes/api/ApiQueryAllRevisions.php
@@ -94,6 +94,9 @@ class ApiQueryAllRevisions extends ApiQueryRevisionsBase {
 			}
 		}
 
+		$dir = $params['dir'];
+		$this->addTimestampWhereRange( 'rev_timestamp', $dir, $params['start'], $params['end'] );
+
 		if ( $this->fld_tags ) {
 			$this->addTables( 'tag_summary' );
 			$this->addJoinConds(
@@ -141,8 +144,6 @@ class ApiQueryAllRevisions extends ApiQueryRevisionsBase {
 			}
 		}
 
-		$dir = $params['dir'];
-
 		if ( $params['continue'] !== null ) {
 			$op = ( $dir == 'newer' ? '>' : '<' );
 			$cont = explode( '|', $params['continue'] );