diff --git a/includes/AutoLoader.php b/includes/AutoLoader.php index 9ebd53afe9e..038604c7c0e 100644 --- a/includes/AutoLoader.php +++ b/includes/AutoLoader.php @@ -93,8 +93,6 @@ $wgAutoloadLocalClasses = array( 'HistoryBlob' => 'includes/HistoryBlob.php', 'HistoryBlobCurStub' => 'includes/HistoryBlob.php', 'HistoryBlobStub' => 'includes/HistoryBlob.php', - 'HistoryPage' => 'includes/HistoryPage.php', - 'HistoryPager' => 'includes/HistoryPage.php', 'Hooks' => 'includes/Hooks.php', 'Html' => 'includes/Html.php', 'HTMLCheckField' => 'includes/HTMLForm.php', @@ -159,8 +157,6 @@ $wgAutoloadLocalClasses = array( 'OldChangesList' => 'includes/ChangesList.php', 'OutputPage' => 'includes/OutputPage.php', 'Page' => 'includes/WikiPage.php', - 'PageHistory' => 'includes/HistoryPage.php', - 'PageHistoryPager' => 'includes/HistoryPage.php', 'PageQueryPage' => 'includes/PageQueryPage.php', 'Pager' => 'includes/Pager.php', 'PasswordError' => 'includes/User.php', @@ -258,6 +254,9 @@ $wgAutoloadLocalClasses = array( # includes/actions 'CreditsAction' => 'includes/actions/CreditsAction.php', 'DeletetrackbackAction' => 'includes/actions/DeletetrackbackAction.php', + 'HistoryAction' => 'includes/actions/HistoryAction.php', + 'HistoryPage' => 'includes/actions/HistoryAction.php', + 'HistoryPager' => 'includes/actions/HistoryAction.php', 'InfoAction' => 'includes/actions/InfoAction.php', 'MarkpatrolledAction' => 'includes/actions/MarkpatrolledAction.php', 'PurgeAction' => 'includes/actions/PurgeAction.php', diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php index b05856d86a4..dbdb52189fb 100644 --- a/includes/DefaultSettings.php +++ b/includes/DefaultSettings.php @@ -5135,6 +5135,7 @@ $wgMaxRedirectLinksRetrieved = 500; $wgActions = array( 'credits' => true, 'deletetrackback' => true, + 'history' => true, 'info' => true, 'markpatrolled' => true, 'purge' => true, diff --git a/includes/Wiki.php b/includes/Wiki.php index 1ba71a5f7a2..277bd998877 100644 --- a/includes/Wiki.php +++ b/includes/Wiki.php @@ -503,13 +503,6 @@ class MediaWiki { } } break; - case 'history': - if ( $request->getFullRequestURL() == $title->getInternalURL( 'action=history' ) ) { - $output->setSquidMaxage( $wgSquidMaxage ); - } - $history = new HistoryPage( $article ); - $history->history(); - break; default: if ( wfRunHooks( 'UnknownAction', array( $act, $article ) ) ) { $output->showErrorPage( 'nosuchaction', 'nosuchactiontext' ); diff --git a/includes/HistoryPage.php b/includes/actions/HistoryAction.php similarity index 74% rename from includes/HistoryPage.php rename to includes/actions/HistoryAction.php index a5508ce63ec..e5954748dd5 100644 --- a/includes/HistoryPage.php +++ b/includes/actions/HistoryAction.php @@ -1,8 +1,8 @@ history() to print the - * history. - * */ -class HistoryPage { +class HistoryAction extends FormlessAction { const DIR_PREV = 0; const DIR_NEXT = 1; - /** Contains the Article object. Passed on construction. */ - private $article; - /** The $article title object. Found on construction. */ - private $title; - /** Shortcut to the user Skin object. */ - private $skin; + public function getName() { + return 'history'; + } + + public function getRestriction() { + return 'read'; + } /** - * Construct a new HistoryPage. + * Get the Page object we are working on. * - * @param $article Article + * @return Page */ - function __construct( $article ) { - global $wgUser; - $this->article = $article; - $this->title = $article->getTitle(); - $this->skin = $wgUser->getSkin(); - $this->preCacheMessages(); - } - - /** Get the Article object we are working on. */ public function getArticle() { - return $this->article; - } - - /** Get the Title object. */ - public function getTitle() { - return $this->title; + return $this->page; } /** @@ -67,53 +50,56 @@ class HistoryPage { * Print the history page for an article. * @return nothing */ - function history() { - global $wgOut, $wgRequest, $wgScript; + function onView() { + global $wgSquidMaxage, $wgScript; - /** - * Allow client caching. - */ - if ( $wgOut->checkLastModified( $this->article->getTouched() ) ) + $out = $this->getOutput(); + $request = $this->getRequest(); + + // Allow client caching. + if ( $out->checkLastModified( $this->page->getTouched() ) ) { return; // Client cache fresh and headers sent, nothing more to do. + } + + if ( $request->getFullRequestURL() == $this->getTitle()->getInternalURL( 'action=history' ) ) { + $out->setSquidMaxage( $wgSquidMaxage ); + } wfProfileIn( __METHOD__ ); + $this->preCacheMessages(); + // Setup page variables. - $wgOut->setPageTitle( wfMsg( 'history-title', $this->title->getPrefixedText() ) ); - $wgOut->setPageTitleActionText( wfMsg( 'history_short' ) ); - $wgOut->setArticleFlag( false ); - $wgOut->setArticleRelated( true ); - $wgOut->setRobotPolicy( 'noindex,nofollow' ); - $wgOut->setSyndicated( true ); - $wgOut->setFeedAppendQuery( 'action=history' ); - $wgOut->addModules( array( 'mediawiki.legacy.history', 'mediawiki.action.history' ) ); + $out->setPageTitle( wfMsg( 'history-title', $this->getTitle()->getPrefixedText() ) ); + $out->setRobotPolicy( 'noindex,nofollow' ); + $out->setFeedAppendQuery( 'action=history' ); + $out->addModules( array( 'mediawiki.legacy.history', 'mediawiki.action.history' ) ); // Creation of a subtitle link pointing to [[Special:Log]] $logPage = SpecialPage::getTitleFor( 'Log' ); - $logLink = $this->skin->link( + $logLink = Linker::linkKnown( $logPage, wfMsgHtml( 'viewpagelogs' ), array(), - array( 'page' => $this->title->getPrefixedText() ), - array( 'known', 'noclasses' ) + array( 'page' => $this->getTitle()->getPrefixedText() ) ); - $wgOut->setSubtitle( $logLink ); + $out->setSubtitle( $logLink ); // Handle atom/RSS feeds. - $feedType = $wgRequest->getVal( 'feed' ); + $feedType = $request->getVal( 'feed' ); if ( $feedType ) { wfProfileOut( __METHOD__ ); return $this->feed( $feedType ); } // Fail nicely if article doesn't exist. - if ( !$this->title->exists() ) { - $wgOut->addWikiMsg( 'nohistory' ); + if ( !$this->getTitle()->exists() ) { + $out->addWikiMsg( 'nohistory' ); # show deletion/move log if there is an entry LogEventsList::showLogExtract( - $wgOut, + $out, array( 'delete', 'move' ), - $this->title->getPrefixedText(), + $this->getTitle()->getPrefixedText(), '', array( 'lim' => 10, 'conds' => array( "log_action != 'revision'" ), @@ -128,32 +114,32 @@ class HistoryPage { /** * Add date selector to quickly get to a certain time */ - $year = $wgRequest->getInt( 'year' ); - $month = $wgRequest->getInt( 'month' ); - $tagFilter = $wgRequest->getVal( 'tagfilter' ); + $year = $request->getInt( 'year' ); + $month = $request->getInt( 'month' ); + $tagFilter = $request->getVal( 'tagfilter' ); $tagSelector = ChangeTags::buildTagFilterSelector( $tagFilter ); /** * Option to show only revisions that have been (partially) hidden via RevisionDelete */ - if ( $wgRequest->getBool( 'deleted' ) ) { + if ( $request->getBool( 'deleted' ) ) { $conds = array( "rev_deleted != '0'" ); } else { $conds = array(); } $checkDeleted = Xml::checkLabel( wfMsg( 'history-show-deleted' ), - 'deleted', 'mw-show-deleted-only', $wgRequest->getBool( 'deleted' ) ) . "\n"; + 'deleted', 'mw-show-deleted-only', $request->getBool( 'deleted' ) ) . "\n"; // Add the general form $action = htmlspecialchars( $wgScript ); - $wgOut->addHTML( + $out->addHTML( "
' ); - wfRunHooks( 'PageHistoryBeforeList', array( &$this->article ) ); + wfRunHooks( 'PageHistoryBeforeList', array( &$this->page ) ); // Create and output the list. $pager = new HistoryPager( $this, $year, $month, $tagFilter, $conds ); - $wgOut->addHTML( + $out->addHTML( $pager->getNavigationBar() . $pager->getBody() . $pager->getNavigationBar() ); - $wgOut->preventClickjacking( $pager->getPreventClickjacking() ); + $out->preventClickjacking( $pager->getPreventClickjacking() ); wfProfileOut( __METHOD__ ); } @@ -183,15 +169,15 @@ class HistoryPage { * * @param $limit Integer: the limit number of revisions to get * @param $offset Integer - * @param $direction Integer: either HistoryPage::DIR_PREV or HistoryPage::DIR_NEXT + * @param $direction Integer: either HistoryAction::DIR_PREV or HistoryAction::DIR_NEXT * @return ResultWrapper */ function fetchRevisions( $limit, $offset, $direction ) { $dbr = wfGetDB( DB_SLAVE ); - if ( $direction == HistoryPage::DIR_PREV ) { + if ( $direction == self::DIR_PREV ) { list( $dirs, $oper ) = array( "ASC", ">=" ); - } else { /* $direction == HistoryPage::DIR_NEXT */ + } else { /* $direction == self::DIR_NEXT */ list( $dirs, $oper ) = array( "DESC", "<=" ); } @@ -201,7 +187,7 @@ class HistoryPage { $offsets = array(); } - $page_id = $this->title->getArticleID(); + $page_id = $this->getTitle()->getArticleID(); return $dbr->select( 'revision', Revision::selectFields(), @@ -218,29 +204,29 @@ class HistoryPage { * @param $type String: feed type */ function feed( $type ) { - global $wgFeedClasses, $wgRequest, $wgFeedLimit; + global $wgFeedClasses, $wgFeedLimit; if ( !FeedUtils::checkFeedOutput( $type ) ) { return; } $feed = new $wgFeedClasses[$type]( - $this->title->getPrefixedText() . ' - ' . + $this->getTitle()->getPrefixedText() . ' - ' . wfMsgForContent( 'history-feed-title' ), wfMsgForContent( 'history-feed-description' ), - $this->title->getFullUrl( 'action=history' ) + $this->getTitle()->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 ); + $limit = $this->getRequest()->getInt( 'limit', 10 ); if ( $limit > $wgFeedLimit || $limit < 1 ) { $limit = 10; } - $items = $this->fetchRevisions( $limit, 0, HistoryPage::DIR_NEXT ); + $items = $this->fetchRevisions( $limit, 0, self::DIR_NEXT ); // Generate feed elements enclosed between header and footer. $feed->outHeader(); - if ( $items ) { + if ( $items->numRows() ) { foreach ( $items as $row ) { $feed->outItem( $this->feedItem( $row ) ); } @@ -251,14 +237,13 @@ class HistoryPage { } function feedEmpty() { - global $wgOut; return new FeedItem( wfMsgForContent( 'nohistory' ), - $wgOut->parse( wfMsgForContent( 'history-feed-empty' ) ), - $this->title->getFullUrl(), + $this->getOutput()->parse( wfMsgForContent( 'history-feed-empty' ) ), + $this->getTitle()->getFullUrl(), wfTimestamp( TS_MW ), '', - $this->title->getTalkPage()->getFullUrl() + $this->getTitle()->getTalkPage()->getFullUrl() ); } @@ -272,10 +257,11 @@ class HistoryPage { */ function feedItem( $row ) { $rev = new Revision( $row ); - $rev->setTitle( $this->title ); + $titleObj = $this->getTitle(); + $rev->setTitle( $titleObj ); $text = FeedUtils::formatDiffRow( - $this->title, - $this->title->getPreviousRevisionID( $rev->getId() ), + $titleObj, + $titleObj->getPreviousRevisionID( $rev->getId() ), $rev->getId(), $rev->getTimestamp(), $rev->getComment() @@ -296,10 +282,10 @@ class HistoryPage { return new FeedItem( $title, $text, - $this->title->getFullUrl( 'diff=' . $rev->getId() . '&oldid=prev' ), + $titleObj->getFullUrl( 'diff=' . $rev->getId() . '&oldid=prev' ), $rev->getTimestamp(), $rev->getUserText(), - $this->title->getTalkPage()->getFullUrl() + $titleObj->getTalkPage()->getFullUrl() ); } } @@ -308,14 +294,13 @@ class HistoryPage { * @ingroup Pager */ class HistoryPager extends ReverseChronologicalPager { - public $lastRow = false, $counter, $historyPage, $title, $buttons, $conds; + public $lastRow = false, $counter, $history, $buttons, $conds; protected $oldIdChecked; protected $preventClickjacking = false; - function __construct( $historyPage, $year = '', $month = '', $tagFilter = '', $conds = array() ) { + function __construct( $history, $year = '', $month = '', $tagFilter = '', $conds = array() ) { parent::__construct(); - $this->historyPage = $historyPage; - $this->title = $this->historyPage->getTitle(); + $this->history = $history; $this->tagFilter = $tagFilter; $this->getDateCond( $year, $month ); $this->conds = $conds; @@ -323,11 +308,7 @@ class HistoryPager extends ReverseChronologicalPager { // For hook compatibility... function getArticle() { - return $this->historyPage->getArticle(); - } - - function getTitle() { - return $this->title; + return $this->history->getArticle(); } function getSqlComment() { @@ -343,7 +324,7 @@ class HistoryPager extends ReverseChronologicalPager { 'tables' => array( 'revision' ), 'fields' => Revision::selectFields(), 'conds' => array_merge( - array( 'rev_page' => $this->title->getArticleID() ), + array( 'rev_page' => $this->getTitle()->getArticleID() ), $this->conds ), 'options' => array( 'USE INDEX' => array( 'revision' => 'page_timestamp' ) ), 'join_conds' => array( 'tag_summary' => array( 'LEFT JOIN', 'ts_rev_id=rev_id' ) ), @@ -370,7 +351,7 @@ class HistoryPager extends ReverseChronologicalPager { $firstInList = $this->counter == 1; $this->counter++; $s = $this->historyLine( $this->lastRow, $row, - $this->title->getNotificationTimestamp(), $latest, $firstInList ); + $this->getTitle()->getNotificationTimestamp(), $latest, $firstInList ); } else { $s = ''; } @@ -384,15 +365,15 @@ class HistoryPager extends ReverseChronologicalPager { * @return string HTML output */ function getStartBody() { - global $wgScript, $wgUser, $wgOut; + global $wgScript; $this->lastRow = false; $this->counter = 1; $this->oldIdChecked = 0; - $wgOut->wrapWikiMsg( "