r28139 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r28138‎ | r28139 | r28140 >
Date:12:22, 4 December 2007
Author:raymond
Status:old
Tags:
Comment:
* PageHistory::diffButtons
** Remove commented out message, no one missed it since r10459
** Document function, use proper XML::element
* Remove other ancient unused messages
Modified paths:
  • /trunk/phase3/includes/PageHistory.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEn.php (modified) (history)
  • /trunk/phase3/maintenance/language/messages.inc (modified) (history)

Diff [purge]

Index: trunk/phase3/maintenance/language/messages.inc
@@ -641,11 +641,7 @@
642642 'diffs' => array(
643643 'history-title',
644644 'difference-title',
645 - 'loadingrev',
646645 'lineno',
647 - 'editcurrent',
648 - 'selectnewerversionfordiff',
649 - 'selectolderversionfordiff',
650646 'compareselectedversions',
651647 'editundo',
652648 'diff-multi',
Index: trunk/phase3/includes/PageHistory.php
@@ -326,14 +326,19 @@
327327 }
328328 }
329329
330 - /** @todo document */
 330+ /**
 331+ * Create radio buttons for page history
 332+ *
 333+ * @param object $rev Revision
 334+ * @param bool $firstInList Is this version the first one?
 335+ * @param int $counter A counter of what row number we're at, counted from the top row = 1.
 336+ * @return string HTML output for the radio buttons
 337+ */
331338 function diffButtons( $rev, $firstInList, $counter ) {
332339 if( $this->linesonpage > 1) {
333340 $radio = array(
334341 'type' => 'radio',
335342 'value' => $rev->getId(),
336 -# do we really need to flood this on every item?
337 -# 'title' => wfMsgHtml( 'selectolderversionfordiff' )
338343 );
339344
340345 if( !$rev->userCan( Revision::DELETED_TEXT ) ) {
@@ -342,7 +347,7 @@
343348
344349 /** @todo: move title texts to javascript */
345350 if ( $firstInList ) {
346 - $first = wfElement( 'input', array_merge(
 351+ $first = Xml::element( 'input', array_merge(
347352 $radio,
348353 array(
349354 'style' => 'visibility:hidden',
@@ -354,13 +359,13 @@
355360 } else {
356361 $checkmark = array();
357362 }
358 - $first = wfElement( 'input', array_merge(
 363+ $first = Xml::element( 'input', array_merge(
359364 $radio,
360365 $checkmark,
361366 array( 'name' => 'oldid' ) ) );
362367 $checkmark = array();
363368 }
364 - $second = wfElement( 'input', array_merge(
 369+ $second = Xml::element( 'input', array_merge(
365370 $radio,
366371 $checkmark,
367372 array( 'name' => 'diff' ) ) );
Index: trunk/phase3/languages/messages/MessagesEn.php
@@ -1216,11 +1216,7 @@
12171217 # Diffs
12181218 'history-title' => 'Revision history of "$1"',
12191219 'difference-title' => 'Difference between revisions of "$1"',
1220 -'loadingrev' => 'loading revision for diff',
12211220 'lineno' => 'Line $1:',
1222 -'editcurrent' => 'Edit the current version of this page',
1223 -'selectnewerversionfordiff' => 'Select a newer version for comparison',
1224 -'selectolderversionfordiff' => 'Select an older version for comparison',
12251221 'compareselectedversions' => 'Compare selected versions',
12261222 'editundo' => 'undo',
12271223 'diff-multi' => '({{PLURAL:$1|One intermediate revision|$1 intermediate revisions}} not shown.)',
Index: trunk/phase3/languages/messages/MessagesDe.php
@@ -877,11 +877,7 @@
878878 # Diffs
879879 'history-title' => 'Versionsgeschichte von „$1“',
880880 'difference-title' => 'Versionsunterschied: „$1“',
881 -'loadingrev' => 'Lade Versionen zur Unterscheidung',
882881 'lineno' => 'Zeile $1:',
883 -'editcurrent' => 'Die aktuelle Version dieser Seite bearbeiten',
884 -'selectnewerversionfordiff' => 'Eine neuere Version zum Vergleich auswählen',
885 -'selectolderversionfordiff' => 'Eine ältere Version zum Vergleich auswählen',
886882 'compareselectedversions' => 'Gewählte Versionen vergleichen',
887883 'editundo' => 'rückgängig',
888884 '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 @@
19301926 'tooltip-save' => 'Änderungen speichern',
19311927 'tooltip-preview' => 'Vorschau der Änderungen an dieser Seite. Bitte vor dem Speichern benutzen!',
19321928 'tooltip-diff' => 'Zeigt Änderungen am Text tabellarisch an',
1933 -'tooltip-compareselectedversions' => 'Unterschiede zwischen zwei ausgewählten Versionen dieser Seite vergleichen.',
 1929+'tooltip-compareselectedversions' => 'Unterschied zwischen zwei ausgewählten Versionen dieser Seite anzeigen.',
19341930 'tooltip-watch' => 'Füge diese Seite deiner Beobachtungsliste hinzu',
19351931 'tooltip-recreate' => 'Seite neu erstellen, obwohl sie gelöscht wurde.',
19361932 'tooltip-upload' => 'Hochladen starten',

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r10459Cleanup, html-safety and output...nikerabbit16:14, 12 August 2005

Status & tagging log