Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

RangeChronologicalPager: Don't clear $offset when $this->rangeConds is set

Browse source 
It has the effect of breaking pagination entirely when a date range
is set. Special:Contributions doesn't break this way because it has
its own code clearing start/end when offset is set, but Special:Newfiles
does break.

Change-Id: Ie9b26a4b54ebc608035c942b5bdc6ee26b82bc1e
This commit is contained in:
Roan Kattouw 2017-06-16 16:12:31 -07:00
parent dad2306a78
commit b4ef22f17f
1 changed files with 0 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
includes/pager/RangeChronologicalPager.php
View file

@ -99,13 +99,6 @@ abstract class RangeChronologicalPager extends ReverseChronologicalPager {
* @return array
*/
protected function buildQueryInfo( $offset, $limit, $descending ) {
if ( count( $this->rangeConds ) > 0 ) {
// If range conditions are set, $offset is not used.
// However, if range conditions aren't set, (such as when using paging links)
// use the provided offset to get the proper query.
$offset = '';
}
list( $tables, $fields, $conds, $fname, $options, $join_conds ) = parent::buildQueryInfo(
$offset,
$limit,