r23523 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r23522‎ | r23523 | r23524 >
Date:21:43, 28 June 2007
Author:raymond
Status:old
Tags:
Comment:
* Remove hard coded colons from upload form code, add them into message file
Modified paths:
  • /trunk/phase3/includes/SpecialUpload.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/SpecialUpload.php
@@ -777,13 +777,13 @@
778778 <table border='0'>
779779 <tr>
780780 {$this->uploadFormTextTop}
781 - <td align='right' valign='top'><label for='wpUploadFile'>{$sourcefilename}:</label></td>
 781+ <td align='right' valign='top'><label for='wpUploadFile'>{$sourcefilename}</label></td>
782782 <td align='left'>
783783 {$filename_form}
784784 </td>
785785 </tr>
786786 <tr>
787 - <td align='right'><label for='wpDestFile'>{$destfilename}:</label></td>
 787+ <td align='right'><label for='wpDestFile'>{$destfilename}</label></td>
788788 <td align='left'>
789789 <input tabindex='2' type='text' name='wpDestFile' id='wpDestFile' size='40' value="$encDestName" />
790790 </td>
@@ -803,7 +803,7 @@
804804 if ( $licenseshtml != '' ) {
805805 global $wgStylePath;
806806 $wgOut->addHTML( "
807 - <td align='right'><label for='wpLicense'>$license:</label></td>
 807+ <td align='right'><label for='wpLicense'>$license</label></td>
808808 <td align='left'>
809809 <script type='text/javascript' src=\"$wgStylePath/common/upload.js\"></script>
810810 <select name='wpLicense' id='wpLicense' tabindex='4'
@@ -824,12 +824,12 @@
825825 $uploadsource = htmlspecialchars( $this->mCopyrightSource );
826826
827827 $wgOut->addHTML( "
828 - <td align='right' nowrap='nowrap'><label for='wpUploadCopyStatus'>$filestatus:</label></td>
 828+ <td align='right' nowrap='nowrap'><label for='wpUploadCopyStatus'>$filestatus</label></td>
829829 <td><input tabindex='5' type='text' name='wpUploadCopyStatus' id='wpUploadCopyStatus'
830830 value=\"$copystatus\" size='40' /></td>
831831 </tr>
832832 <tr>
833 - <td align='right'><label for='wpUploadCopyStatus'>$filesource:</label></td>
 833+ <td align='right'><label for='wpUploadCopyStatus'>$filesource</label></td>
834834 <td><input tabindex='6' type='text' name='wpUploadSource' id='wpUploadCopyStatus'
835835 value=\"$uploadsource\" size='40' /></td>
836836 </tr>
Index: trunk/phase3/languages/messages/MessagesEn.php
@@ -1360,8 +1360,8 @@
13611361 'filename' => 'Filename',
13621362 'filedesc' => 'Summary',
13631363 'fileuploadsummary' => 'Summary:',
1364 -'filestatus' => 'Copyright status',
1365 -'filesource' => 'Source',
 1364+'filestatus' => 'Copyright status:',
 1365+'filesource' => 'Source:',
13661366 'uploadedfiles' => 'Uploaded files',
13671367 'ignorewarning' => 'Ignore warning and save file anyway.',
13681368 'ignorewarnings' => 'Ignore any warnings',
@@ -1400,8 +1400,8 @@
14011401 'uploadscripted' => 'This file contains HTML or script code that may be erroneously be interpreted by a web browser.',
14021402 'uploadcorrupt' => 'The file is corrupt or has an incorrect extension. Please check the file and upload again.',
14031403 'uploadvirus' => 'The file contains a virus! Details: $1',
1404 -'sourcefilename' => 'Source filename',
1405 -'destfilename' => 'Destination filename',
 1404+'sourcefilename' => 'Source filename:',
 1405+'destfilename' => 'Destination filename:',
14061406 'watchthisupload' => 'Watch this page',
14071407 'filewasdeleted' => 'A file of this name has been previously uploaded and subsequently deleted. You should check the $1 before proceeding to upload it again.',
14081408
@@ -1418,7 +1418,7 @@
14191419 'upload-curl-error28' => 'Upload timeout',
14201420 'upload-curl-error28-text' => 'The site took too long to respond. Please check the site is up, wait a short while and try again. You may want to try at a less busy time.',
14211421
1422 -'license' => 'Licensing',
 1422+'license' => 'Licensing:',
14231423 'nolicense' => 'None selected',
14241424 'licenses' => '-', # don't translate or duplicate this message to other languages
14251425 'upload_source_url' => ' (a valid, publicly accessible URL)',
Index: trunk/phase3/languages/messages/MessagesDe.php
@@ -998,11 +998,11 @@
999999 'uploadlog' => 'Datei-Logbuch',
10001000 'uploadlogpage' => 'Datei-Logbuch',
10011001 'uploadlogpagetext' => 'Dies ist das Logbuch der hochgeladenen Dateien, siehe auch [[{{ns:special}}:Newimages]].',
1002 -'filename' => 'Dateiname',
 1002+'filename' => 'Dateiname:',
10031003 'filedesc' => 'Beschreibung',
10041004 'fileuploadsummary' => 'Beschreibung/Quelle:',
1005 -'filestatus' => 'Copyright-Status',
1006 -'filesource' => 'Quelle',
 1005+'filestatus' => 'Copyright-Status:',
 1006+'filesource' => 'Quelle:',
10071007 'uploadedfiles' => 'Hochgeladene Dateien',
10081008 'ignorewarning' => 'Warnung ignorieren und Datei speichern.',
10091009 'ignorewarnings' => 'Warnungen ignorieren',
@@ -1040,8 +1040,8 @@
10411041 'uploadscripted' => 'Diese Datei enthält HTML- oder Scriptcode, der irrtümlich von einem Webbrowser ausgeführt werden könnte.',
10421042 'uploadcorrupt' => 'Die Datei ist beschädigt oder hat einen falschen Namen. Bitte überprüfen Sie die Datei und laden Sie sie erneut hoch.',
10431043 'uploadvirus' => 'Diese Datei enthält einen Virus! Details: $1',
1044 -'sourcefilename' => 'Quelldatei',
1045 -'destfilename' => 'Zielname',
 1044+'sourcefilename' => 'Quelldatei:',
 1045+'destfilename' => 'Zielname:',
10461046 'watchthisupload' => 'Diese Seite beobachten',
10471047 'filewasdeleted' => 'Eine Datei mit diesem Namen wurde schon einmal hochgeladen und zwischenzeitlich wieder gelöscht. Bitte prüfen Sie zuerst den Eintrag im $1, bevor Sie die Datei wirklich speichern.',
10481048
@@ -1058,7 +1058,7 @@
10591059 'upload-curl-error28' => 'Zeitüberschreitung beim Hochladen',
10601060 'upload-curl-error28-text' => 'Die Seite braucht zu lange für eine Antwort. Prüfen Sie, ob die Seite online ist, warten Sie einen kurzen Moment und versuchen Sie es dann erneut. Es kann sinnvoll sein, einen erneuten Versuch zu einer anderen Zeit zu probieren..',
10611061
1062 -'license' => 'Lizenz',
 1062+'license' => 'Lizenz:',
10631063 'nolicense' => 'keine Vorauswahl',
10641064 'upload_source_url' => ' (gültige, öffentlich zugängliche URL)',
10651065 'upload_source_file' => ' (eine Datei auf Ihrem Computer)',

Follow-up revisions

RevisionCommit summaryAuthorDate
r23524Cleanup for r23523 using sedsimetrical21:57, 28 June 2007
r23547Revert r23523, r23524, stabbed by Tim :-(...raymond10:04, 29 June 2007
r23581Merged revisions 23406-23580 via svnmerge from...david04:50, 30 June 2007

Status & tagging log