From: Happy-melon <happy-melon@users.mediawiki.org>
Date: Sun, 27 Mar 2011 16:49:01 +0000 (+0000)
Subject: Follow-up r84805: convert SpecialActiveusers to not use the now-absent UsersPager.
X-Git-Tag: 1.31.0-rc.0~31162
X-Git-Url: https://git.cyclocoop.org/%28%28?a=commitdiff_plain;h=3de23c36bb288204a040bc9ffbb77776b68c37db;p=lhc%2Fweb%2Fwiklou.git

Follow-up r84805: convert SpecialActiveusers to not use the now-absent UsersPager.
---

diff --git a/includes/AutoLoader.php b/includes/AutoLoader.php
index 85785d3ec6..41bd17138e 100644
--- a/includes/AutoLoader.php
+++ b/includes/AutoLoader.php
@@ -585,7 +585,6 @@ $wgAutoloadLocalClasses = array(
 
 	# includes/specials
 	'SpecialAllmessages' => 'includes/specials/SpecialAllmessages.php',
-	'ActiveUsersPager' => 'includes/specials/SpecialActiveusers.php',
 	'AllmessagesTablePager' => 'includes/specials/SpecialAllmessages.php',
 	'AncientPagesPage' => 'includes/specials/SpecialAncientpages.php',
 	'BrokenRedirectsPage' => 'includes/specials/SpecialBrokenRedirects.php',
diff --git a/includes/specials/SpecialActiveusers.php b/includes/specials/SpecialActiveusers.php
index caba7f3a43..2d0b582885 100644
--- a/includes/specials/SpecialActiveusers.php
+++ b/includes/specials/SpecialActiveusers.php
@@ -30,7 +30,7 @@
  *
  * @ingroup SpecialPage
  */
-class ActiveUsersPager extends UsersPager {
+class SpecialActiveUsers extends QueryPage {
 
 	/**
 	 * @var FormOptions
@@ -56,7 +56,7 @@ class ActiveUsersPager extends UsersPager {
 
 		$this->setupOptions();
 
-		parent::__construct();
+		parent::__construct( 'Activeusers' );
 	}
 
 	public function setupOptions() {
@@ -66,6 +66,8 @@ class ActiveUsersPager extends UsersPager {
 
 		$this->opts->add( 'hidebots', false, FormOptions::BOOL );
 		$this->opts->add( 'hidesysops', false, FormOptions::BOOL );
+		$this->opts->add( 'limit', 0, FormOptions::INT );
+		$this->opts->add( 'offset', 0, FormOptions::INT );
 
 		$this->opts->fetchValuesFromRequest( $wgRequest );
 
@@ -78,8 +80,8 @@ class ActiveUsersPager extends UsersPager {
 		}
 	}
 
-	function getIndexField() {
-		return 'rc_user_text';
+	function getOrderFields() {
+		return array( 'rc_user_text', 'user_id' );
 	}
 
 	function getQueryInfo() {
@@ -114,19 +116,23 @@ class ActiveUsersPager extends UsersPager {
 		return $query;
 	}
 
-	function formatRow( $row ) {
+	function formatResult( $skin, $row ) {
 		global $wgLang;
 		$userName = $row->user_name;
 
-		$ulinks = $this->getSkin()->userLink( $row->user_id, $userName );
-		$ulinks .= $this->getSkin()->userToolLinks( $row->user_id, $userName );
+		$ulinks = $skin->userLink( $row->user_id, $userName );
+		$ulinks .= $skin->userToolLinks( $row->user_id, $userName );
 
 		$list = array();
-		foreach( self::getGroups( $row->user_id ) as $group ) {
+		$user = User::newFromId( $row->user_id );
+		foreach( $user->getEffectiveGroups() as $group ) {
 			if ( isset( $this->groups[$group] ) ) {
-				return;
+				return false;
+			}
+			if( in_array( $group, array( '*', 'user' ) ) ){
+				continue;
 			}
-			$list[] = self::buildGroupLink( $group );
+			$list[] = SpecialListUsers::buildGroupLink( $group );
 		}
 		$groups = $wgLang->commaList( $list );
 
@@ -139,18 +145,35 @@ class ActiveUsersPager extends UsersPager {
 		);
 		$blocked = $row->blocked ? ' ' . wfMsgExt( 'listusers-blocked', array( 'parsemag' ), $userName ) : '';
 
-		return Html::rawElement( 'li', array(), "{$item} [{$count}]{$blocked}" );
+		return "{$item} [{$count}]{$blocked}";
+	}
+
+	function linkParameters() {
+		return array(
+			'hidebots' => isset( $this->groups['bot'] ),
+			'hidesysops' => isset( $this->groups['sysop'] ),
+			'username' => $this->requestedUser,
+		);
 	}
 
 	function getPageHeader() {
-		global $wgScript;
+		global $wgScript, $wgActiveUserDays, $wgLang;
 
 		$self = $this->getTitle();
-		$limit = $this->mLimit ? Html::hidden( 'limit', $this->mLimit ) : '';
+		$limit = $this->opts->getValue( 'limit' )
+			? Html::hidden( 'limit', $this->opts->getValue( 'limit' ) )
+			: '';
+		$offset = $this->opts->getValue( 'offset' )
+			? Html::hidden( 'offset', $this->opts->getValue( 'offset' ) )
+			: '';
+
+		$out = Html::rawElement( 'div', array( 'class' => 'mw-activeusers-intro' ),
+			wfMsgExt( 'activeusers-intro', array( 'parsemag', 'escape' ), $wgLang->formatNum( $wgActiveUserDays ) )
+		);
 
-		$out = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ); # Form tag
+		$out .= Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ); # Form tag
 		$out .= Xml::fieldset( wfMsg( 'activeusers' ) ) . "\n";
-		$out .= Html::hidden( 'title', $self->getPrefixedDBkey() ) . $limit . "\n";
+		$out .= Html::hidden( 'title', $self->getPrefixedDBkey() ) . $limit . $offset . "\n";
 
 		$out .= Xml::inputLabel( wfMsg( 'activeusers-from' ), 'username', 'offset', 20, $this->requestedUser ) . '<br />';# Username field
 
@@ -164,50 +187,4 @@ class ActiveUsersPager extends UsersPager {
 
 		return $out;
 	}
-}
-
-/**
- * @ingroup SpecialPage
- */
-class SpecialActiveUsers extends SpecialPage {
-
-	/**
-	 * Constructor
-	 */
-	public function __construct() {
-		parent::__construct( 'Activeusers' );
-	}
-
-	/**
-	 * Show the special page
-	 *
-	 * @param $par Mixed: parameter passed to the page or null
-	 */
-	public function execute( $par ) {
-		global $wgOut, $wgLang, $wgActiveUserDays;
-
-		$this->setHeaders();
-		$this->outputHeader();
-
-		$up = new ActiveUsersPager();
-
-		# getBody() first to check, if empty
-		$usersbody = $up->getBody();
-
-		$s = Html::rawElement( 'div', array( 'class' => 'mw-activeusers-intro' ),
-			wfMsgExt( 'activeusers-intro', array( 'parsemag', 'escape' ), $wgLang->formatNum( $wgActiveUserDays ) )
-		);
-
-		$s .= $up->getPageHeader();
-		if( $usersbody ) {
-			$s .= $up->getNavigationBar();
-			$s .= Html::rawElement( 'ul', array(), $usersbody );
-			$s .= $up->getNavigationBar();
-		} else {
-			$s .= Html::element( 'p', array(), wfMsg( 'activeusers-noresult' ) );
-		}
-
-		$wgOut->addHTML( $s );
-	}
-
-}
+}
\ No newline at end of file
diff --git a/includes/specials/SpecialListusers.php b/includes/specials/SpecialListusers.php
index 9f68dffbcc..a682914d50 100644
--- a/includes/specials/SpecialListusers.php
+++ b/includes/specials/SpecialListusers.php
@@ -259,11 +259,11 @@ class SpecialListUsers extends QueryPage {
 
 	/**
 	 * Format a link to a group description page
-	 *
+	 * Also called by SpecialActiveUsers
 	 * @param $group String: group name
 	 * @return string
 	 */
-	protected static function buildGroupLink( $group ) {
+	public static function buildGroupLink( $group ) {
 		static $cache = array();
 		if( !isset( $cache[$group] ) )
 			$cache[$group] = User::makeGroupLinkHtml( $group, htmlspecialchars( User::getGroupMember( $group ) ) );