| Summary: | missing i18n() in magnatuneredownloadhandler.cpp | ||
|---|---|---|---|
| Product: | [Applications] amarok | Reporter: | Sven Krohlas <sven> |
| Component: | Internet Services | Assignee: | Amarok Bugs <amarok-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
Updated patch, including a grammar fix:
Index: src/magnatunebrowser/magnatuneredownloadhandler.cpp
===================================================================
--- src/magnatunebrowser/magnatuneredownloadhandler.cpp (Revision 628783)
+++ src/magnatunebrowser/magnatuneredownloadhandler.cpp (Arbeitskopie)
@@ -26,6 +26,7 @@
#include "qdir.h"
#include "qmessagebox.h"
+#include <klocale.h>
@@ -50,8 +51,8 @@
if (previousDownloads.isEmpty()) {
//No previously purchased trak information found. No more to do here...
- QMessageBox::information( m_parent, "No purchases found!",
- "No previous purchases has been found. Nothing to redownload...\n" );
+ QMessageBox::information( m_parent, i18n( "No purchases found!" ) ,
+ i18n( "No previous purchases have been found. Nothing to redownload..." ) + "\n" );
return;
}
@@ -122,8 +123,8 @@
else
{
- QMessageBox::information( m_parent, "Could not re-download album",
- "There seems to be a problem with the selected redownload info file.\n" );
+ QMessageBox::information( m_parent, i18n( "Could not re-download album" ),
+ i18n( "There seems to be a problem with the selected redownload info file." ) + "\n" );
}
SVN commit 628879 by krohlas: Dear translators, we are really sorry: we forgot to put i18n() around those five strings. Anyway, now there is at least a chance that some of you might be able to translate them. BUG: 140957 BUG: 140963 CCMAIL: kde-i18n-doc@kde.org M +11 -1 magnatunebrowser.cpp M +5 -4 magnatuneredownloadhandler.cpp --- trunk/extragear/multimedia/amarok/src/magnatunebrowser/magnatunebrowser.cpp #628878:628879 @@ -500,7 +500,17 @@ if (!m_polished) { m_polished = true; updateList( ); - m_artistInfobox->openURL( KURL( locate( "data", "amarok/data/magnatune_start_page.html" ) ) ); + m_artistInfobox->begin( KURL( locate( "data", "amarok/data/" ) ) ); + m_artistInfobox->write( "<table align='center' border='0'><tbody align='center' valign='top'>" + "<tr align='center'><td><div align='center'>" + "<IMG src='magnatune_logo.png' width='200' height='36' align='center' border='0'>" + "</div></td></tr><tr><td><BR>" + + i18n( "Welcome to Amarok's integrated Magnatune.com store. If this is the " + "first time you run it, you must update the database by pressing the " + "'Update' button below." ) + + "</td></tr></tbody></table>" ); + m_artistInfobox->end(); + } } --- trunk/extragear/multimedia/amarok/src/magnatunebrowser/magnatuneredownloadhandler.cpp #628878:628879 @@ -26,6 +26,7 @@ #include "qdir.h" #include "qmessagebox.h" +#include <klocale.h> @@ -50,8 +51,8 @@ if (previousDownloads.isEmpty()) { //No previously purchased trak information found. No more to do here... - QMessageBox::information( m_parent, "No purchases found!", - "No previous purchases has been found. Nothing to redownload...\n" ); + QMessageBox::information( m_parent, i18n( "No purchases found!" ) , + i18n( "No previous purchases has been found. Nothing to redownload..." ) + "\n" ); return; } @@ -122,8 +123,8 @@ else { - QMessageBox::information( m_parent, "Could not re-download album", - "There seems to be a problem with the selected redownload info file.\n" ); + QMessageBox::information( m_parent, i18n( "Could not re-download album" ), + i18n( "There seems to be a problem with the selected redownload info file." ) + "\n" ); } |
Version: SVN (using KDE KDE 3.5.5) Installed from: SuSE RPMs Compiler: - - OS: Linux There are some i18n() missing in magnatuneredownloadhandler.cpp. Here is a patch, due to the string freeze I'm not allowed to commit it atm. ;-) Index: src/magnatunebrowser/magnatuneredownloadhandler.cpp =================================================================== --- src/magnatunebrowser/magnatuneredownloadhandler.cpp (Revision 628783) +++ src/magnatunebrowser/magnatuneredownloadhandler.cpp (Arbeitskopie) @@ -26,6 +26,7 @@ #include "qdir.h" #include "qmessagebox.h" +#include <klocale.h> @@ -50,8 +51,8 @@ if (previousDownloads.isEmpty()) { //No previously purchased trak information found. No more to do here... - QMessageBox::information( m_parent, "No purchases found!", - "No previous purchases has been found. Nothing to redownload...\n" ); + QMessageBox::information( m_parent, i18n( "No purchases found!" ) , + i18n( "No previous purchases has been found. Nothing to redownload..." ) + "\n" ); return; } @@ -122,8 +123,8 @@ else { - QMessageBox::information( m_parent, "Could not re-download album", - "There seems to be a problem with the selected redownload info file.\n" ); + QMessageBox::information( m_parent, i18n( "Could not re-download album" ), + i18n( "There seems to be a problem with the selected redownload info file." ) + "\n" ); }