From: Raimond Spekking <raymond@users.mediawiki.org>
Date: Tue, 4 Dec 2007 12:22:23 +0000 (+0000)
Subject: * PageHistory::diffButtons
X-Git-Tag: 1.31.0-rc.0~50559
X-Git-Url: http://git.cyclocoop.org/%22.%24h.%22?a=commitdiff_plain;h=e98d0bf64c9a0d051f2414576a9c7e68b9c454d0;p=lhc%2Fweb%2Fwiklou.git

* 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 9886e2fc8b..5f1388ae8b 100644
--- 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 7bcc7949b3..b5a0cfb520 100644
--- 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 c777df2bbb..41474e47ab 100644
--- 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 525c60f5ed..4d713000b5 100644
--- 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',