Use local context instead of global variables

[lhc/web/wiklou.git] / includes / specials / SpecialMergeHistory.php

diff --git a/includes/specials/SpecialMergeHistory.php b/includes/specials/SpecialMergeHistory.php
index f9c4a2c..0f046ab 100644 (file)
--- a/includes/specials/SpecialMergeHistory.php
+++ b/includes/specials/SpecialMergeHistory.php
@@ -40,12 +40,10 @@ class SpecialMergeHistory extends SpecialPage {
        }
 
        /**
-        * @param $request WebRequest
         * @return void
         */
-       private function loadRequestParams( $request ) {
-               global $wgUser;
-
+       private function loadRequestParams() {
+               $request = $this->getRequest();
                $this->mAction = $request->getVal( 'action' );
                $this->mTarget = $request->getVal( 'target' );
                $this->mDest = $request->getVal( 'dest' );
@@ -59,7 +57,7 @@ class SpecialMergeHistory extends SpecialPage {
                }
                $this->mComment = $request->getText( 'wpComment' );
 
-               $this->mMerge = $request->wasPosted() && $wgUser->matchEditToken( $request->getVal( 'wpEditToken' ) );
+               $this->mMerge = $request->wasPosted() && $this->getUser()->matchEditToken( $request->getVal( 'wpEditToken' ) );
                // target page
                if( $this->mSubmitted ) {
                        $this->mTargetObj = Title::newFromURL( $this->mTarget );
@@ -83,19 +81,17 @@ class SpecialMergeHistory extends SpecialPage {
        }
 
        public function execute( $par ) {
-               global $wgOut, $wgRequest, $wgUser;
-
-               if ( wfReadOnly() ) {
-                       $wgOut->readOnlyPage();
+               $user = $this->getUser();
+               if( !$this->userCanExecute( $user ) ) {
+                       $this->displayRestrictionError();
                        return;
                }
 
-               if( !$this->userCanExecute( $wgUser ) ) {
-                       $this->displayRestrictionError();
-                       return;
+               if ( wfReadOnly() ) {
+                       throw new ReadOnlyError;
                }
 
-               $this->loadRequestParams( $wgRequest );
+               $this->loadRequestParams();
 
                $this->setHeaders();
                $this->outputHeader();
@@ -132,7 +128,7 @@ class SpecialMergeHistory extends SpecialPage {
 
                if ( count( $errors ) ) {
                        $this->showMergeForm();
-                       $wgOut->addHTML( implode( "\n", $errors ) );
+                       $this->getOutput()->addHTML( implode( "\n", $errors ) );
                } else {
                        $this->showHistory();
                }
@@ -140,11 +136,11 @@ class SpecialMergeHistory extends SpecialPage {
        }
 
        function showMergeForm() {
-               global $wgOut, $wgScript;
+               global $wgScript;
 
-               $wgOut->addWikiMsg( 'mergehistory-header' );
+               $this->getOutput()->addWikiMsg( 'mergehistory-header' );
 
-               $wgOut->addHTML(
+               $this->getOutput()->addHTML(
                        Xml::openElement( 'form', array(
                                'method' => 'get',
                                'action' => $wgScript ) ) .
@@ -171,11 +167,8 @@ class SpecialMergeHistory extends SpecialPage {
        }
 
        private function showHistory() {
-               global $wgUser, $wgOut;
-
-               $this->sk = $this->getSkin();
-
-               $wgOut->setPageTitle( wfMsg( 'mergehistory' ) );
+               $out = $this->getOutput();
+               $out->setPageTitle( wfMsg( 'mergehistory' ) );
 
                $this->showMergeForm();
 
@@ -196,7 +189,7 @@ class SpecialMergeHistory extends SpecialPage {
                                'id' => 'merge'
                        )
                );
-               $wgOut->addHTML( $top );
+               $out->addHTML( $top );
 
                if( $haveRevisions ) {
                        # Format the user-visible controls (comment field, submission button)
@@ -223,27 +216,27 @@ class SpecialMergeHistory extends SpecialPage {
                                Xml::closeElement( 'table' ) .
                                Xml::closeElement( 'fieldset' );
 
-                       $wgOut->addHTML( $table );
+                       $out->addHTML( $table );
                }
 
-               $wgOut->addHTML(
+               $out->addHTML(
                        '<h2 id="mw-mergehistory">' .
                        wfMsgHtml( 'mergehistory-list' ) . "</h2>\n"
                );
 
                if( $haveRevisions ) {
-                       $wgOut->addHTML( $revisions->getNavigationBar() );
-                       $wgOut->addHTML( '<ul>' );
-                       $wgOut->addHTML( $revisions->getBody() );
-                       $wgOut->addHTML( '</ul>' );
-                       $wgOut->addHTML( $revisions->getNavigationBar() );
+                       $out->addHTML( $revisions->getNavigationBar() );
+                       $out->addHTML( '<ul>' );
+                       $out->addHTML( $revisions->getBody() );
+                       $out->addHTML( '</ul>' );
+                       $out->addHTML( $revisions->getNavigationBar() );
                } else {
-                       $wgOut->addWikiMsg( 'mergehistory-empty' );
+                       $out->addWikiMsg( 'mergehistory-empty' );
                }
 
                # Show relevant lines from the deletion log:
-               $wgOut->addHTML( '<h2>' . htmlspecialchars( LogPage::logName( 'merge' ) ) . "</h2>\n" );
-               LogEventsList::showLogExtract( $wgOut, 'merge', $this->mTargetObj->getPrefixedText() );
+               $out->addHTML( '<h2>' . htmlspecialchars( LogPage::logName( 'merge' ) ) . "</h2>\n" );
+               LogEventsList::showLogExtract( $out, 'merge', $this->mTargetObj );
 
                # When we submit, go by page ID to avoid some nasty but unlikely collisions.
                # Such would happen if a page was renamed after the form loaded, but before submit
@@ -251,16 +244,14 @@ class SpecialMergeHistory extends SpecialPage {
                $misc .= Html::hidden( 'destID', $this->mDestObj->getArticleID() );
                $misc .= Html::hidden( 'target', $this->mTarget );
                $misc .= Html::hidden( 'dest', $this->mDest );
-               $misc .= Html::hidden( 'wpEditToken', $wgUser->editToken() );
+               $misc .= Html::hidden( 'wpEditToken', $this->getUser()->editToken() );
                $misc .= Xml::closeElement( 'form' );
-               $wgOut->addHTML( $misc );
+               $out->addHTML( $misc );
 
                return true;
        }
 
        function formatRevisionRow( $row ) {
-               global $wgLang;
-
                $rev = new Revision( $row );
 
                $stxt = '';
@@ -269,9 +260,9 @@ class SpecialMergeHistory extends SpecialPage {
                $ts = wfTimestamp( TS_MW, $row->rev_timestamp );
                $checkBox = Xml::radio( 'mergepoint', $ts, false );
 
-               $pageLink = $this->sk->linkKnown(
+               $pageLink = Linker::linkKnown(
                        $rev->getTitle(),
-                       htmlspecialchars( $wgLang->timeanddate( $ts ) ),
+                       htmlspecialchars( $this->getLang()->timeanddate( $ts ) ),
                        array(),
                        array( 'oldid' => $rev->getId() )
                );
@@ -283,7 +274,7 @@ class SpecialMergeHistory extends SpecialPage {
                if( !$rev->userCan( Revision::DELETED_TEXT ) ) {
                        $last = $this->message['last'];
                } elseif( isset( $this->prevId[$row->rev_id] ) ) {
-                       $last = $this->sk->linkKnown(
+                       $last = Linker::linkKnown(
                                $rev->getTitle(),
                                $this->message['last'],
                                array(),
@@ -294,13 +285,13 @@ class SpecialMergeHistory extends SpecialPage {
                        );
                }
 
-               $userLink = $this->sk->revUserTools( $rev );
+               $userLink = Linker::revUserTools( $rev );
 
                $size = $row->rev_len;
                if( !is_null( $size ) ) {
-                       $stxt = $this->sk->formatRevisionSize( $size );
+                       $stxt = Linker::formatRevisionSize( $size );
                }
-               $comment = $this->sk->revComment( $rev );
+               $comment = Linker::revComment( $rev );
 
                return "<li>$checkBox ($last) $pageLink . . $userLink $stxt $comment</li>";
        }
@@ -310,15 +301,13 @@ class SpecialMergeHistory extends SpecialPage {
         * @return string
         */
        function getPageLink( $row, $titleObj, $ts, $target ) {
-               global $wgLang;
-
                if( !$this->userCan( $row, Revision::DELETED_TEXT ) ) {
                        return '<span class="history-deleted">' .
-                               $wgLang->timeanddate( $ts, true ) . '</span>';
+                               $this->getLang()->timeanddate( $ts, true ) . '</span>';
                } else {
-                       $link = $this->sk->linkKnown(
+                       $link = Linker::linkKnown(
                                $titleObj,
-                               $wgLang->timeanddate( $ts, true ),
+                               $this->getLang()->timeanddate( $ts, true ),
                                array(),
                                array(
                                        'target' => $target,
@@ -333,7 +322,6 @@ class SpecialMergeHistory extends SpecialPage {
        }
 
        function merge() {
-               global $wgOut;
                # Get the titles directly from the IDs, in case the target page params
                # were spoofed. The queries are done based on the IDs, so it's best to
                # keep it consistent...
@@ -359,7 +347,7 @@ class SpecialMergeHistory extends SpecialPage {
                );
                # Destination page must exist with revisions
                if( !$maxtimestamp ) {
-                       $wgOut->addWikiMsg( 'mergehistory-fail' );
+                       $this->getOutput()->addWikiMsg( 'mergehistory-fail' );
                        return false;
                }
                # Get the latest timestamp of the source
@@ -371,7 +359,7 @@ class SpecialMergeHistory extends SpecialPage {
                );
                # $this->mTimestamp must be older than $maxtimestamp
                if( $this->mTimestamp >= $maxtimestamp ) {
-                       $wgOut->addWikiMsg( 'mergehistory-fail' );
+                       $this->getOutput()->addWikiMsg( 'mergehistory-fail' );
                        return false;
                }
                # Update the revisions
@@ -440,7 +428,7 @@ class SpecialMergeHistory extends SpecialPage {
                $destTitle->invalidateCache(); // update histories
                # Check if this did anything
                if( !$count ) {
-                       $wgOut->addWikiMsg( 'mergehistory-fail' );
+                       $this->getOutput()->addWikiMsg( 'mergehistory-fail' );
                        return false;
                }
                # Update our logs
@@ -450,7 +438,7 @@ class SpecialMergeHistory extends SpecialPage {
                        array( $destTitle->getPrefixedText(), $timestampLimit )
                );
 
-               $wgOut->addHTML(
+               $this->getOutput()->addHTML(
                        wfMsgExt( 'mergehistory-success', array('parseinline'),
                        $targetTitle->getPrefixedText(), $destTitle->getPrefixedText(), $count ) );
 
@@ -478,7 +466,7 @@ class MergeHistoryPager extends ReverseChronologicalPager {
                );
                $this->maxTimestamp = $maxtimestamp;
 
-               parent::__construct();
+               parent::__construct( $form->getContext() );
        }
 
        function getStartBody() {