From dde8b480700b25615f4ceb0f50620b6e5f07ef25 Mon Sep 17 00:00:00 2001 From: Brad Jorsch Date: Wed, 21 Oct 2015 10:23:37 -0400 Subject: [PATCH] ApiQueryAllRevisions: Actually use 'start' and 'end' No idea how that got missed. Bug: T116177 Change-Id: I69cb4d6ce12fe39ab5683f7791d00ee16958981d --- includes/api/ApiQueryAllRevisions.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) 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'] ); -- 2.20.1