X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FPageHistory.php;h=4ba67a289bac0847548ef5bdc67bc3fa1dfc465f;hb=a35b7d2a5e5e695d54ef6cacd9194a35e502c048;hp=b397e8cb0874e003c1067eaf70a580b247d35cd3;hpb=b909447ed2e9e7643f94f91fb1b1582f5eda8d19;p=lhc%2Fweb%2Fwiklou.git
diff --git a/includes/PageHistory.php b/includes/PageHistory.php
index b397e8cb08..4ba67a289b 100644
--- a/includes/PageHistory.php
+++ b/includes/PageHistory.php
@@ -3,6 +3,7 @@
* Page history
*
* Split off from Article.php and Skin.php, 2003-12-22
+ * @file
*/
/**
@@ -14,11 +15,10 @@
* history.
*
*/
-
class PageHistory {
const DIR_PREV = 0;
const DIR_NEXT = 1;
-
+
var $mArticle, $mTitle, $mSkin;
var $lastdate;
var $linesonpage;
@@ -38,6 +38,28 @@ class PageHistory {
$this->mTitle =& $article->mTitle;
$this->mNotificationTimestamp = NULL;
$this->mSkin = $wgUser->getSkin();
+ $this->preCacheMessages();
+ }
+
+ function getArticle() {
+ return $this->mArticle;
+ }
+
+ function getTitle() {
+ return $this->mTitle;
+ }
+
+ /**
+ * As we use the same small set of messages in various methods and that
+ * they are called often, we call them once and save them in $this->message
+ */
+ function preCacheMessages() {
+ // Precache various messages
+ if( !isset( $this->message ) ) {
+ foreach( explode(' ', 'cur last rev-delundel' ) as $msg ) {
+ $this->message[$msg] = wfMsgExt( $msg, array( 'escape') );
+ }
+ }
}
/**
@@ -46,37 +68,36 @@ class PageHistory {
* @returns nothing
*/
function history() {
- global $wgOut, $wgRequest, $wgTitle;
+ global $wgOut, $wgRequest, $wgTitle, $wgScript;
/*
* Allow client caching.
*/
+ if( $wgOut->checkLastModified( $this->mArticle->getTouched() ) )
+ /* Client cache fresh and headers sent, nothing more to do. */
+ return;
- if( $wgOut->checkLastModified( $this->mArticle->getTimestamp() ) )
- /* Client cache fresh and headers sent, nothing more to do. */
- return;
-
- $fname = 'PageHistory::history';
- wfProfileIn( $fname );
+ wfProfileIn( __METHOD__ );
/*
* Setup page variables.
*/
- $wgOut->setPageTitle( $this->mTitle->getPrefixedText() );
+ $wgOut->setPageTitle( wfMsg( 'history-title', $this->mTitle->getPrefixedText() ) );
+ $wgOut->setPageTitleActionText( wfMsg( 'history_short' ) );
$wgOut->setArticleFlag( false );
$wgOut->setArticleRelated( true );
- $wgOut->setRobotpolicy( 'noindex,nofollow' );
+ $wgOut->setRobotPolicy( 'noindex,nofollow' );
$wgOut->setSyndicated( true );
+ $wgOut->setFeedAppendQuery( 'action=history' );
+ $wgOut->addScriptFile( 'history.js' );
$logPage = SpecialPage::getTitleFor( 'Log' );
$logLink = $this->mSkin->makeKnownLinkObj( $logPage, wfMsgHtml( 'viewpagelogs' ), 'page=' . $this->mTitle->getPrefixedUrl() );
-
- $subtitle = wfMsgHtml( 'revhistory' ) . '
' . $logLink;
- $wgOut->setSubtitle( $subtitle );
+ $wgOut->setSubtitle( $logLink );
$feedType = $wgRequest->getVal( 'feed' );
if( $feedType ) {
- wfProfileOut( $fname );
+ wfProfileOut( __METHOD__ );
return $this->feed( $feedType );
}
@@ -84,83 +105,180 @@ class PageHistory {
* Fail if article doesn't exist.
*/
if( !$this->mTitle->exists() ) {
- $wgOut->addWikiText( wfMsg( 'nohistory' ) );
- wfProfileOut( $fname );
+ $wgOut->addWikiMsg( 'nohistory' );
+ wfProfileOut( __METHOD__ );
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;
}
- /**
- * Do the list
+ /**
+ * Add date selector to quickly get to a certain time
*/
- $pager = new PageHistoryPager( $this );
- $this->linesonpage = $pager->getNumRows();
+ $year = $wgRequest->getInt( 'year' );
+ $month = $wgRequest->getInt( 'month' );
+
+ $action = htmlspecialchars( $wgScript );
$wgOut->addHTML(
- $pager->getNavigationBar() .
- $this->beginHistoryList() .
+ Xml::fieldset( wfMsg( 'history-search' ), false, array( 'id' => 'mw-history-search' ) ) .
+ "'
+ );
+
+ wfRunHooks( 'PageHistoryBeforeList', array( &$this->mArticle ) );
+
+ /**
+ * Do the list
+ */
+ $pager = new PageHistoryPager( $this, $year, $month );
+ $this->linesonpage = $pager->getNumRows();
+ $wgOut->addHTML(
+ $pager->getNavigationBar() .
+ $this->beginHistoryList() .
$pager->getBody() .
$this->endHistoryList() .
$pager->getNavigationBar()
- );
- wfProfileOut( $fname );
+ );
+ wfProfileOut( __METHOD__ );
}
- /** @todo document */
+ /**
+ * @return string Formatted HTML
+ * @param int $year
+ * @param int $month
+ */
+ private function getDateMenu( $year, $month ) {
+ # Offset overrides year/month selection
+ if( $month && $month !== -1 ) {
+ $encMonth = intval( $month );
+ } else {
+ $encMonth = '';
+ }
+ if ( $year ) {
+ $encYear = intval( $year );
+ } else if( $encMonth ) {
+ $thisMonth = intval( gmdate( 'n' ) );
+ $thisYear = intval( gmdate( 'Y' ) );
+ if( intval($encMonth) > $thisMonth ) {
+ $thisYear--;
+ }
+ $encYear = $thisYear;
+ } else {
+ $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 );
+ }
+
+ /**
+ * Creates begin of history list with a submit button
+ *
+ * @return string HTML output
+ */
function beginHistoryList() {
- global $wgTitle;
+ global $wgTitle, $wgScript, $wgEnableHtmlDiff;
$this->lastdate = '';
$s = wfMsgExt( 'histlegend', array( 'parse') );
- $s .= '';
return $s;
}
- /** @todo document */
- function submitButton( $bits = array() ) {
- return ( $this->linesonpage > 0 )
- ? wfElement( 'input', array_merge( $bits,
- array(
- 'class' => 'historysubmit',
- 'type' => 'submit',
- 'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
- 'title' => wfMsg( 'tooltip-compareselectedversions' ).' ['.wfMsg( 'accesskey-compareselectedversions' ).']',
- 'value' => wfMsg( 'compareselectedversions' ),
- ) ) )
- : '';
+ /**
+ * Creates a submit button
+ *
+ * @param array $attributes attributes
+ * @return string HTML output for the submit button
+ */
+ function submitButton($message, $attributes = array() ) {
+ # Disable submit button if history has 1 revision only
+ if ( $this->linesonpage > 1 ) {
+ return Xml::submitButton( $message , $attributes );
+ } else {
+ return '';
+ }
}
/**
@@ -168,8 +286,8 @@ class PageHistory {
*
* @todo document some more, and maybe clean up the code (some params redundant?)
*
- * @param object $row The database row corresponding to the line (or is it the previous line?).
- * @param object $next The database row corresponding to the next line (or is it this one?).
+ * @param Row $row The database row corresponding to the previous line.
+ * @param mixed $next The database row corresponding to the next line.
* @param int $counter Apparently a counter of what row number we're at, counted from the top row = 1.
* @param $notificationtimestamp
* @param bool $latest Whether this row corresponds to the page's latest revision.
@@ -177,78 +295,109 @@ class PageHistory {
* @return string HTML output for the row
*/
function historyLine( $row, $next, $counter = '', $notificationtimestamp = false, $latest = false, $firstInList = false ) {
- global $wgUser;
+ global $wgUser, $wgLang;
$rev = new Revision( $row );
$rev->setTitle( $this->mTitle );
- $s = '';
+ $s = '';
$curlink = $this->curLink( $rev, $latest );
$lastlink = $this->lastLink( $rev, $next, $counter );
$arbitrary = $this->diffButtons( $rev, $firstInList, $counter );
$link = $this->revLink( $rev );
-
- $user = $this->mSkin->userLink( $rev->getUser(), $rev->getUserText() )
- . $this->mSkin->userToolLinks( $rev->getUser(), $rev->getUserText() );
-
+
$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 = wfMsgHtml( 'rev-delundel' );
+ $del = $this->message['rev-delundel'];
} else if( !$rev->userCan( Revision::DELETED_RESTRICTED ) ) {
- // If revision was hidden from sysops
- $del = wfMsgHtml( 'rev-delundel' );
+ // If revision was hidden from sysops
+ $del = $this->message['rev-delundel'];
} else {
$del = $this->mSkin->makeKnownLinkObj( $revdel,
- wfMsg( 'rev-delundel' ),
+ $this->message['rev-delundel'],
'target=' . urlencode( $this->mTitle->getPrefixedDbkey() ) .
'&oldid=' . urlencode( $rev->getId() ) );
+ // Bolden oversighted content
+ if( $rev->isDeleted( Revision::DELETED_RESTRICTED ) )
+ $del = "$del";
}
- $s .= " ($del) ";
+ $s .= " ($del) ";
}
-
+
$s .= " $link";
- #getUser is safe, but this avoids making the invalid untargeted contribs links
- if( $row->rev_deleted & Revision::DELETED_USER ) {
- $user = '' . wfMsg('rev-deleted-user') . '';
- }
- $s .= " $user";
+ $s .= " " . $this->mSkin->revUserTools( $rev, true ) . "";
if( $row->rev_minor_edit ) {
- $s .= ' ' . wfElement( 'span', array( 'class' => 'minor' ), wfMsg( 'minoreditletter') );
+ $s .= ' ' . Xml::element( 'span', array( 'class' => 'minor' ), wfMsg( 'minoreditletter') );
}
- #getComment is safe, but this is better formatted
- if( $rev->isDeleted( Revision::DELETED_COMMENT ) ) {
- $s .= " ";
- } else {
- $s .= $this->mSkin->revComment( $rev );
+
+ 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)) {
$s .= ' ' . wfMsgHtml( 'updatedmarker' ) . '';
}
#add blurb about text having been deleted
- if( $row->rev_deleted & Revision::DELETED_TEXT ) {
- $s .= ' ' . wfMsgHtml( 'deletedrev' );
+ if( $rev->isDeleted( Revision::DELETED_TEXT ) ) {
+ $s .= ' ' . wfMsgHtml( 'deletedrev' ) . '';
+ }
+
+ $tools = array();
+
+ if ( !is_null( $next ) && is_object( $next ) ) {
+ if( !$this->mTitle->getUserPermissionsErrors( 'rollback', $wgUser )
+ && !$this->mTitle->getUserPermissionsErrors( 'edit', $wgUser )
+ && $latest ) {
+ $tools[] = ''
+ . $this->mSkin->buildRollbackLink( $rev )
+ . '';
+ }
+
+ 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' ) )
+ : array();
+ $undolink = $this->mSkin->link(
+ $this->mTitle,
+ wfMsgHtml( 'editundo' ),
+ $undoTooltip,
+ array( 'action' => 'edit', 'undoafter' => $next->rev_id, 'undo' => $rev->getId() ),
+ array( 'known', 'noclasses' )
+ );
+ $tools[] = "{$undolink}";
+ }
}
- if( $wgUser->isAllowed( 'rollback' ) && $latest ) {
- $s .= ' '.$this->mSkin->generateRollback( $rev );
+
+ if( $tools ) {
+ $s .= ' (' . implode( ' | ', $tools ) . ')';
}
- $s .= "\n";
- return $s;
+ wfRunHooks( 'PageHistoryLineEnding', array( $this, &$row , &$s ) );
+
+ return "$s\n";
}
-
- /** @todo document */
+
+ /**
+ * Create a link to view this revision of the page
+ * @param Revision $rev
+ * @returns string
+ */
function revLink( $rev ) {
global $wgLang;
$date = $wgLang->timeanddate( wfTimestamp(TS_MW, $rev->getTimestamp()), true );
if( $rev->userCan( Revision::DELETED_TEXT ) ) {
$link = $this->mSkin->makeKnownLinkObj(
- $this->mTitle, $date, "oldid=" . $rev->getId() );
+ $this->mTitle, $date, "oldid=" . $rev->getId() );
} else {
$link = $date;
}
@@ -258,53 +407,71 @@ class PageHistory {
return $link;
}
- /** @todo document */
+ /**
+ * Create a diff-to-current link for this revision for this page
+ * @param Revision $rev
+ * @param Bool $latest, this is the latest revision of the page?
+ * @returns string
+ */
function curLink( $rev, $latest ) {
- $cur = wfMsgExt( 'cur', array( 'escape') );
+ $cur = $this->message['cur'];
if( $latest || !$rev->userCan( Revision::DELETED_TEXT ) ) {
return $cur;
} else {
return $this->mSkin->makeKnownLinkObj(
- $this->mTitle, $cur,
+ $this->mTitle, $cur,
'diff=' . $this->getLatestID() .
"&oldid=" . $rev->getId() );
}
}
- /** @todo document */
- function lastLink( $rev, $next, $counter ) {
- $last = wfMsgExt( 'last', array( 'escape' ) );
- if ( is_null( $next ) ) {
+ /**
+ * Create a diff-to-previous link for this revision for this page.
+ * @param Revision $prevRev, the previous revision
+ * @param mixed $next, the newer revision
+ * @param int $counter, what row on the history list this is
+ * @returns string
+ */
+ function lastLink( $prevRev, $next, $counter ) {
+ $last = $this->message['last'];
+ # $next may either be a Row, null, or "unkown"
+ $nextRev = is_object($next) ? new Revision( $next ) : $next;
+ if( is_null($next) ) {
# Probably no next row
return $last;
- } elseif ( $next === 'unknown' ) {
+ } elseif( $next === 'unknown' ) {
# Next row probably exists but is unknown, use an oldid=prev link
return $this->mSkin->makeKnownLinkObj(
- $this->mTitle,
- $last,
- "diff=" . $rev->getId() . "&oldid=prev" );
- } elseif( !$rev->userCan( Revision::DELETED_TEXT ) ) {
+ $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=" . $rev->getId() . "&oldid={$next->rev_id}"
- /*,
+ $this->mTitle,
+ $last,
+ "diff=" . $prevRev->getId() . "&oldid={$next->rev_id}"
+ /*,
'',
'',
"tabindex={$counter}"*/ );
}
}
- /** @todo document */
+ /**
+ * Create radio buttons for page history
+ *
+ * @param object $rev Revision
+ * @param bool $firstInList Is this version the first one?
+ * @param int $counter A counter of what row number we're at, counted from the top row = 1.
+ * @return string HTML output for the radio buttons
+ */
function diffButtons( $rev, $firstInList, $counter ) {
if( $this->linesonpage > 1) {
$radio = array(
'type' => 'radio',
'value' => $rev->getId(),
-# do we really need to flood this on every item?
-# 'title' => wfMsgHtml( 'selectolderversionfordiff' )
);
if( !$rev->userCan( Revision::DELETED_TEXT ) ) {
@@ -313,9 +480,9 @@ class PageHistory {
/** @todo: move title texts to javascript */
if ( $firstInList ) {
- $first = wfElement( 'input', array_merge(
- $radio,
- array(
+ $first = Xml::element( 'input', array_merge(
+ $radio,
+ array(
'style' => 'visibility:hidden',
'name' => 'oldid' ) ) );
$checkmark = array( 'checked' => 'checked' );
@@ -325,16 +492,16 @@ class PageHistory {
} else {
$checkmark = array();
}
- $first = wfElement( 'input', array_merge(
- $radio,
- $checkmark,
- array( 'name' => 'oldid' ) ) );
- $checkmark = array();
- }
- $second = wfElement( 'input', array_merge(
+ $first = Xml::element( 'input', array_merge(
$radio,
$checkmark,
- array( 'name' => 'diff' ) ) );
+ array( 'name' => 'oldid' ) ) );
+ $checkmark = array();
+ }
+ $second = Xml::element( 'input', array_merge(
+ $radio,
+ $checkmark,
+ array( 'name' => 'diff' ) ) );
return $first . $second;
} else {
return '';
@@ -345,76 +512,66 @@ class PageHistory {
function getLatestId() {
if( is_null( $this->mLatestId ) ) {
$id = $this->mTitle->getArticleID();
- $db = wfGetDB(DB_SLAVE);
+ $db = wfGetDB( DB_SLAVE );
$this->mLatestId = $db->selectField( 'page',
"page_latest",
- array( 'page_id' => $id ),
- 'PageHistory::getLatestID' );
+ 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
+ * direction. This is now only used by the feeds. It was previously
* used by the main UI but that's now handled by the pager.
*/
function fetchRevisions($limit, $offset, $direction) {
- $fname = 'PageHistory::fetchRevisions';
-
$dbr = wfGetDB( DB_SLAVE );
if ($direction == PageHistory::DIR_PREV)
- list($dirs, $oper) = array("ASC", ">=");
+ 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'");
+ $offsets = array("rev_timestamp $oper '$offset'");
else
- $offsets = array();
+ $offsets = array();
$page_id = $this->mTitle->getArticleID();
- $res = $dbr->select(
+ return $dbr->select(
'revision',
- array('rev_id', 'rev_page', 'rev_text_id', 'rev_user', 'rev_comment', 'rev_user_text',
- 'rev_timestamp', 'rev_minor_edit', 'rev_deleted'),
- array_merge(array("rev_page=$page_id"), $offsets),
- $fname,
- array('ORDER BY' => "rev_timestamp $dirs",
+ Revision::selectFields(),
+ array_merge(array("rev_page=$page_id"), $offsets),
+ __METHOD__,
+ array('ORDER BY' => "rev_timestamp $dirs",
'USE INDEX' => 'page_timestamp', 'LIMIT' => $limit)
- );
-
- $result = array();
- while (($obj = $dbr->fetchObject($res)) != NULL)
- $result[] = $obj;
-
- return $result;
+ );
}
/** @todo document */
function getNotificationTimestamp() {
global $wgUser, $wgShowUpdatedMarker;
- $fname = 'PageHistory::getNotficationTimestamp';
if ($this->mNotificationTimestamp !== NULL)
- return $this->mNotificationTimestamp;
+ return $this->mNotificationTimestamp;
if ($wgUser->isAnon() || !$wgShowUpdatedMarker)
- return $this->mNotificationTimestamp = false;
+ return $this->mNotificationTimestamp = false;
$dbr = wfGetDB(DB_SLAVE);
$this->mNotificationTimestamp = $dbr->selectField(
'watchlist',
'wl_notificationtimestamp',
- array( 'wl_namespace' => $this->mTitle->getNamespace(),
+ array( 'wl_namespace' => $this->mTitle->getNamespace(),
'wl_title' => $this->mTitle->getDBkey(),
- 'wl_user' => $wgUser->getID()
- ),
- $fname);
-
+ '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;
@@ -422,28 +579,31 @@ class PageHistory {
return $this->mNotificationTimestamp;
}
-
+
/**
* Output a subscription feed listing recent edits to this page.
* @param string $type
*/
function feed( $type ) {
- require_once 'SpecialRecentchanges.php';
-
- global $wgFeedClasses;
- if( !isset( $wgFeedClasses[$type] ) ) {
- global $wgOut;
- $wgOut->addWikiText( wfMsg( 'feed-invalid' ) );
+ global $wgFeedClasses, $wgRequest, $wgFeedLimit;
+ if ( !FeedUtils::checkFeedOutput($type) ) {
return;
}
-
+
$feed = new $wgFeedClasses[$type](
- $this->mTitle->getPrefixedText() . ' - ' .
- wfMsgForContent( 'history-feed-title' ),
- wfMsgForContent( 'history-feed-description' ),
- $this->mTitle->getFullUrl( 'action=history' ) );
+ $this->mTitle->getPrefixedText() . ' - ' .
+ wfMsgForContent( 'history-feed-title' ),
+ wfMsgForContent( 'history-feed-description' ),
+ $this->mTitle->getFullUrl( 'action=history' ) );
+
+ // Get a limit on number of feed entries. Provide a sane default
+ // of 10 if none is defined (but limit to $wgFeedLimit max)
+ $limit = $wgRequest->getInt( 'limit', 10 );
+ if( $limit > $wgFeedLimit || $limit < 1 ) {
+ $limit = 10;
+ }
+ $items = $this->fetchRevisions($limit, 0, PageHistory::DIR_NEXT);
- $items = $this->fetchRevisions(10, 0, PageHistory::DIR_NEXT);
$feed->outHeader();
if( $items ) {
foreach( $items as $row ) {
@@ -454,18 +614,18 @@ class PageHistory {
}
$feed->outFooter();
}
-
+
function feedEmpty() {
global $wgOut;
return new FeedItem(
- wfMsgForContent( 'nohistory' ),
- $wgOut->parse( wfMsgForContent( 'history-feed-empty' ) ),
- $this->mTitle->getFullUrl(),
- wfTimestamp( TS_MW ),
+ wfMsgForContent( 'nohistory' ),
+ $wgOut->parse( wfMsgForContent( 'history-feed-empty' ) ),
+ $this->mTitle->getFullUrl(),
+ wfTimestamp( TS_MW ),
'',
- $this->mTitle->getTalkPage()->getFullUrl() );
+ $this->mTitle->getTalkPage()->getFullUrl() );
}
-
+
/**
* Generate a FeedItem object from a given revision table row
* Borrows Recent Changes' feed generation functions for formatting;
@@ -477,30 +637,30 @@ class PageHistory {
function feedItem( $row ) {
$rev = new Revision( $row );
$rev->setTitle( $this->mTitle );
- $text = rcFormatDiffRow( $this->mTitle,
- $this->mTitle->getPreviousRevisionID( $rev->getId() ),
- $rev->getId(),
- $rev->getTimestamp(),
- $rev->getComment() );
-
+ $text = FeedUtils::formatDiffRow( $this->mTitle,
+ $this->mTitle->getPreviousRevisionID( $rev->getId() ),
+ $rev->getId(),
+ $rev->getTimestamp(),
+ $rev->getComment() );
+
if( $rev->getComment() == '' ) {
global $wgContLang;
$title = wfMsgForContent( 'history-feed-item-nocomment',
- $rev->getUserText(),
- $wgContLang->timeanddate( $rev->getTimestamp() ) );
+ $rev->getUserText(),
+ $wgContLang->timeanddate( $rev->getTimestamp() ) );
} else {
$title = $rev->getUserText() . ": " . $this->stripComment( $rev->getComment() );
}
return new FeedItem(
- $title,
- $text,
- $this->mTitle->getFullUrl( 'diff=' . $rev->getId() . '&oldid=prev' ),
- $rev->getTimestamp(),
- $rev->getUserText(),
- $this->mTitle->getTalkPage()->getFullUrl() );
+ $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.
*/
@@ -510,22 +670,27 @@ class PageHistory {
}
+/**
+ * @ingroup Pager
+ */
class PageHistoryPager extends ReverseChronologicalPager {
public $mLastRow = false, $mPageHistory;
-
- function __construct( $pageHistory ) {
+
+ function __construct( $pageHistory, $year='', $month='' ) {
parent::__construct();
$this->mPageHistory = $pageHistory;
+ $this->getDateCond( $year, $month );
}
function getQueryInfo() {
- return array(
- 'tables' => 'revision',
- 'fields' => array('rev_id', 'rev_page', 'rev_text_id', 'rev_user', 'rev_comment', 'rev_user_text',
- 'rev_timestamp', 'rev_minor_edit', 'rev_deleted'),
- 'conds' => array('rev_page' => $this->mPageHistory->mTitle->getArticleID() ),
- 'options' => array( 'USE INDEX' => 'page_timestamp' )
+ $queryInfo = array(
+ 'tables' => array('revision'),
+ 'fields' => Revision::selectFields(),
+ 'conds' => array('rev_page' => $this->mPageHistory->mTitle->getArticleID() ),
+ 'options' => array( 'USE INDEX' => array('revision' => 'page_timestamp') )
);
+ wfRunHooks( 'PageHistoryPager::getQueryInfo', array( &$this, &$queryInfo ) );
+ return $queryInfo;
}
function getIndexField() {
@@ -534,17 +699,17 @@ class PageHistoryPager extends ReverseChronologicalPager {
function formatRow( $row ) {
if ( $this->mLastRow ) {
- $latest = $this->mCounter == 1 && $this->mOffset == '';
+ $latest = $this->mCounter == 1 && $this->mIsFirst;
$firstInList = $this->mCounter == 1;
- $s = $this->mPageHistory->historyLine( $this->mLastRow, $row, $this->mCounter++,
- $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
+ $s = $this->mPageHistory->historyLine( $this->mLastRow, $row, $this->mCounter++,
+ $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
} else {
$s = '';
}
$this->mLastRow = $row;
return $s;
}
-
+
function getStartBody() {
$this->mLastRow = false;
$this->mCounter = 1;
@@ -553,7 +718,7 @@ class PageHistoryPager extends ReverseChronologicalPager {
function getEndBody() {
if ( $this->mLastRow ) {
- $latest = $this->mCounter == 1 && $this->mOffset == 0;
+ $latest = $this->mCounter == 1 && $this->mIsFirst;
$firstInList = $this->mCounter == 1;
if ( $this->mIsBackwards ) {
# Next row is unknown, but for UI reasons, probably exists if an offset has been specified
@@ -566,13 +731,11 @@ class PageHistoryPager extends ReverseChronologicalPager {
# The next row is the past-the-end row
$next = $this->mPastTheEndRow;
}
- $s = $this->mPageHistory->historyLine( $this->mLastRow, $next, $this->mCounter++,
- $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
+ $s = $this->mPageHistory->historyLine( $this->mLastRow, $next, $this->mCounter++,
+ $this->mPageHistory->getNotificationTimestamp(), $latest, $firstInList );
} else {
$s = '';
}
return $s;
}
}
-
-?>