ActiveUsersPager: Fix ordering and return 0-action users
authorBrad Jorsch <bjorsch@wikimedia.org>
Mon, 4 Mar 2019 04:01:29 +0000 (23:01 -0500)
committerBrad Jorsch <bjorsch@wikimedia.org>
Wed, 6 Mar 2019 19:52:27 +0000 (14:52 -0500)
commitf74efe7fec180e9c4d1a730539f293711adb7939
tree02f833ec66e8e0c23c0ae2715bbc7276b2b2a5f5
parent27a74895d8ffce6a1c15df3ed1f234b9fd10a048
ActiveUsersPager: Fix ordering and return 0-action users

Ia9d2ff00 introduced two incompatibilities with what the base classes
were trying to do:

* The $descending flag passed to buildQueryInfo() does not actually mean
  DESC if $this->mIsBackwards is set. The old code just blindly assumed
  that was the case.
* For paging to work right, we must return all $limit rows, we can't
  filter out users with 0 actions. Thus we need to LEFT JOIN
  recentchanges (and move some conditions around to cope)..

Bug: T217525
Change-Id: Iba105a31ff61fbb14931bf8903538bfe7b0ba48b
includes/specials/pagers/ActiveUsersPager.php