Branch merge of change-tagging branch with trunk
[lhc/web/wiklou.git] / includes / PageHistory.php
index cdf382d..0441cf2 100644 (file)
@@ -22,7 +22,6 @@ class PageHistory {
        var $mArticle, $mTitle, $mSkin;
        var $lastdate;
        var $linesonpage;
-       var $mNotificationTimestamp;
        var $mLatestId = null;
 
        /**
@@ -31,12 +30,10 @@ class PageHistory {
         * @param Article $article
         * @returns nothing
         */
-       function __construct($article) {
+       function __construct( $article ) {
                global $wgUser;
-
                $this->mArticle =& $article;
                $this->mTitle =& $article->mTitle;
-               $this->mNotificationTimestamp = NULL;
                $this->mSkin = $wgUser->getSkin();
                $this->preCacheMessages();
        }
@@ -74,8 +71,7 @@ class PageHistory {
                 * Allow client caching.
                 */
                if( $wgOut->checkLastModified( $this->mArticle->getTouched() ) )
-               /* Client cache fresh and headers sent, nothing more to do. */
-               return;
+                       return; // Client cache fresh and headers sent, nothing more to do.
 
                wfProfileIn( __METHOD__ );
 
@@ -92,7 +88,8 @@ class PageHistory {
                $wgOut->addScriptFile( 'history.js' );
 
                $logPage = SpecialPage::getTitleFor( 'Log' );
-               $logLink = $this->mSkin->makeKnownLinkObj( $logPage, wfMsgHtml( 'viewpagelogs' ), 'page=' . $this->mTitle->getPrefixedUrl() );
+               $logLink = $this->mSkin->makeKnownLinkObj( $logPage, wfMsgHtml( 'viewpagelogs' ),
+                       'page=' . $this->mTitle->getPrefixedUrl() );
                $wgOut->setSubtitle( $logLink );
 
                $feedType = $wgRequest->getVal( 'feed' );
@@ -110,52 +107,41 @@ class PageHistory {
                        return;
                }
 
-               /*
-                * "go=first" means to jump to the last (earliest) history page.
-                * This is deprecated, it no longer appears in the user interface
-                */
-               if ( $wgRequest->getText("go") == 'first' ) {
-                       $limit = $wgRequest->getInt( 'limit', 50 );
-                       global $wgFeedLimit;
-                       if( $limit > $wgFeedLimit ) {
-                               $limit = $wgFeedLimit;
-                       }
-                       $wgOut->redirect( $wgTitle->getLocalURL( "action=history&limit={$limit}&dir=prev" ) );
-                       return;
-               }
-
                /**
                 * Add date selector to quickly get to a certain time
                 */
                $year = $wgRequest->getInt( 'year' );
                $month = $wgRequest->getInt( 'month' );
+               $tagFilter = $wgRequest->getVal( 'tagfilter' );
 
                $action = htmlspecialchars( $wgScript );
                $wgOut->addHTML(
-               Xml::fieldset( wfMsg( 'history-fieldset-title' ), false, array( 'id' => 'mw-history-search' ) ) .
-                       "<form action=\"$action\" method=\"get\">" .
-               Xml::hidden( 'title', $this->mTitle->getPrefixedDBKey() ) . "\n" .
-               Xml::hidden( 'action', 'history' ) . "\n" .
-               $this->getDateMenu( $year, $month ) . '&nbsp;' .
-               Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . "\n" .
-                       '</form></fieldset>'
-                       );
+                       "<form action=\"$action\" method=\"get\" id=\"mw-history-searchform\">" .
+                       Xml::fieldset( wfMsg( 'history-fieldset-title' ), false, array( 'id' => 'mw-history-search' ) ) .
+                       Xml::hidden( 'title', $this->mTitle->getPrefixedDBKey() ) . "\n" .
+                       Xml::hidden( 'action', 'history' ) . "\n" .
+                       $this->getDateMenu( $year, $month ) . '&nbsp;' .
+                       implode( '&nbsp;', ChangeTags::buildTagFilterSelector( $tagFilter ) ) . '&nbsp;' .
+                       Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . "\n" .
+                       '</fieldset></form>'
+               );
 
-                       wfRunHooks( 'PageHistoryBeforeList', array( &$this->mArticle ) );
+               wfRunHooks( 'PageHistoryBeforeList', array( &$this->mArticle ) );
 
-                       /**
-                        * Do the list
-                        */
-                       $pager = new PageHistoryPager( $this, $year, $month );
-                       $this->linesonpage = $pager->getNumRows();
-                       $wgOut->addHTML(
+               /**
+                * Do the list
+                */
+               $pager = new PageHistoryPager( $this, $year, $month, $tagFilter );
+               $this->linesonpage = $pager->getNumRows();
+               $wgOut->addHTML(
                        $pager->getNavigationBar() .
                        $this->beginHistoryList() .
                        $pager->getBody() .
                        $this->endHistoryList() .
                        $pager->getNavigationBar()
-                       );
-                       wfProfileOut( __METHOD__ );
+               );
+
+               wfProfileOut( __METHOD__ );
        }
 
        /**
@@ -170,7 +156,7 @@ class PageHistory {
                } else {
                        $encMonth = '';
                }
-               if ( $year ) {
+               if( $year ) {
                        $encYear = intval( $year );
                } else if( $encMonth ) {
                        $thisMonth = intval( gmdate( 'n' ) );
@@ -183,10 +169,10 @@ class PageHistory {
                        $encYear = '';
                }
                return Xml::label( wfMsg( 'year' ), 'year' ) . ' '.
-               Xml::input( 'year', 4, $encYear, array('id' => 'year', 'maxlength' => 4) ) .
-                       ' '.
-               Xml::label( wfMsg( 'month' ), 'month' ) . ' '.
-               Xml::monthSelector( $encMonth, -1 );
+                       Xml::input( 'year', 4, $encYear, array('id' => 'year', 'maxlength' => 4) ) .
+                               ' '.
+                       Xml::label( wfMsg( 'month' ), 'month' ) . ' '.
+                       Xml::monthSelector( $encMonth, -1 );
        }
 
        /**
@@ -198,31 +184,31 @@ class PageHistory {
                global $wgTitle, $wgScript, $wgEnableHtmlDiff;
                $this->lastdate = '';
                $s = wfMsgExt( 'histlegend', array( 'parse') );
-               $s .= Xml::openElement( 'form', array( 'action' => $wgScript ) );
+               $s .= Xml::openElement( 'form', array( 'action' => $wgScript, 'id' => 'mw-history-compare' ) );
                $s .= Xml::hidden( 'title', $wgTitle->getPrefixedDbKey() );
-               if($wgEnableHtmlDiff){
+               if( $wgEnableHtmlDiff ) {
                        $s .= $this->submitButton( wfMsg( 'visualcomparison'),
-                       array(
-                                       'name' => 'htmldiff',
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-visualcomparison' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'name' => 'htmldiff',
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-visualcomparison' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
                        $s .= $this->submitButton( wfMsg( 'wikicodecomparison'),
-                       array(
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
-               }else{
+               } else {
                        $s .= $this->submitButton( wfMsg( 'compareselectedversions'),
-                       array(
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
                }
                $s .= '<ul id="pagehistory">' . "\n";
@@ -237,29 +223,29 @@ class PageHistory {
        function endHistoryList() {
                global $wgEnableHtmlDiff;
                $s = '</ul>';
-               if($wgEnableHtmlDiff){
+               if( $wgEnableHtmlDiff ) {
                        $s .= $this->submitButton( wfMsg( 'visualcomparison'),
-                       array(
-                                       'name' => 'htmldiff',
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-visualcomparison' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'name' => 'htmldiff',
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-visualcomparison' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
                        $s .= $this->submitButton( wfMsg( 'wikicodecomparison'),
-                       array(
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
-               }else{
+               } else {
                        $s .= $this->submitButton( wfMsg( 'compareselectedversions'),
-                       array(
-                                       'class'     => 'historysubmit',
-                                       'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
-                                       'title'     => wfMsg( 'tooltip-compareselectedversions' ),
-                       )
+                               array(
+                                               'class'     => 'historysubmit',
+                                               'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+                                               'title'     => wfMsg( 'tooltip-compareselectedversions' ),
+                               )
                        );
                }
                $s .= '</form>';
@@ -274,7 +260,7 @@ class PageHistory {
         */
        function submitButton($message, $attributes = array() ) {
                # Disable submit button if history has 1 revision only
-               if ( $this->linesonpage > 1 ) {
+               if( $this->linesonpage > 1 ) {
                        return Xml::submitButton( $message , $attributes );
                } else {
                        return '';
@@ -299,32 +285,28 @@ class PageHistory {
                $rev = new Revision( $row );
                $rev->setTitle( $this->mTitle );
 
-               $s = '';
                $curlink = $this->curLink( $rev, $latest );
                $lastlink = $this->lastLink( $rev, $next, $counter );
                $arbitrary = $this->diffButtons( $rev, $firstInList, $counter );
                $link = $this->revLink( $rev );
+               $classes = array();
 
-               $s .= "($curlink) ($lastlink) $arbitrary";
+               $s = "($curlink) ($lastlink) $arbitrary";
 
                if( $wgUser->isAllowed( 'deleterevision' ) ) {
-                       $revdel = SpecialPage::getTitleFor( 'Revisiondelete' );
                        if( $firstInList ) {
                                // We don't currently handle well changing the top revision's settings
-                               $del = $this->message['rev-delundel'];
+                               $del = Xml::tags( 'span', array( 'class'=>'mw-revdelundel-link' ), '('.$this->message['rev-delundel'].')' );
                        } else if( !$rev->userCan( Revision::DELETED_RESTRICTED ) ) {
                                // If revision was hidden from sysops
-                               $del = $this->message['rev-delundel'];
+                               $del = Xml::tags( 'span', array( 'class'=>'mw-revdelundel-link' ), '('.$this->message['rev-delundel'].')' );
                        } else {
-                               $del = $this->mSkin->makeKnownLinkObj( $revdel,
-                               $this->message['rev-delundel'],
-                                       'target=' . urlencode( $this->mTitle->getPrefixedDbkey() ) .
-                                       '&oldid=' . urlencode( $rev->getId() ) );
-                               // Bolden oversighted content
-                               if( $rev->isDeleted( Revision::DELETED_RESTRICTED ) )
-                               $del = "<strong>$del</strong>";
+                               $query = array( 'target' => $this->mTitle->getPrefixedDbkey(),
+                                       'oldid' => $rev->getId()
+                               );
+                               $del = $this->mSkin->revDeleteLink( $query, $rev->isDeleted( Revision::DELETED_RESTRICTED ) );
                        }
-                       $s .= " <tt>(<small>$del</small>)</tt> ";
+                       $s .= " $del ";
                }
 
                $s .= " $link";
@@ -334,35 +316,29 @@ class PageHistory {
                        $s .= ' ' . Xml::element( 'span', array( 'class' => 'minor' ), wfMsg( 'minoreditletter') );
                }
 
-               if ( !is_null( $size = $rev->getSize() ) && $rev->userCan( Revision::DELETED_TEXT ) ) {
+               if( !is_null( $size = $rev->getSize() ) && $rev->userCan( Revision::DELETED_TEXT ) ) {
                        $s .= ' ' . $this->mSkin->formatRevisionSize( $size );
                }
 
                $s .= $this->mSkin->revComment( $rev, false, true );
 
-               if ($notificationtimestamp && ($row->rev_timestamp >= $notificationtimestamp)) {
+               if( $notificationtimestamp && ($row->rev_timestamp >= $notificationtimestamp) ) {
                        $s .= ' <span class="updatedmarker">' .  wfMsgHtml( 'updatedmarker' ) . '</span>';
                }
-               #add blurb about text having been deleted
                if( $rev->isDeleted( Revision::DELETED_TEXT ) ) {
                        $s .= ' <tt>' . wfMsgHtml( 'deletedrev' ) . '</tt>';
                }
 
                $tools = array();
 
-               if ( !is_null( $next ) && is_object( $next ) ) {
-                       if( !$this->mTitle->getUserPermissionsErrors( 'rollback', $wgUser )
-                       && !$this->mTitle->getUserPermissionsErrors( 'edit', $wgUser )
-                       && $latest ) {
-                               $tools[] = '<span class="mw-rollback-link">'
-                               . $this->mSkin->buildRollbackLink( $rev )
-                               . '</span>';
+               if( !is_null( $next ) && is_object( $next ) ) {
+                       if( $latest && $this->mTitle->userCan( 'rollback' ) && $this->mTitle->userCan( 'edit' ) ) {
+                               $tools[] = '<span class="mw-rollback-link">'.$this->mSkin->buildRollbackLink( $rev ).'</span>';
                        }
 
-                       if( $this->mTitle->quickUserCan( 'edit' ) &&
-                       !$rev->isDeleted( Revision::DELETED_TEXT ) &&
-                       !$next->rev_deleted & Revision::DELETED_TEXT ) {
-
+                       if( $this->mTitle->quickUserCan( 'edit' ) && !$rev->isDeleted( Revision::DELETED_TEXT ) &&
+                               !$next->rev_deleted & Revision::DELETED_TEXT )
+                       {
                                # Create undo tooltip for the first (=latest) line only
                                $undoTooltip = $latest
                                        ? array( 'title' => wfMsg( 'tooltip-undo' ) )
@@ -382,9 +358,16 @@ class PageHistory {
                        $s .= ' (' . implode( ' | ', $tools ) . ')';
                }
 
+               # Tags
+               list($tagSummary, $newClasses) = ChangeTags::formatSummaryRow( $row->ts_tags, 'history' );
+               $classes = array_merge( $classes, $newClasses );
+               $s .= " $tagSummary";
+
                wfRunHooks( 'PageHistoryLineEnding', array( $this, &$row , &$s ) );
 
-               return "<li>$s</li>\n";
+               $classes = implode( ' ', $classes );
+
+               return "<li class=\"$classes\">$s</li>\n";
        }
 
        /**
@@ -418,10 +401,8 @@ class PageHistory {
                if( $latest || !$rev->userCan( Revision::DELETED_TEXT ) ) {
                        return $cur;
                } else {
-                       return $this->mSkin->makeKnownLinkObj(
-                       $this->mTitle, $cur,
-                               'diff=' . $this->getLatestID() .
-                               "&oldid=" . $rev->getId() );
+                       return $this->mSkin->makeKnownLinkObj( $this->mTitle, $cur,
+                               'diff=' . $this->mTitle->getLatestRevID() . "&oldid=" . $rev->getId() );
                }
        }
 
@@ -441,21 +422,13 @@ class PageHistory {
                        return $last;
                } elseif( $next === 'unknown' ) {
                        # Next row probably exists but is unknown, use an oldid=prev link
-                       return $this->mSkin->makeKnownLinkObj(
-                       $this->mTitle,
-                       $last,
+                       return $this->mSkin->makeKnownLinkObj( $this->mTitle, $last,
                                "diff=" . $prevRev->getId() . "&oldid=prev" );
                } elseif( !$prevRev->userCan(Revision::DELETED_TEXT) || !$nextRev->userCan(Revision::DELETED_TEXT) ) {
                        return $last;
                } else {
-                       return $this->mSkin->makeKnownLinkObj(
-                       $this->mTitle,
-                       $last,
-                               "diff=" . $prevRev->getId() . "&oldid={$next->rev_id}"
-                       /*,
-                               '',
-                               '',
-                               "tabindex={$counter}"*/ );
+                       return $this->mSkin->makeKnownLinkObj( $this->mTitle, $last,
+                               "diff=" . $prevRev->getId() . "&oldid={$next->rev_id}" );
                }
        }
 
@@ -479,7 +452,7 @@ class PageHistory {
                        }
 
                        /** @todo: move title texts to javascript */
-                       if ( $firstInList ) {
+                       if( $firstInList ) {
                                $first = Xml::element( 'input', array_merge(
                                $radio,
                                array(
@@ -508,19 +481,6 @@ class PageHistory {
                }
        }
 
-       /** @todo document */
-       function getLatestId() {
-               if( is_null( $this->mLatestId ) ) {
-                       $id = $this->mTitle->getArticleID();
-                       $db = wfGetDB( DB_SLAVE );
-                       $this->mLatestId = $db->selectField( 'page',
-                               "page_latest",
-                       array( 'page_id' => $id ),
-                       __METHOD__ );
-               }
-               return $this->mLatestId;
-       }
-
        /**
         * Fetch an array of revisions, specified by a given limit, offset and
         * direction. This is now only used by the feeds. It was previously
@@ -529,64 +489,34 @@ class PageHistory {
        function fetchRevisions($limit, $offset, $direction) {
                $dbr = wfGetDB( DB_SLAVE );
 
-               if ($direction == PageHistory::DIR_PREV)
-               list($dirs, $oper) = array("ASC", ">=");
+               if( $direction == PageHistory::DIR_PREV )
+                       list($dirs, $oper) = array("ASC", ">=");
                else /* $direction == PageHistory::DIR_NEXT */
-               list($dirs, $oper) = array("DESC", "<=");
+                       list($dirs, $oper) = array("DESC", "<=");
 
-               if ($offset)
-               $offsets = array("rev_timestamp $oper '$offset'");
+               if( $offset )
+                       $offsets = array("rev_timestamp $oper '$offset'");
                else
-               $offsets = array();
+                       $offsets = array();
 
                $page_id = $this->mTitle->getArticleID();
 
-               return $dbr->select(
-                       'revision',
-               Revision::selectFields(),
-               array_merge(array("rev_page=$page_id"), $offsets),
-               __METHOD__,
-               array('ORDER BY' => "rev_timestamp $dirs",
+               return $dbr->select( 'revision',
+                       Revision::selectFields(),
+                       array_merge(array("rev_page=$page_id"), $offsets),
+                       __METHOD__,
+                       array( 'ORDER BY' => "rev_timestamp $dirs", 
                                'USE INDEX' => 'page_timestamp', 'LIMIT' => $limit)
                );
        }
 
-       /** @todo document */
-       function getNotificationTimestamp() {
-               global $wgUser, $wgShowUpdatedMarker;
-
-               if ($this->mNotificationTimestamp !== NULL)
-               return $this->mNotificationTimestamp;
-
-               if ($wgUser->isAnon() || !$wgShowUpdatedMarker)
-               return $this->mNotificationTimestamp = false;
-
-               $dbr = wfGetDB(DB_SLAVE);
-
-               $this->mNotificationTimestamp = $dbr->selectField(
-                       'watchlist',
-                       'wl_notificationtimestamp',
-               array(  'wl_namespace' => $this->mTitle->getNamespace(),
-                               'wl_title' => $this->mTitle->getDBkey(),
-                               'wl_user' => $wgUser->getId()
-               ),
-               __METHOD__ );
-
-               // Don't use the special value reserved for telling whether the field is filled
-               if ( is_null( $this->mNotificationTimestamp ) ) {
-                       $this->mNotificationTimestamp = false;
-               }
-
-               return $this->mNotificationTimestamp;
-       }
-
        /**
         * Output a subscription feed listing recent edits to this page.
         * @param string $type
         */
        function feed( $type ) {
                global $wgFeedClasses, $wgRequest, $wgFeedLimit;
-               if ( !FeedUtils::checkFeedOutput($type) ) {
+               if( !FeedUtils::checkFeedOutput($type) ) {
                        return;
                }
 
@@ -618,12 +548,12 @@ class PageHistory {
        function feedEmpty() {
                global $wgOut;
                return new FeedItem(
-               wfMsgForContent( 'nohistory' ),
-               $wgOut->parse( wfMsgForContent( 'history-feed-empty' ) ),
-               $this->mTitle->getFullUrl(),
-               wfTimestamp( TS_MW ),
-                       '',
-               $this->mTitle->getTalkPage()->getFullUrl() );
+                       wfMsgForContent( 'nohistory' ),
+                       $wgOut->parse( wfMsgForContent( 'history-feed-empty' ) ),
+                       $this->mTitle->getFullUrl(),
+                       wfTimestamp( TS_MW ),
+                               '',
+                       $this->mTitle->getTalkPage()->getFullUrl() );
        }
 
        /**
@@ -649,23 +579,16 @@ class PageHistory {
                        $rev->getUserText(),
                        $wgContLang->timeanddate( $rev->getTimestamp() ) );
                } else {
-                       $title = $rev->getUserText() . ": " . $this->stripComment( $rev->getComment() );
+                       $title = $rev->getUserText() . wfMsgForContent( 'colon-separator' ) . FeedItem::stripComment( $rev->getComment() );
                }
 
                return new FeedItem(
-               $title,
-               $text,
-               $this->mTitle->getFullUrl( 'diff=' . $rev->getId() . '&oldid=prev' ),
-               $rev->getTimestamp(),
-               $rev->getUserText(),
-               $this->mTitle->getTalkPage()->getFullUrl() );
-       }
-
-       /**
-        * Quickie hack... strip out wikilinks to more legible form from the comment.
-        */
-       function stripComment( $text ) {
-               return preg_replace( '/\[\[([^]]*\|)?([^]]+)\]\]/', '\2', $text );
+                       $title,
+                       $text,
+                       $this->mTitle->getFullUrl( 'diff=' . $rev->getId() . '&oldid=prev' ),
+                       $rev->getTimestamp(),
+                       $rev->getUserText(),
+                       $this->mTitle->getTalkPage()->getFullUrl() );
        }
 }
 
@@ -674,21 +597,25 @@ class PageHistory {
  * @ingroup Pager
  */
 class PageHistoryPager extends ReverseChronologicalPager {
-       public $mLastRow = false, $mPageHistory;
+       public $mLastRow = false, $mPageHistory, $mTitle;
 
-       function __construct( $pageHistory, $year='', $month='' ) {
+       function __construct( $pageHistory, $year='', $month='', $tagFilter = '' ) {
                parent::__construct();
                $this->mPageHistory = $pageHistory;
+               $this->mTitle =& $this->mPageHistory->mTitle;
+               $this->tagFilter = $tagFilter;
                $this->getDateCond( $year, $month );
        }
 
        function getQueryInfo() {
                $queryInfo = array(
                        'tables'  => array('revision'),
-                       'fields'  => Revision::selectFields(),
+                       'fields'  => array_merge( Revision::selectFields(), array('ts_tags') ),
                        'conds'   => array('rev_page' => $this->mPageHistory->mTitle->getArticleID() ),
-                       'options' => array( 'USE INDEX' => array('revision' => 'page_timestamp') )
+                       'options' => array( 'USE INDEX' => array('revision' => 'page_timestamp') ),
+                       'join_conds' => array( 'tag_summary' => array( 'LEFT JOIN', 'ts_rev_id=rev_id' ) ),
                );
+               ChangeTags::modifyDisplayQuery( $queryInfo['tables'], $queryInfo['fields'], $queryInfo['conds'], $queryInfo['join_conds'], $this->tagFilter );
                wfRunHooks( 'PageHistoryPager::getQueryInfo', array( &$this, &$queryInfo ) );
                return $queryInfo;
        }
@@ -698,11 +625,11 @@ class PageHistoryPager extends ReverseChronologicalPager {
        }
 
        function formatRow( $row ) {
-               if ( $this->mLastRow ) {
+               if( $this->mLastRow ) {
                        $latest = $this->mCounter == 1 && $this->mIsFirst;
                        $firstInList = $this->mCounter == 1;
                        $s = $this->mPageHistory->historyLine( $this->mLastRow, $row, $this->mCounter++,
-                       $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
+                               $this->mTitle->getNotificationTimestamp(), $latest, $firstInList );
                } else {
                        $s = '';
                }
@@ -717,12 +644,12 @@ class PageHistoryPager extends ReverseChronologicalPager {
        }
 
        function getEndBody() {
-               if ( $this->mLastRow ) {
+               if( $this->mLastRow ) {
                        $latest = $this->mCounter == 1 && $this->mIsFirst;
                        $firstInList = $this->mCounter == 1;
-                       if ( $this->mIsBackwards ) {
+                       if( $this->mIsBackwards ) {
                                # Next row is unknown, but for UI reasons, probably exists if an offset has been specified
-                               if ( $this->mOffset == '' ) {
+                               if( $this->mOffset == '' ) {
                                        $next = null;
                                } else {
                                        $next = 'unknown';
@@ -732,7 +659,7 @@ class PageHistoryPager extends ReverseChronologicalPager {
                                $next = $this->mPastTheEndRow;
                        }
                        $s = $this->mPageHistory->historyLine( $this->mLastRow, $next, $this->mCounter++,
-                       $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
+                               $this->mTitle->getNotificationTimestamp(), $latest, $firstInList );
                } else {
                        $s = '';
                }