r36189 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r36188‎ | r36189 | r36190 >
Date:12:07, 11 June 2008
Author:raymond
Status:old
Tags:
Comment:
Reapply support of plural for 'linkstoimage', 'redirectstofile' and 'duplicatesoffile'
Now w/o breaking customized messages.
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 );
@@ -642,15 +642,15 @@
643643 __METHOD__,
644644 array( 'LIMIT' => $limit + 1)
645645 );
646 -
647 - if ( 0 == $dbr->numRows( $res ) ) {
 646+ $count = $dbr->numRows( $res );
 647+ if ( $count == 0 ) {
648648 $wgOut->addHTML( "<div id='mw-imagepage-nolinkstoimage'>\n" );
649649 $wgOut->addWikiMsg( 'nolinkstoimage' );
650650 $wgOut->addHTML( "</div>\n" );
651651 return;
652652 }
653653 $wgOut->addHTML( "<div id='mw-imagepage-section-linkstoimage'>\n" );
654 - $wgOut->addWikiMsg( 'linkstoimage' );
 654+ $wgOut->addWikiText( wfMsgExt( 'linkstoimage', array( 'parsemag' ), $count ) );
655655 $wgOut->addHTML( "<ul class='mw-imagepage-linktoimage'>\n" );
656656
657657 $sk = $wgUser->getSkin();
@@ -666,21 +666,21 @@
667667 }
668668 $wgOut->addHTML( "</ul></div>\n" );
669669 $res->free();
670 -
 670+
671671 // Add a links to [[Special:Whatlinkshere]]
672672 if ( $count > $limit )
673673 $wgOut->addWikiMsg( 'morelinkstoimage', $this->mTitle->getPrefixedDBkey() );
674674 }
675675
676 - function imageRedirects()
 676+ function imageRedirects()
677677 {
678678 global $wgUser, $wgOut;
679 -
 679+
680680 $redirects = $this->getTitle()->getRedirectsHere( NS_IMAGE );
681681 if ( count( $redirects ) == 0 ) return;
682682
683683 $wgOut->addHTML( "<div id='mw-imagepage-section-redirectstofile'>\n" );
684 - $wgOut->addWikiMsg( 'redirectstofile' );
 684+ $wgOut->addWikiText( wfMsgExt( 'redirectstofile', array( 'parsemag' ), count( $redirects ) ) );
685685 $wgOut->addHTML( "<ul class='mw-imagepage-redirectstofile'>\n" );
686686
687687 $sk = $wgUser->getSkin();
@@ -691,9 +691,9 @@
692692 $wgOut->addHTML( "</ul></div>\n" );
693693
694694 }
695 -
 695+
696696 function imageDupes() {
697 - global $wgOut, $wgUser;
 697+ global $wgOut, $wgUser;
698698
699699 $this->loadFile();
700700
@@ -701,7 +701,7 @@
702702 if ( count( $dupes ) == 0 ) return;
703703
704704 $wgOut->addHTML( "<div id='mw-imagepage-section-duplicates'>\n" );
705 - $wgOut->addWikiMsg( 'duplicatesoffile' );
 705+ $wgOut->addWikiText( wfMsgExt( 'duplicatesoffile', array( 'parsemag' ), count( $dupes ) ) );
706706 $wgOut->addHTML( "<ul class='mw-imagepage-duplicates'>\n" );
707707
708708 $sk = $wgUser->getSkin();
Index: trunk/phase3/languages/messages/MessagesEn.php
@@ -1726,11 +1726,11 @@
17271727 'filehist-filesize' => 'File size',
17281728 'filehist-comment' => 'Comment',
17291729 'imagelinks' => 'Links',
1730 -'linkstoimage' => 'The following pages link to this file:',
 1730+'linkstoimage' => 'The following {{PLURAL:$1|page links|$1 pages link}} to this file:',
17311731 'nolinkstoimage' => 'There are no pages that link to this file.',
17321732 'morelinkstoimage' => 'View [[Special:Whatlinkshere/$1|more links]] to this file.',
1733 -'redirectstofile' => 'The following files redirect to this file:',
1734 -'duplicatesoffile' => 'The following files are duplicates of this file:',
 1733+'redirectstofile' => 'The following {{PLURAL:$1|file redirects|$1 files redirect}} to this file:',
 1734+'duplicatesoffile' => 'The following {{PLURAL:$1|file is a duplicate|$1 files are duplicates}} of this file:',
17351735 'sharedupload' => 'This file is a shared upload and may be used by other projects.',
17361736 'shareduploadwiki' => 'Please see the $1 for further information.',
17371737 'shareduploadwiki-desc' => 'The description on its $1 on the shared repository is shown below.',
Index: trunk/phase3/languages/messages/MessagesDe.php
@@ -1317,11 +1317,11 @@
13181318 'filehist-filesize' => 'Dateigröße',
13191319 'filehist-comment' => 'Kommentar',
13201320 'imagelinks' => 'Verwendung',
1321 -'linkstoimage' => 'Die folgenden Seiten benutzen diese Datei:',
 1321+'linkstoimage' => 'Die {{PLURAL:$1|folgende Seite verwendet|folgenden $1 Seiten verwenden}} diese Datei:',
13221322 'nolinkstoimage' => 'Keine Seite benutzt diese Datei.',
13231323 'morelinkstoimage' => '[[Special:Whatlinkshere/$1|Weitere Links]] für diese Datei.',
1324 -'redirectstofile' => 'Die folgenden Dateien leiten auf diese Datei weiter:',
1325 -'duplicatesoffile' => 'Die folgenden Dateien sind Duplikate dieser Datei:',
 1324+'redirectstofile' => 'Die {{PLURAL:$1|folgende Datei leitet|folgenden $1 Dateien leiten}} auf diese Datei weiter:',
 1325+'duplicatesoffile' => 'Die {{PLURAL:$1|folgende Datei ist ein Duplikat|folgenden $1 Dateien sind Duplikate}} dieser Datei:',
13261326 'sharedupload' => 'Diese Datei ist ein gemeinsam genutzter Upload und kann von anderen Projekten verwendet werden.',
13271327 'shareduploadwiki' => 'Für weitere Informationen siehe die $1.',
13281328 'shareduploadwiki-desc' => 'Es folgt der Inhalt der $1 aus dem gemeinsam benutzten Repositorium.',

Follow-up revisions

RevisionCommit summaryAuthorDate
r36190Tweak for r36189: Back to addWikiMsg but now with the appropriate parameter. ...raymond12:53, 11 June 2008

Status & tagging log