r36139 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r36138‎ | r36139 | r36140 >
Date:11:19, 10 June 2008
Author:raymond
Status:old
Tags:
Comment:
Support plural for 'linkstoimage', 'redirectstofile' and 'duplicatesoffile'
Modified paths:
  • /trunk/phase3/includes/ImagePage.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesEn.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/ImagePage.php
@@ -630,7 +630,7 @@
631631 function imageLinks()
632632 {
633633 global $wgUser, $wgOut;
634 -
 634+
635635 $limit = 100;
636636
637637 $dbr = wfGetDB( DB_SLAVE );
@@ -643,15 +643,17 @@
644644 array( 'LIMIT' => $limit + 1)
645645 );
646646
647 - if ( 0 == $dbr->numRows( $res ) ) {
 647+ $count = $dbr->numRows( $res );
 648+ if ( $count == 0 ) {
648649 $wgOut->addHTML( "<div id='mw-imagepage-nolinkstoimage'>\n" );
649650 $wgOut->addWikiMsg( 'nolinkstoimage' );
650651 $wgOut->addHTML( "</div>\n" );
651652 return;
652653 }
653 - $wgOut->addHTML( "<div id='mw-imagepage-section-linkstoimage'>\n" );
654 - $wgOut->addWikiMsg( 'linkstoimage' );
655 - $wgOut->addHTML( "<ul class='mw-imagepage-linktoimage'>\n" );
 654+ $wgOut->addHTML( "<div id='mw-imagepage-section-linkstoimage'>\n" .
 655+ wfMsgExt( 'linkstoimage', array( 'parseinline', 'escape' ), min( $count, $limit ) ) .
 656+ "<ul class='mw-imagepage-linktoimage'>\n"
 657+ );
656658
657659 $sk = $wgUser->getSkin();
658660 $count = 0;
@@ -675,13 +677,14 @@
676678 function imageRedirects()
677679 {
678680 global $wgUser, $wgOut;
679 -
 681+
680682 $redirects = $this->getTitle()->getRedirectsHere( NS_IMAGE );
681683 if ( count( $redirects ) == 0 ) return;
682684
683 - $wgOut->addHTML( "<div id='mw-imagepage-section-redirectstofile'>\n" );
684 - $wgOut->addWikiMsg( 'redirectstofile' );
685 - $wgOut->addHTML( "<ul class='mw-imagepage-redirectstofile'>\n" );
 685+ $wgOut->addHTML( "<div id='mw-imagepage-section-redirectstofile'>\n" .
 686+ wfMsgExt( 'redirectstofile', array( 'parseinline', 'escape' ), count( $redirects ) ) .
 687+ "<ul class='mw-imagepage-redirectstofile'>\n"
 688+ );
686689
687690 $sk = $wgUser->getSkin();
688691 foreach ( $redirects as $title ) {
@@ -700,9 +703,10 @@
701704 $dupes = $this->getDuplicates();
702705 if ( count( $dupes ) == 0 ) return;
703706
704 - $wgOut->addHTML( "<div id='mw-imagepage-section-duplicates'>\n" );
705 - $wgOut->addWikiMsg( 'duplicatesoffile' );
706 - $wgOut->addHTML( "<ul class='mw-imagepage-duplicates'>\n" );
 707+ $wgOut->addHTML( "<div id='mw-imagepage-section-duplicates'>\n" .
 708+ wfMsgExt( 'duplicatesoffile', array( 'parseinline', 'escape' ), count( $dupes ) ) .
 709+ "<ul class='mw-imagepage-duplicates'>\n"
 710+ );
707711
708712 $sk = $wgUser->getSkin();
709713 foreach ( $dupes as $file ) {
Index: trunk/phase3/languages/messages/MessagesEn.php
@@ -1725,11 +1725,11 @@
17261726 'filehist-filesize' => 'File size',
17271727 'filehist-comment' => 'Comment',
17281728 'imagelinks' => 'Links',
1729 -'linkstoimage' => 'The following pages link to this file:',
 1729+'linkstoimage' => 'The following {{PLURAL:$1|page links|$1 pages link}} to this file:',
17301730 'nolinkstoimage' => 'There are no pages that link to this file.',
17311731 'morelinkstoimage' => 'View [[Special:Whatlinkshere/$1|more links]] to this file.',
1732 -'redirectstofile' => 'The following files redirect to this file:',
1733 -'duplicatesoffile' => 'The following files are duplicates of this file:',
 1732+'redirectstofile' => 'The following {{PLURAL:$1|file redirects|$1 files redirect}} to this file:',
 1733+'duplicatesoffile' => 'The following {{PLURAL:$1|file is a duplicate|$1 files are duplicates}} of this file:',
17341734 'sharedupload' => 'This file is a shared upload and may be used by other projects.',
17351735 'shareduploadwiki' => 'Please see the $1 for further information.',
17361736 'shareduploadwiki-desc' => 'The description on its $1 on the shared repository is shown below.',
Index: trunk/phase3/languages/messages/MessagesDe.php
@@ -1316,11 +1316,11 @@
13171317 'filehist-filesize' => 'Dateigröße',
13181318 'filehist-comment' => 'Kommentar',
13191319 'imagelinks' => 'Verwendung',
1320 -'linkstoimage' => 'Die folgenden Seiten benutzen diese Datei:',
 1320+'linkstoimage' => 'Die {{PLURAL:$1|folgende Seite verwendet|folgenden $1 Seiten verwenden}} diese Datei:',
13211321 'nolinkstoimage' => 'Keine Seite benutzt diese Datei.',
13221322 'morelinkstoimage' => '[[Special:Whatlinkshere/$1|Weitere Links]] für diese Datei.',
1323 -'redirectstofile' => 'Die folgenden Dateien leiten auf diese Datei weiter:',
1324 -'duplicatesoffile' => 'Die folgenden Dateien sind Duplikate dieser Datei:',
 1323+'redirectstofile' => 'Die {{PLURAL:$1|folgende Datei leitet|folgenden $1 Dateien leiten}} auf diese Datei weiter:',
 1324+'duplicatesoffile' => 'Die {{PLURAL:$1|folgende Datei ist ein Duplikat|folgenden $1 Dateien sind Duplikate}} dieser Datei:',
13251325 'sharedupload' => 'Diese Datei ist ein gemeinsam genutzter Upload und kann von anderen Projekten verwendet werden.',
13261326 'shareduploadwiki' => 'Für weitere Informationen siehe die $1.',
13271327 'shareduploadwiki-desc' => 'Es folgt der Inhalt der $1 aus dem gemeinsam benutzten Repositorium.',

Follow-up revisions

RevisionCommit summaryAuthorDate
r36182Reverting r36139 -- broke wikitext in linkstoimage AGAIN.brion01:51, 11 June 2008

Status & tagging log