* PageHistory::diffButtons

** Remove commented out message, no one missed it since r10459
** Document function, use proper XML::element
* Remove other ancient unused messages

diff --git a/includes/PageHistory.php b/includes/PageHistory.php
index 9886e2f..5f1388a 100644 (file)
--- a/includes/PageHistory.php
+++ b/includes/PageHistory.php
@@ -326,14 +326,19 @@ class PageHistory {
                }
        }
 
-       /** @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 ) ) {
@@ -342,7 +347,7 @@ class PageHistory {
 
                        /** @todo: move title texts to javascript */
                        if ( $firstInList ) {
-                               $first = wfElement( 'input', array_merge(
+                               $first = Xml::element( 'input', array_merge(
                                        $radio,
                                        array(
                                                'style' => 'visibility:hidden',
@@ -354,13 +359,13 @@ class PageHistory {
                                } else {
                                        $checkmark = array();
                                }
-                               $first = wfElement( 'input', array_merge(
+                               $first = Xml::element( 'input', array_merge(
                                        $radio,
                                        $checkmark,
                                        array( 'name'  => 'oldid' ) ) );
                                $checkmark = array();
                        }
-                       $second = wfElement( 'input', array_merge(
+                       $second = Xml::element( 'input', array_merge(
                                $radio,
                                $checkmark,
                                array( 'name'  => 'diff' ) ) );

diff --git a/languages/messages/MessagesDe.php b/languages/messages/MessagesDe.php
index 7bcc794..b5a0cfb 100644 (file)
--- a/languages/messages/MessagesDe.php
+++ b/languages/messages/MessagesDe.php
@@ -877,11 +877,7 @@ Stelle dadurch sicher, dass die Versionsgeschichte eines Artikel historisch korr
 # Diffs
 'history-title'             => 'Versionsgeschichte von „$1“',
 'difference-title'          => 'Versionsunterschied: „$1“',
-'loadingrev'                => 'Lade Versionen zur Unterscheidung',
 'lineno'                    => 'Zeile $1:',
-'editcurrent'               => 'Die aktuelle Version dieser Seite bearbeiten',
-'selectnewerversionfordiff' => 'Eine neuere Version zum Vergleich auswählen',
-'selectolderversionfordiff' => 'Eine ältere Version zum Vergleich auswählen',
 'compareselectedversions'   => 'Gewählte Versionen vergleichen',
 'editundo'                  => 'rückgängig',
 'diff-multi'                => "<span style='font-size: smaller'>(Der Versionsvergleich bezieht {{plural:$1|eine dazwischen liegende Version|$1 dazwischen liegende Versionen}} mit ein.)</span>",
@@ -1929,7 +1925,7 @@ Alle Transwiki-Import-Aktionen werden im [[Special:Log/import|Import-Logbuch]] p
 'tooltip-save'                    => 'Änderungen speichern',
 'tooltip-preview'                 => 'Vorschau der Änderungen an dieser Seite. Bitte vor dem Speichern benutzen!',
 'tooltip-diff'                    => 'Zeigt Änderungen am Text tabellarisch an',
-'tooltip-compareselectedversions' => 'Unterschiede zwischen zwei ausgewählten Versionen dieser Seite vergleichen.',
+'tooltip-compareselectedversions' => 'Unterschied zwischen zwei ausgewählten Versionen dieser Seite anzeigen.',
 'tooltip-watch'                   => 'Füge diese Seite deiner Beobachtungsliste hinzu',
 'tooltip-recreate'                => 'Seite neu erstellen, obwohl sie gelöscht wurde.',
 'tooltip-upload'                  => 'Hochladen starten',

diff --git a/languages/messages/MessagesEn.php b/languages/messages/MessagesEn.php
index c777df2..41474e4 100644 (file)
--- a/languages/messages/MessagesEn.php
+++ b/languages/messages/MessagesEn.php
@@ -1216,11 +1216,7 @@ Make sure that this change will maintain historical page continuity.
 # Diffs
 'history-title'             => 'Revision history of "$1"',
 'difference-title'          => 'Difference between revisions of "$1"',
-'loadingrev'                => 'loading revision for diff',
 'lineno'                    => 'Line $1:',
-'editcurrent'               => 'Edit the current version of this page',
-'selectnewerversionfordiff' => 'Select a newer version for comparison',
-'selectolderversionfordiff' => 'Select an older version for comparison',
 'compareselectedversions'   => 'Compare selected versions',
 'editundo'                  => 'undo',
 'diff-multi'                => '({{PLURAL:$1|One intermediate revision|$1 intermediate revisions}} not shown.)',

diff --git a/maintenance/language/messages.inc b/maintenance/language/messages.inc
index 525c60f..4d71300 100644 (file)
--- a/maintenance/language/messages.inc
+++ b/maintenance/language/messages.inc
@@ -641,11 +641,7 @@ $wgMessageStructure = array(
        'diffs' => array(
                'history-title',
                'difference-title',
-               'loadingrev',
                'lineno',
-               'editcurrent',
-               'selectnewerversionfordiff',
-               'selectolderversionfordiff',
                'compareselectedversions',
                'editundo',
                'diff-multi',