renamed getDifferenceEngine to createDifferenceEngine
authordaniel <daniel.kinzler@wikimedia.de>
Tue, 17 Apr 2012 19:02:03 +0000 (21:02 +0200)
committerdaniel <daniel.kinzler@wikimedia.de>
Tue, 17 Apr 2012 19:02:03 +0000 (21:02 +0200)
includes/Article.php
includes/ContentHandler.php
includes/EditPage.php
includes/FeedUtils.php
includes/actions/RollbackAction.php
includes/api/ApiComparePages.php
includes/api/ApiQueryRevisions.php
includes/specials/SpecialComparePages.php
includes/specials/SpecialUndelete.php

index dbdae4c..2493448 100644 (file)
@@ -744,7 +744,7 @@ class Article extends Page {
                $oldid = $this->getOldID();
 
                $contentHandler = ContentHandler::getForTitle( $this->getTitle() );
-               $de = $contentHandler->getDifferenceEngine( $this->getContext(), $oldid, $diff, $rcid, $purge, $unhide );
+               $de = $contentHandler->createDifferenceEngine( $this->getContext(), $oldid, $diff, $rcid, $purge, $unhide );
 
                // DifferenceEngine directly fetched the revision:
                $this->mRevIdFetched = $de->mNewid;
index 8f3c6ee..9ee33c9 100644 (file)
@@ -420,9 +420,8 @@ abstract class ContentHandler {
      * @param $unhide boolean If set, allow viewing deleted revs
         *
         * @return DifferenceEngine
-     * @todo rename to createDifferenceEngine for consistency.
      */
-    public function getDifferenceEngine( IContextSource $context, $old = 0, $new = 0, $rcid = 0, #FIMXE: use everywhere!
+    public function createDifferenceEngine( IContextSource $context, $old = 0, $new = 0, $rcid = 0, #FIMXE: use everywhere!
                                          $refreshCache = false, $unhide = false ) {
 
         $this->checkModelName( $context->getTitle()->getModelName() );
index a0cf421..ed3e9d2 100644 (file)
@@ -2598,7 +2598,7 @@ HTML
                        $content2 = ContentHandler::makeContent( $this->textbox2, $this->getTitle(), $this->content_model, $this->content_format ); #XXX: handle parse errors?
 
                        $handler = ContentHandler::getForModelName( $this->content_model );
-                       $de = $handler->getDifferenceEngine( $this->mArticle->getContext() );
+                       $de = $handler->createDifferenceEngine( $this->mArticle->getContext() );
                        $de->setContent( $content2, $content1 );
                        $de->showDiff( wfMsgExt( 'yourtext', 'parseinline' ), wfMsg( 'storedversion' ) );
 
index bb47439..8a01ab8 100644 (file)
@@ -118,7 +118,7 @@ class FeedUtils {
                        // Don't bother generating the diff if we won't be able to show it
                        if ( $wgFeedDiffCutoff > 0 ) {
                 $contentHandler = ContentHandler::getForTitle( $title );
-                $de = $contentHandler->getDifferenceEngine( $title, $oldid, $newid );
+                $de = $contentHandler->createDifferenceEngine( $title, $oldid, $newid );
                                $diffText = $de->getDiff(
                                        wfMsg( 'previousrevision' ), // hack
                                        wfMsg( 'revisionasof',
index 5aba0b5..5c85d53 100644 (file)
@@ -110,7 +110,7 @@ class RollbackAction extends FormlessAction {
 
                if ( !$request->getBool( 'hidediff', false ) && !$this->getUser()->getBoolOption( 'norollbackdiff', false ) ) {
             $contentHandler = ContentHandler::getForTitle( $this->getTitle() );
-            $de = $contentHandler->getDifferenceEngine( $this->getContext(), $current->getId(), $newId, false, true );
+            $de = $contentHandler->createDifferenceEngine( $this->getContext(), $current->getId(), $newId, false, true );
                        $de->showDiff( '', '' );
                }
        }
index 1fac996..cd50030 100644 (file)
@@ -36,7 +36,7 @@ class ApiComparePages extends ApiBase {
                $rev2 = $this->revisionOrTitleOrId( $params['torev'], $params['totitle'], $params['toid'] );
 
         $contentHandler = ContentHandler::getForModelName( $rev1->getContentModelName() );
-        $de = $contentHandler->getDifferenceEngine( $this->getContext(),
+        $de = $contentHandler->createDifferenceEngine( $this->getContext(),
                        $rev1,
                        $rev2,
                        null, // rcid
index 11ea371..f5cb72b 100644 (file)
@@ -506,10 +506,10 @@ class ApiQueryRevisions extends ApiQueryBase {
                 $handler = ContentHandler::getForTitle( $title );
 
                                if ( !is_null( $this->difftotext ) ) {
-                                       $engine = $handler->getDifferenceEngine( $context );
+                                       $engine = $handler->createDifferenceEngine( $context );
                                        $engine->setText( $text, $this->difftotext ); #FIXME: use content objects!...
                                } else {
-                                       $engine = $handler->getDifferenceEngine( $context, $revision->getID(), $this->diffto );
+                                       $engine = $handler->createDifferenceEngine( $context, $revision->getID(), $this->diffto );
                                        $vals['diff']['from'] = $engine->getOldid();
                                        $vals['diff']['to'] = $engine->getNewid();
                                }
index 878bda0..ba433c8 100644 (file)
@@ -112,7 +112,7 @@ class SpecialComparePages extends SpecialPage {
 
                if( $rev1 && $rev2 ) {
             $contentHandler = ContentHandler::getForModelName( $rev1->getContentModelName() );
-                       $de = $contentHandler->getDifferenceEngine( $form->getContext(),
+                       $de = $contentHandler->createDifferenceEngine( $form->getContext(),
                                $rev1,
                                $rev2,
                                null, // rcid
index 47e88d0..c2ca7ce 100644 (file)
@@ -898,7 +898,7 @@ class SpecialUndelete extends SpecialPage {
         */
        function showDiff( $previousRev, $currentRev ) {
         $contentHandler = ContentHandler::getForTitle( $this->getTitle() );
-               $diffEngine = $contentHandler->getDifferenceEngine( $this->getContext() );
+               $diffEngine = $contentHandler->createDifferenceEngine( $this->getContext() );
                $diffEngine->showDiffStyle();
                $this->getOutput()->addHTML(
                        "<div>" .