r36182 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r36181‎ | r36182 | r36183 >
Date:01:51, 11 June 2008
Author:brion
Status:old
Tags:
Comment:
Reverting r36139 -- broke wikitext in linkstoimage AGAIN.
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,17 +643,15 @@
644644 array( 'LIMIT' => $limit + 1)
645645 );
646646
647 - $count = $dbr->numRows( $res );
648 - if ( $count == 0 ) {
 647+ if ( 0 == $dbr->numRows( $res ) ) {
649648 $wgOut->addHTML( "<div id='mw-imagepage-nolinkstoimage'>\n" );
650649 $wgOut->addWikiMsg( 'nolinkstoimage' );
651650 $wgOut->addHTML( "</div>\n" );
652651 return;
653652 }
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 - );
 653+ $wgOut->addHTML( "<div id='mw-imagepage-section-linkstoimage'>\n" );
 654+ $wgOut->addWikiMsg( 'linkstoimage' );
 655+ $wgOut->addHTML( "<ul class='mw-imagepage-linktoimage'>\n" );
658656
659657 $sk = $wgUser->getSkin();
660658 $count = 0;
@@ -677,14 +675,13 @@
678676 function imageRedirects()
679677 {
680678 global $wgUser, $wgOut;
681 -
 679+
682680 $redirects = $this->getTitle()->getRedirectsHere( NS_IMAGE );
683681 if ( count( $redirects ) == 0 ) return;
684682
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 - );
 683+ $wgOut->addHTML( "<div id='mw-imagepage-section-redirectstofile'>\n" );
 684+ $wgOut->addWikiMsg( 'redirectstofile' );
 685+ $wgOut->addHTML( "<ul class='mw-imagepage-redirectstofile'>\n" );
689686
690687 $sk = $wgUser->getSkin();
691688 foreach ( $redirects as $title ) {
@@ -703,10 +700,9 @@
704701 $dupes = $this->getDuplicates();
705702 if ( count( $dupes ) == 0 ) return;
706703
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 - );
 704+ $wgOut->addHTML( "<div id='mw-imagepage-section-duplicates'>\n" );
 705+ $wgOut->addWikiMsg( 'duplicatesoffile' );
 706+ $wgOut->addHTML( "<ul class='mw-imagepage-duplicates'>\n" );
711707
712708 $sk = $wgUser->getSkin();
713709 foreach ( $dupes as $file ) {
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 {{PLURAL:$1|page links|$1 pages link}} to this file:',
 1730+'linkstoimage' => 'The following 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 {{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:',
 1733+'redirectstofile' => 'The following files redirect to this file:',
 1734+'duplicatesoffile' => 'The following 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 {{PLURAL:$1|folgende Seite verwendet|folgenden $1 Seiten verwenden}} diese Datei:',
 1321+'linkstoimage' => 'Die folgenden Seiten benutzen diese Datei:',
13221322 'nolinkstoimage' => 'Keine Seite benutzt diese Datei.',
13231323 'morelinkstoimage' => '[[Special:Whatlinkshere/$1|Weitere Links]] für diese 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:',
 1324+'redirectstofile' => 'Die folgenden Dateien leiten auf diese Datei weiter:',
 1325+'duplicatesoffile' => 'Die folgenden 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.',

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r36139Support plural for 'linkstoimage', 'redirectstofile' and 'duplicatesoffile'raymond11:19, 10 June 2008

Status & tagging log