Follow-up r84814: revert redundant summary message addition.
[lhc/web/wiklou.git] / includes / specials / SpecialContributions.php
index f6f81d0..4823af6 100644 (file)
@@ -1,21 +1,46 @@
 <?php
 /**
- * Special:Contributions, show user contributions in a paged list
+ * Implements Special:Contributions
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @file
  * @ingroup SpecialPage
  */
+
+/**
+ * Special:Contributions, show user contributions in a paged list
+ *
+ * @ingroup SpecialPage
+ */
+
 class SpecialContributions extends SpecialPage {
 
+       protected $opts;
+
        public function __construct() {
                parent::__construct( 'Contributions' );
        }
 
        public function execute( $par ) {
-               global $wgUser, $wgOut, $wgLang, $wgRequest;
+               global $wgUser, $wgOut, $wgRequest;
 
                $this->setHeaders();
                $this->outputHeader();
+               $wgOut->addModuleStyles( 'mediawiki.special' );
 
                $this->opts = array();
 
@@ -34,6 +59,8 @@ class SpecialContributions extends SpecialPage {
                        $this->opts['contribs'] = 'newbie';
                }
 
+               $this->opts['deletedOnly'] = $wgRequest->getBool( 'deletedOnly' );
+
                if( !strlen( $target ) ) {
                        $wgOut->addHTML( $this->getForm() );
                        return;
@@ -41,6 +68,7 @@ class SpecialContributions extends SpecialPage {
 
                $this->opts['limit'] = $wgRequest->getInt( 'limit', $wgUser->getOption('rclimit') );
                $this->opts['target'] = $target;
+               $this->opts['topOnly'] = $wgRequest->getBool( 'topOnly' );
 
                $nt = Title::makeTitleSafe( NS_USER, $target );
                if( !$nt ) {
@@ -53,6 +81,10 @@ class SpecialContributions extends SpecialPage {
                        $target = $nt->getText();
                        $wgOut->setSubtitle( $this->contributionsSub( $nt, $id ) );
                        $wgOut->setHTMLTitle( wfMsg( 'pagetitle', wfMsgExt( 'contributions-title', array( 'parsemag' ),$target ) ) );
+                       $user = User::newFromName( $target, false );
+                       if ( is_object($user) ) {
+                               $wgUser->getSkin()->setRelevantUser( $user );
+                       }
                } else {
                        $wgOut->setSubtitle( wfMsgHtml( 'sp-contributions-newbies-sub') );
                        $wgOut->setHTMLTitle( wfMsg( 'pagetitle', wfMsg( 'sp-contributions-newbies-title' ) ) );
@@ -64,10 +96,10 @@ class SpecialContributions extends SpecialPage {
                        $this->opts['namespace'] = '';
                }
 
-               $this->opts['tagfilter'] = (string) $wgRequest->getVal( 'tagfilter' );
-       
+               $this->opts['tagFilter'] = (string) $wgRequest->getVal( 'tagFilter' );
+
                // Allows reverts to have the bot flag in recent changes. It is just here to
-               // be passed in the form at the top of the page 
+               // be passed in the form at the top of the page
                if( $wgUser->isAllowed( 'markbotedits' ) && $wgRequest->getBool( 'bot' ) ) {
                        $this->opts['bot'] = '1';
                }
@@ -81,7 +113,7 @@ class SpecialContributions extends SpecialPage {
                        $this->opts['year'] = $wgRequest->getIntOrNull( 'year' );
                        $this->opts['month'] = $wgRequest->getIntOrNull( 'month' );
                }
-               
+
                // Add RSS/atom links
                $this->setSyndicated();
                $feedType = $wgRequest->getVal( 'feed' );
@@ -89,249 +121,298 @@ class SpecialContributions extends SpecialPage {
                        return $this->feed( $feedType );
                }
 
-               wfRunHooks( 'SpecialContributionsBeforeMainOutput', $id );
-
-               $wgOut->addHTML( $this->getForm() );
-
-               $pager = new ContribsPager( $target, $this->opts['namespace'], $this->opts['year'], $this->opts['month'] );
-               if( !$pager->getNumRows() ) {
-                       $wgOut->addWikiMsg( 'nocontribs', $target );
-               } else {
-                       # Show a message about slave lag, if applicable
-                       if( ( $lag = $pager->getDatabase()->getLag() ) > 0 )
-                               $wgOut->showLagWarning( $lag );
-
-                       $wgOut->addHTML(
-                               '<p>' . $pager->getNavigationBar() . '</p>' .
-                               $pager->getBody() .
-                               '<p>' . $pager->getNavigationBar() . '</p>'
-                       );
-               }
+               if ( wfRunHooks( 'SpecialContributionsBeforeMainOutput', array( $id ) ) ) {
 
+                       $wgOut->addHTML( $this->getForm() );
 
-               # Show the appropriate "footer" message - WHOIS tools, etc.
-               if( $target != 'newbies' ) {
-                       $message = 'sp-contributions-footer';
-                       if ( IP::isIPAddress( $target ) ) {
-                               $message = 'sp-contributions-footer-anon';
+                       $pager = new ContribsPager( array(
+                               'target' => $target,
+                               'namespace' => $this->opts['namespace'],
+                               'year' => $this->opts['year'],
+                               'month' => $this->opts['month'],
+                               'deletedOnly' => $this->opts['deletedOnly'],
+                               'topOnly' => $this->opts['topOnly'],
+                       ) );
+                       if( !$pager->getNumRows() ) {
+                               $wgOut->addWikiMsg( 'nocontribs', $target );
                        } else {
-                               $user = User::newFromName( $target );
-                               if ( !$user || $user->isAnon() ) {
-                                       // No message for non-existing users
-                                       return;
-                               }
+                               # Show a message about slave lag, if applicable
+                               if( ( $lag = $pager->getDatabase()->getLag() ) > 0 )
+                                       $wgOut->showLagWarning( $lag );
+
+                               $wgOut->addHTML(
+                                       '<p>' . $pager->getNavigationBar() . '</p>' .
+                                       $pager->getBody() .
+                                       '<p>' . $pager->getNavigationBar() . '</p>'
+                               );
                        }
+                       $wgOut->preventClickjacking( $pager->getPreventClickjacking() );
+
+
+                       # Show the appropriate "footer" message - WHOIS tools, etc.
+                       if( $target != 'newbies' ) {
+                               $message = 'sp-contributions-footer';
+                               if ( IP::isIPAddress( $target ) ) {
+                                       $message = 'sp-contributions-footer-anon';
+                               } else {
+                                       $user = User::newFromName( $target );
+                                       if ( !$user || $user->isAnon() ) {
+                                               // No message for non-existing users
+                                               return;
+                                       }
+                               }
 
-                       $text = wfMsgNoTrans( $message, $target );
-                       if( !wfEmptyMsg( $message, $text ) && $text != '-' ) {
-                               $wgOut->wrapWikiMsg(
-                                       "<div class='mw-contributions-footer'>\n$1\n</div>", $message );
+                               if( !wfMessage( $message, $target )->isDisabled() ) {
+                                       $wgOut->wrapWikiMsg(
+                                               "<div class='mw-contributions-footer'>\n$1\n</div>",
+                                               array( $message, $target ) );
+                               }
                        }
                }
        }
-       
+
        protected function setSyndicated() {
                global $wgOut;
-               $queryParams = array(
-                       'namespace' => $this->opts['namespace'],
-                       'target'  => $this->opts['target']
-               );
                $wgOut->setSyndicated( true );
-               $wgOut->setFeedAppendQuery( wfArrayToCGI( $queryParams ) );
+               $wgOut->setFeedAppendQuery( wfArrayToCGI( $this->opts ) );
        }
 
        /**
-       * Generates the subheading with links
-       * @param Title $nt Title object for the target
-       * @param integer $id User ID for the target
-       * @return String: appropriately-escaped HTML to be output literally
-       */
+        * Generates the subheading with links
+        * @param $nt Title object for the target
+        * @param $id Integer: User ID for the target
+        * @return String: appropriately-escaped HTML to be output literally
+        * @todo Fixme: almost the same as getSubTitle in SpecialDeletedContributions.php. Could be combined.
+        */
        protected function contributionsSub( $nt, $id ) {
-               global $wgSysopUserBans, $wgLang, $wgUser;
+               global $wgLang, $wgUser, $wgOut;
 
                $sk = $wgUser->getSkin();
 
-               if( 0 == $id ) {
+               if ( $id === null ) {
                        $user = htmlspecialchars( $nt->getText() );
                } else {
                        $user = $sk->link( $nt, htmlspecialchars( $nt->getText() ) );
                }
+               $userObj = User::newFromName( $nt->getText(), /* check for username validity not needed */ false );
                $talk = $nt->getTalkPage();
                if( $talk ) {
-                       # Talk page link
-                       $tools[] = $sk->link( $talk, wfMsgHtml( 'sp-contributions-talk' ) );
-                       if( ( $id != 0 && $wgSysopUserBans ) || ( $id == 0 && IP::isIPAddress( $nt->getText() ) ) ) {
-                               if( $wgUser->isAllowed( 'block' ) ) { # Block / Change block / Unblock links
-                                       if ( $wgUser->isBlocked() ) {
-                                               $tools[] = $sk->linkKnown( # Change block link
-                                                       SpecialPage::getTitleFor( 'Blockip', $nt->getDBkey() ),
-                                                       wfMsgHtml( 'change-blocklink' )
-                                               );
-                                               $tools[] = $sk->linkKnown( # Unblock link
-                                                       SpecialPage::getTitleFor( 'BlockList' ),
-                                                       wfMsgHtml( 'unblocklink' ),
-                                                       array(),
-                                                       array(
-                                                               'action' => 'unblock',
-                                                               'ip' => $nt->getDBkey() 
-                                                       ),
-                                                       array( 'known' )
-                                               );
-                                       }
-                                       else { # User is not blocked
-                                               $tools[] = $sk->linkKnown( # Block link
-                                                       SpecialPage::getTitleFor( 'Blockip', $nt->getDBkey() ),
-                                                       wfMsgHtml( 'blocklink' )
-                                               );
-                                       }
-                               }
-                               # Block log link
-                               $tools[] = $sk->linkKnown(
-                                       SpecialPage::getTitleFor( 'Log' ),
-                                       wfMsgHtml( 'sp-contributions-blocklog' ),
-                                       array(),
+                       $tools = self::getUserLinks( $nt, $talk, $userObj, $wgUser );
+                       $links = $wgLang->pipeList( $tools );
+
+                       // Show a note if the user is blocked and display the last block log entry.
+                       if ( $userObj->isBlocked() ) {
+                               LogEventsList::showLogExtract(
+                                       $wgOut,
+                                       'block',
+                                       $nt->getPrefixedText(),
+                                       '',
                                        array(
-                                               'type' => 'block',
-                                               'page' => $nt->getPrefixedText()
+                                               'lim' => 1,
+                                               'showIfEmpty' => false,
+                                               'msgKey' => array(
+                                                       $userObj->isAnon() ?
+                                                               'sp-contributions-blocked-notice-anon' :
+                                                               'sp-contributions-blocked-notice',
+                                                       $nt->getText() # Support GENDER in 'sp-contributions-blocked-notice'
+                                               ),
+                                               'offset' => '' # don't use $wgRequest parameter offset
                                        )
                                );
                        }
-                       # Other logs link
-                       $tools[] = $sk->linkKnown(
-                               SpecialPage::getTitleFor( 'Log' ),
-                               wfMsgHtml( 'sp-contributions-logs' ),
-                               array(),
-                               array( 'user' => $nt->getText() )
-                       );
-
-                       # Add link to deleted user contributions for priviledged users
-                       if( $wgUser->isAllowed( 'deletedhistory' ) ) {
-                               $tools[] = $sk->linkKnown(
-                                       SpecialPage::getTitleFor( 'DeletedContributions', $nt->getDBkey() ),
-                                       wfMsgHtml( 'sp-contributions-deleted' )
-                               );
-                       }
-
-                       # Add a link to change user rights for privileged users
-                       $userrightsPage = new UserrightsPage();
-                       if( 0 !== $id && $userrightsPage->userCanChangeRights( User::newFromId( $id ) ) ) {
-                               $tools[] = $sk->linkKnown(
-                                       SpecialPage::getTitleFor( 'Userrights', $nt->getDBkey() ),
-                                       wfMsgHtml( 'sp-contributions-userrights' )
-                               );
-                       }
-
-                       wfRunHooks( 'ContributionsToolLinks', array( $id, $nt, &$tools ) );
-       
-                       $links = $wgLang->pipeList( $tools );
                }
-       
+
                // Old message 'contribsub' had one parameter, but that doesn't work for
                // languages that want to put the "for" bit right after $user but before
                // $links.  If 'contribsub' is around, use it for reverse compatibility,
                // otherwise use 'contribsub2'.
-               if( wfEmptyMsg( 'contribsub', wfMsg( 'contribsub' ) ) ) {
+               if( wfEmptyMsg( 'contribsub' ) ) {
                        return wfMsgHtml( 'contribsub2', $user, $links );
                } else {
                        return wfMsgHtml( 'contribsub', "$user ($links)" );
                }
        }
 
+       /**
+        * Links to different places.
+        * @param $userpage Title: Target user page
+        * @param $talkpage Title: Talk page
+        * @param $target User: Target user object
+        * @param $subject User: The viewing user ($wgUser is still checked in some cases, like userrights page!!)
+        */
+       public static function getUserLinks( Title $userpage, Title $talkpage, User $target, User $subject ) {
+
+               $sk = $subject->getSkin();
+               $id = $target->getId();
+               $username = $target->getName();
+
+               $tools[] = $sk->link( $talkpage, wfMsgHtml( 'sp-contributions-talk' ) );
+
+               if( ( $id !== null ) || ( $id === null && IP::isIPAddress( $username ) ) ) {
+                       if( $subject->isAllowed( 'block' ) ) { # Block / Change block / Unblock links
+                               if ( $target->isBlocked() ) {
+                                       $tools[] = $sk->linkKnown( # Change block link
+                                               SpecialPage::getTitleFor( 'Block', $username ),
+                                               wfMsgHtml( 'change-blocklink' )
+                                       );
+                                       $tools[] = $sk->linkKnown( # Unblock link
+                                               SpecialPage::getTitleFor( 'Unblock', $username ),
+                                               wfMsgHtml( 'unblocklink' )
+                                       );
+                               } else { # User is not blocked
+                                       $tools[] = $sk->linkKnown( # Block link
+                                               SpecialPage::getTitleFor( 'Block', $username ),
+                                               wfMsgHtml( 'blocklink' )
+                                       );
+                               }
+                       }
+                       # Block log link
+                       $tools[] = $sk->linkKnown(
+                               SpecialPage::getTitleFor( 'Log' ),
+                               wfMsgHtml( 'sp-contributions-blocklog' ),
+                               array(),
+                               array(
+                                       'type' => 'block',
+                                       'page' => $userpage->getPrefixedText()
+                               )
+                       );
+               }
+               # Uploads
+               $tools[] = $sk->linkKnown(
+                       SpecialPage::getTitleFor( 'Listfiles' ),
+                       wfMsgHtml( 'sp-contributions-uploads' ),
+                       array(),
+                       array( 'user' => $username )
+               );
+
+               # Other logs link
+               $tools[] = $sk->linkKnown(
+                       SpecialPage::getTitleFor( 'Log' ),
+                       wfMsgHtml( 'sp-contributions-logs' ),
+                       array(),
+                       array( 'user' => $username )
+               );
+
+               # Add link to deleted user contributions for priviledged users
+               if( $subject->isAllowed( 'deletedhistory' ) ) {
+                       $tools[] = $sk->linkKnown(
+                               SpecialPage::getTitleFor( 'DeletedContributions', $username ),
+                               wfMsgHtml( 'sp-contributions-deleted' )
+                       );
+               }
+
+               # Add a link to change user rights for privileged users
+               $userrightsPage = new UserrightsPage();
+               if( $id !== null && $userrightsPage->userCanChangeRights( $target ) ) {
+                       $tools[] = $sk->linkKnown(
+                               SpecialPage::getTitleFor( 'Userrights', $username ),
+                               wfMsgHtml( 'sp-contributions-userrights' )
+                       );
+               }
+
+               wfRunHooks( 'ContributionsToolLinks', array( $id, $userpage, &$tools ) );
+               return $tools;
+       }
+
        /**
         * Generates the namespace selector form with hidden attributes.
-        * @param $this->opts Array: the options to be included.
+        * @return String: HTML fragment
         */
        protected function getForm() {
                global $wgScript;
-       
+
                $this->opts['title'] = $this->getTitle()->getPrefixedText();
                if( !isset( $this->opts['target'] ) ) {
                        $this->opts['target'] = '';
                } else {
                        $this->opts['target'] = str_replace( '_' , ' ' , $this->opts['target'] );
                }
-       
+
                if( !isset( $this->opts['namespace'] ) ) {
                        $this->opts['namespace'] = '';
                }
-       
+
                if( !isset( $this->opts['contribs'] ) ) {
                        $this->opts['contribs'] = 'user';
                }
-       
+
                if( !isset( $this->opts['year'] ) ) {
                        $this->opts['year'] = '';
                }
-       
+
                if( !isset( $this->opts['month'] ) ) {
                        $this->opts['month'] = '';
                }
-       
+
                if( $this->opts['contribs'] == 'newbie' ) {
                        $this->opts['target'] = '';
                }
 
-               if( !isset( $this->opts['tagfilter'] ) ) {
-                       $this->opts['tagfilter'] = '';
+               if( !isset( $this->opts['tagFilter'] ) ) {
+                       $this->opts['tagFilter'] = '';
                }
-       
-               $f = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) );
-               # Add hidden params for tracking
+
+               if( !isset( $this->opts['topOnly'] ) ) {
+                       $this->opts['topOnly'] = false;
+               }
+
+               $f = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript, 'class' => 'mw-contributions-form' ) );
+
+               # Add hidden params for tracking except for parameters in $skipParameters
+               $skipParameters = array( 'namespace', 'deletedOnly', 'target', 'contribs', 'year', 'month', 'topOnly' );
                foreach ( $this->opts as $name => $value ) {
-                       if( in_array( $name, array( 'namespace', 'target', 'contribs', 'year', 'month' ) ) ) {
+                       if( in_array( $name, $skipParameters ) ) {
                                continue;
                        }
-                       $f .= "\t" . Xml::hidden( $name, $value ) . "\n";
+                       $f .= "\t" . Html::hidden( $name, $value ) . "\n";
                }
 
-               $tagFilter = ChangeTags::buildTagFilterSelector( $this->opts['tagfilter'] );
-       
-               $f .= '<fieldset>' .
-                       Xml::element( 'legend', array(), wfMsg( 'sp-contributions-search' ) ) .
-                       Xml::radioLabel( wfMsgExt( 'sp-contributions-newbies', array( 'parsemag' ) ), 
-                               'contribs', 'newbie' , 'newbie', $this->opts['contribs'] == 'newbie' ? true : false ) . '<br />' .
-                       Xml::radioLabel( wfMsgExt( 'sp-contributions-username', array( 'parsemag' ) ), 
-                               'contribs' , 'user', 'user', $this->opts['contribs'] == 'user' ? true : false ) . ' ' .
+               $tagFilter = ChangeTags::buildTagFilterSelector( $this->opts['tagFilter'] );
+
+               $f .=   Xml::fieldset( wfMsg( 'sp-contributions-search' ) ) .
+                       Xml::radioLabel( wfMsgExt( 'sp-contributions-newbies', array( 'parsemag' ) ),
+                               'contribs', 'newbie' , 'newbie', $this->opts['contribs'] == 'newbie' ) . '<br />' .
+                       Xml::radioLabel( wfMsgExt( 'sp-contributions-username', array( 'parsemag' ) ),
+                               'contribs' , 'user', 'user', $this->opts['contribs'] == 'user' ) . ' ' .
                        Html::input( 'target', $this->opts['target'], 'text', array(
                                'size' => '20',
                                'required' => ''
                        ) + ( $this->opts['target'] ? array() : array( 'autofocus' ) ) ) . ' '.
-                       '<span style="white-space: nowrap">' .
-                       Xml::label( wfMsg( 'namespace' ), 'namespace' ) . ' ' .
-                       Xml::namespaceSelector( $this->opts['namespace'], '' ) .
-                       '</span>' .
-                       ( $tagFilter ? Xml::tags( 'p', null, implode( '&nbsp;', $tagFilter ) ) : '' ) .
-                       Xml::openElement( 'p' ) .
-                       '<span style="white-space: nowrap">' .
-                       Xml::dateMenu( $this->opts['year'], $this->opts['month'] ) .
-                       '</span>' . ' ' .
-                       Xml::submitButton( wfMsg( 'sp-contributions-submit' ) ) .
-                       Xml::closeElement( 'p' );
-       
+                       Html::rawElement( 'span', array( 'style' => 'white-space: nowrap' ),
+                               Xml::label( wfMsg( 'namespace' ), 'namespace' ) . ' ' .
+                               Xml::namespaceSelector( $this->opts['namespace'], '' )
+                       ) .
+                       Xml::checkLabel( wfMsg( 'history-show-deleted' ),
+                               'deletedOnly', 'mw-show-deleted-only', $this->opts['deletedOnly'] ) . '<br />' .
+                       Xml::tags( 'p', null, Xml::checkLabel( wfMsg( 'sp-contributions-toponly' ),
+                               'topOnly', 'mw-show-top-only', $this->opts['topOnly'] ) ) .
+                       ( $tagFilter ? Xml::tags( 'p', null, implode( '&#160;', $tagFilter ) ) : '' ) .
+                       Html::rawElement( 'p', array( 'style' => 'white-space: nowrap' ),
+                               Xml::dateMenu( $this->opts['year'], $this->opts['month'] ) . ' ' .
+                               Xml::submitButton( wfMsg( 'sp-contributions-submit' ) )
+                       ) . ' ';
                $explain = wfMsgExt( 'sp-contributions-explain', 'parseinline' );
-               if( !wfEmptyMsg( 'sp-contributions-explain', $explain ) )
-                       $f .= "<p>{$explain}</p>";
-       
-               $f .= '</fieldset>' .
+               if( !wfEmptyMsg( 'sp-contributions-explain' ) ) {
+                       $f .= "<p id='mw-sp-contributions-explain'>{$explain}</p>";
+               }
+               $f .= Xml::closeElement('fieldset' ) .
                        Xml::closeElement( 'form' );
                return $f;
        }
-       
+
        /**
         * Output a subscription feed listing recent edits to this page.
-        * @param string $type
+        * @param $type String
         */
        protected function feed( $type ) {
-               global $wgRequest, $wgFeed, $wgFeedClasses, $wgFeedLimit;
+               global $wgFeed, $wgFeedClasses, $wgFeedLimit, $wgOut;
 
                if( !$wgFeed ) {
-                       global $wgOut;
                        $wgOut->addWikiMsg( 'feed-unavailable' );
                        return;
                }
 
                if( !isset( $wgFeedClasses[$type] ) ) {
-                       global $wgOut;
                        $wgOut->addWikiMsg( 'feed-invalid' );
                        return;
                }
@@ -341,19 +422,26 @@ class SpecialContributions extends SpecialPage {
                        wfMsgExt( 'tagline', 'parsemag' ),
                        $this->getTitle()->getFullUrl() . "/" . urlencode($this->opts['target'])
                );
-                       
+
                // Already valid title
                $nt = Title::makeTitleSafe( NS_USER, $this->opts['target'] );
                $target = $this->opts['target'] == 'newbies' ? 'newbies' : $nt->getText();
-                       
-               $pager = new ContribsPager( $target, $this->opts['namespace'], 
-                       $this->opts['year'], $this->opts['month'], $this->opts['tagfilter'] );
+
+               $pager = new ContribsPager( array(
+                       'target' => $target,
+                       'namespace' => $this->opts['namespace'],
+                       'year' => $this->opts['year'],
+                       'month' => $this->opts['month'],
+                       'tagFilter' => $this->opts['tagFilter'],
+                       'deletedOnly' => $this->opts['deletedOnly'],
+                       'topOnly' => $this->opts['topOnly'],
+               ) );
 
                $pager->mLimit = min( $this->opts['limit'], $wgFeedLimit );
 
                $feed->outHeader();
                if( $pager->getNumRows() > 0 ) {
-                       while( $row = $pager->mResult->fetchObject() ) {
+                       foreach ( $pager->mResult as $row ) {
                                $feed->outItem( $this->feedItem( $row ) );
                        }
                }
@@ -361,10 +449,10 @@ class SpecialContributions extends SpecialPage {
        }
 
        protected function feedTitle() {
-               global $wgContLanguageCode, $wgSitename;
+               global $wgLanguageCode, $wgSitename;
                $page = SpecialPage::getPage( 'Contributions' );
                $desc = $page->getDescription();
-               return "$wgSitename - $desc [$wgContLanguageCode]";
+               return "$wgSitename - $desc [$wgLanguageCode]";
        }
 
        protected function feedItem( $row ) {
@@ -383,18 +471,26 @@ class SpecialContributions extends SpecialPage {
                                $comments
                        );
                } else {
-                       return NULL;
+                       return null;
                }
        }
 
+       /**
+        * @param $revision Revision
+        * @return string
+        */
        protected function feedItemAuthor( $revision ) {
                return $revision->getUserText();
        }
 
+       /**
+        * @param $revision Revision
+        * @return string
+        */
        protected function feedItemDesc( $revision ) {
                if( $revision ) {
                        return '<p>' . htmlspecialchars( $revision->getUserText() ) . wfMsgForContent( 'colon-separator' ) .
-                               htmlspecialchars( FeedItem::stripComment( $revision->getComment() ) ) . 
+                               htmlspecialchars( FeedItem::stripComment( $revision->getComment() ) ) .
                                "</p>\n<hr />\n<div>" .
                                nl2br( htmlspecialchars( $revision->getText() ) ) . "</div>";
                }
@@ -410,16 +506,26 @@ class ContribsPager extends ReverseChronologicalPager {
        public $mDefaultDirection = true;
        var $messages, $target;
        var $namespace = '', $mDb;
+       var $preventClickjacking = false;
 
-       function __construct( $target, $namespace = false, $year = false, $month = false, $tagFilter = false ) {
+       function __construct( $options ) {
                parent::__construct();
-               foreach( explode( ' ', 'uctop diff newarticle rollbacklink diff hist' ) as $msg ) {
-                       $this->messages[$msg] = wfMsgExt( $msg, array( 'escape') );
+
+               $msgs = array( 'uctop', 'diff', 'newarticle', 'rollbacklink', 'diff', 'hist', 'rev-delundel', 'pipe-separator' );
+
+               foreach( $msgs as $msg ) {
+                       $this->messages[$msg] = wfMsgExt( $msg, array( 'escapenoentities' ) );
                }
-               $this->target = $target;
-               $this->namespace = $namespace;
-               $this->tagFilter = $tagFilter;
 
+               $this->target = isset( $options['target'] ) ? $options['target'] : '';
+               $this->namespace = isset( $options['namespace'] ) ? $options['namespace'] : '';
+               $this->tagFilter = isset( $options['tagFilter'] ) ? $options['tagFilter'] : false;
+
+               $this->deletedOnly = !empty( $options['deletedOnly'] );
+               $this->topOnly = !empty( $options['topOnly'] );
+
+               $year = isset( $options['year'] ) ? $options['year'] : false;
+               $month = isset( $options['month'] ) ? $options['month'] : false;
                $this->getDateCond( $year, $month );
 
                $this->mDb = wfGetDB( DB_SLAVE, 'contributions' );
@@ -434,36 +540,38 @@ class ContribsPager extends ReverseChronologicalPager {
        function getQueryInfo() {
                global $wgUser;
                list( $tables, $index, $userCond, $join_cond ) = $this->getUserCond();
-               
+
                $conds = array_merge( $userCond, $this->getNamespaceCond() );
                // Paranoia: avoid brute force searches (bug 17342)
-               if( !$wgUser->isAllowed( 'deleterevision' ) ) {
+               if( !$wgUser->isAllowed( 'deletedhistory' ) ) {
                        $conds[] = $this->mDb->bitAnd('rev_deleted',Revision::DELETED_USER) . ' = 0';
                } else if( !$wgUser->isAllowed( 'suppressrevision' ) ) {
                        $conds[] = $this->mDb->bitAnd('rev_deleted',Revision::SUPPRESSED_USER) .
                                ' != ' . Revision::SUPPRESSED_USER;
                }
                $join_cond['page'] = array( 'INNER JOIN', 'page_id=rev_page' );
-               
+
                $queryInfo = array(
                        'tables' => $tables,
                        'fields' => array(
                                'page_namespace', 'page_title', 'page_is_new', 'page_latest', 'page_is_redirect',
-                               'page_len','rev_id', 'rev_page', 'rev_text_id', 'rev_timestamp', 'rev_comment', 
+                               'page_len','rev_id', 'rev_page', 'rev_text_id', 'rev_timestamp', 'rev_comment',
                                'rev_minor_edit', 'rev_user', 'rev_user_text', 'rev_parent_id', 'rev_deleted'
                        ),
                        'conds' => $conds,
                        'options' => array( 'USE INDEX' => array('revision' => $index) ),
                        'join_conds' => $join_cond
                );
-               
-               ChangeTags::modifyDisplayQuery( $queryInfo['tables'],
-                                                                               $queryInfo['fields'],
-                                                                               $queryInfo['conds'],
-                                                                               $queryInfo['join_conds'],
-                                                                               $queryInfo['options'],
-                                                                               $this->tagFilter );
-               
+
+               ChangeTags::modifyDisplayQuery(
+                       $queryInfo['tables'],
+                       $queryInfo['fields'],
+                       $queryInfo['conds'],
+                       $queryInfo['join_conds'],
+                       $queryInfo['options'],
+                       $this->tagFilter
+               );
+
                wfRunHooks( 'ContribsPager::getQueryInfo', array( &$this, &$queryInfo ) );
                return $queryInfo;
        }
@@ -484,6 +592,12 @@ class ContribsPager extends ReverseChronologicalPager {
                        $condition['rev_user_text'] = $this->target;
                        $index = 'usertext_timestamp';
                }
+               if( $this->deletedOnly ) {
+                       $condition[] = "rev_deleted != '0'";
+               }
+               if( $this->topOnly ) {
+                       $condition[] = "rev_id = page_latest";
+               }
                return array( $tables, $index, $condition, $join_conds );
        }
 
@@ -526,7 +640,6 @@ class ContribsPager extends ReverseChronologicalPager {
                $classes = array();
 
                $page = Title::newFromRow( $row );
-               $page->resetArticleId( $row->rev_page ); // use process cache
                $link = $sk->link(
                        $page,
                        htmlspecialchars( $page->getPrefixedText() ),
@@ -534,27 +647,20 @@ class ContribsPager extends ReverseChronologicalPager {
                        $page->isRedirect() ? array( 'redirect' => 'no' ) : array()
                );
                # Mark current revisions
-               $difftext = $topmarktext = '';
+               $topmarktext = '';
                if( $row->rev_id == $row->page_latest ) {
                        $topmarktext .= '<span class="mw-uctop">' . $this->messages['uctop'] . '</span>';
-                       if( !$row->page_is_new ) {
-                               $difftext .= '(' . $sk->linkKnown(
-                                       $page,
-                                       $this->messages['diff'],
-                                       array(),
-                                       array( 'diff' => 0 )
-                               ) . ')';
-                               # Add rollback link
-                               if( $page->quickUserCan( 'rollback') && $page->quickUserCan( 'edit' ) ) {
-                                       $topmarktext .= ' '.$sk->generateRollback( $rev );
-                               }
-                       } else {
-                               $difftext .= $this->messages['newarticle'];
+                       # Add rollback link
+                       if( !$row->page_is_new && $page->quickUserCan( 'rollback' )
+                               && $page->quickUserCan( 'edit' ) )
+                       {
+                               $this->preventClickjacking();
+                               $topmarktext .= ' '.$sk->generateRollback( $rev );
                        }
                }
                # Is there a visible previous revision?
-               if( !$rev->isDeleted(Revision::DELETED_TEXT) ) {
-                       $difftext = '(' . $sk->linkKnown(
+               if( $rev->userCan( Revision::DELETED_TEXT ) && $rev->getParentId() !== 0 ) {
+                       $difftext = $sk->linkKnown(
                                $page,
                                $this->messages['diff'],
                                array(),
@@ -562,33 +668,36 @@ class ContribsPager extends ReverseChronologicalPager {
                                        'diff' => 'prev',
                                        'oldid' => $row->rev_id
                                )
-                       ) . ')';
+                       );
                } else {
-                       $difftext = '(' . $this->messages['diff'] . ')';
+                       $difftext = $this->messages['diff'];
                }
-               $histlink = '('.$sk->linkKnown(
+               $histlink = $sk->linkKnown(
                        $page,
                        $this->messages['hist'],
                        array(),
                        array( 'action' => 'history' )
-               ) . ')';
+               );
 
                $comment = $wgContLang->getDirMark() . $sk->revComment( $rev, false, true );
                $date = $wgLang->timeanddate( wfTimestamp( TS_MW, $row->rev_timestamp ), true );
-               if( $rev->isDeleted( Revision::DELETED_TEXT ) ) {
-                       $d = '<span class="history-deleted">' . $date . '</span>';
-               } else {
+               if( $rev->userCan( Revision::DELETED_TEXT ) ) {
                        $d = $sk->linkKnown(
                                $page,
                                htmlspecialchars($date),
                                array(),
                                array( 'oldid' => intval( $row->rev_id ) )
                        );
+               } else {
+                       $d = htmlspecialchars( $date );
+               }
+               if( $rev->isDeleted( Revision::DELETED_TEXT ) ) {
+                       $d = '<span class="history-deleted">' . $d . '</span>';
                }
 
                if( $this->target == 'newbies' ) {
                        $userlink = ' . . ' . $sk->userLink( $row->rev_user, $row->rev_user_text );
-                       $userlink .= ' (' . $sk->userTalkLink( $row->rev_user, $row->rev_user_text ) . ') ';
+                       $userlink .= ' ' . wfMsg( 'parentheses', $sk->userTalkLink( $row->rev_user, $row->rev_user_text ) ) . ' ';
                } else {
                        $userlink = '';
                }
@@ -604,26 +713,33 @@ class ContribsPager extends ReverseChronologicalPager {
                } else {
                        $mflag = '';
                }
-               
-               if( $wgUser->isAllowed( 'deleterevision' ) ) {
-                       // If revision was hidden from sysops
+
+               // Don't show useless link to people who cannot hide revisions
+               $canHide = $wgUser->isAllowed( 'deleterevision' );
+               if( $canHide || ($rev->getVisibility() && $wgUser->isAllowed('deletedhistory')) ) {
                        if( !$rev->userCan( Revision::DELETED_RESTRICTED ) ) {
-                               $del = Xml::tags( 'span', array( 'class'=>'mw-revdelundel-link' ),
-                                       '(' . $this->message['rev-delundel'] . ')' ) . ' ';
-                       // Otherwise, show the link...
+                               $del = $this->mSkin->revDeleteLinkDisabled( $canHide ); // revision was hidden from sysops
                        } else {
                                $query = array(
-                                       'type' => 'revision',
+                                       'type'   => 'revision',
                                        'target' => $page->getPrefixedDbkey(),
-                                       'ids' => $rev->getId() );
+                                       'ids'    => $rev->getId()
+                               );
                                $del = $this->mSkin->revDeleteLink( $query,
-                                       $rev->isDeleted( Revision::DELETED_RESTRICTED ) ) . ' ';
+                                       $rev->isDeleted( Revision::DELETED_RESTRICTED ), $canHide );
                        }
+                       $del .= ' ';
                } else {
                        $del = '';
                }
 
-               $ret = "{$del}{$d} {$histlink} {$difftext} {$nflag}{$mflag} {$link}{$userlink} {$comment} {$topmarktext}";
+               $diffHistLinks = '(' . $difftext . $this->messages['pipe-separator'] . $histlink . ')';
+               $ret = "{$del}{$d} {$diffHistLinks} {$nflag}{$mflag} {$link}{$userlink} {$comment} {$topmarktext}";
+
+               # Denote if username is redacted for this edit
+               if( $rev->isDeleted( Revision::DELETED_USER ) ) {
+                       $ret .= " <strong>" . wfMsgHtml('rev-deleted-user-contribs') . "</strong>";
+               }
 
                # Tags, if any.
                list($tagSummary, $newClasses) = ChangeTags::formatSummaryRow( $row->ts_tags, 'contributions' );
@@ -648,4 +764,22 @@ class ContribsPager extends ReverseChronologicalPager {
                return $this->mDb;
        }
 
+       /**
+        * Overwrite Pager function and return a helpful comment
+        */
+       function getSqlComment() {
+               if ( $this->namespace || $this->deletedOnly ) {
+                       return 'contributions page filtered for namespace or RevisionDeleted edits'; // potentially slow, see CR r58153
+               } else {
+                       return 'contributions page unfiltered';
+               }
+       }
+
+       protected function preventClickjacking() {
+               $this->preventClickjacking = true;
+       }
+
+       public function getPreventClickjacking() {
+               return $this->preventClickjacking;
+       }
 }