author | zecke <zecke> | 2004-10-23 21:51:16 (UTC) |
---|---|---|
committer | zecke <zecke> | 2004-10-23 21:51:16 (UTC) |
commit | 9c5964f8d2be467aa53d3d8255b499890556e320 (patch) (side-by-side diff) | |
tree | 91507b2507cbfda0896541dde082e065444126df /noncore/tools/pimconverter | |
parent | cbcdbc9d68e493a4229b24a57075d617ae99ca70 (diff) | |
download | opie-9c5964f8d2be467aa53d3d8255b499890556e320.zip opie-9c5964f8d2be467aa53d3d8255b499890556e320.tar.gz opie-9c5964f8d2be467aa53d3d8255b499890556e320.tar.bz2 |
-Move code for converting sqlite2 to sqlite3 to a method on its own
-Use Global::applicationFileName to get the filenames for the data
-Use tr around messages
-Do not use custom labels for "Ok" but use Qts QMessageBox ones to get
its translation for free
TODO: Test on ARM
-rw-r--r-- | noncore/tools/pimconverter/converter.cpp | 139 | ||||
-rwxr-xr-x | noncore/tools/pimconverter/converter.h | 3 |
2 files changed, 83 insertions, 59 deletions
diff --git a/noncore/tools/pimconverter/converter.cpp b/noncore/tools/pimconverter/converter.cpp index 2cd6969..fa3e714 100644 --- a/noncore/tools/pimconverter/converter.cpp +++ b/noncore/tools/pimconverter/converter.cpp @@ -3,12 +3,13 @@ #include <stdlib.h> // For "system()" command /* OPIE */ #include <opie2/oapplicationfactory.h> #include <opie2/odebug.h> #include <opie2/opimglobal.h> +#include <qpe/global.h> // Include SQL related header files #define __USE_SQL #include <opie2/opimaccessfactory.h> /* QT */ #include <qdatetime.h> @@ -58,16 +59,15 @@ void Converter::start_conversion(){ OPimBase* destDB; odebug << "SourceFormat: " << m_selectedSourceFormat << oendl; odebug << "DestFormat: " << m_selectedDestFormat << oendl; if ( m_selectedSourceFormat == m_selectedDestFormat ){ - QMessageBox::warning( this, "PimConverter", - tr( "It is not a good idea to use\n" ) - +tr( "the same source and destformat !" ), - tr( "Ok" ) ); + QMessageBox::warning( this, tr("PimConverter"), + tr( "<qt>It is not a good idea to use" + "the same source and destformat !</qt>")); return; } switch( m_selectedSourceFormat ){ case XML: odebug << "XMLSourceDB = " << m_selectedDatabase << "" << oendl; @@ -224,90 +224,111 @@ void Converter::closeEvent( QCloseEvent *e ) e->ignore(); return; } e->accept(); } +bool Converter::sqliteCopyAndConvert(const QString& src, const QString &dest, QString &cmd) +{ + if (!QFile::exists( src ) ) { + cmd = tr("No SQLite2 database exists"); + return false; + } + + if( QFile::exists( dest ) ) { + cmd = tr("A copy of the SQLite2 exists"); + return false; + } + + /* + * Copy it over + */ + cmd = "cp " + Global::shellQuote(src) + " " + Global::shellQuote(dest); + if( ::system( cmd ) != 0 ) + return false; + + /* + * Delete it to make place for the new file + */ + cmd = "rm " + Global::shellQuote(src); + if( ::system( cmd ) != 0 ) + return false; + + /* + * Convert it + */ + cmd = "sqlite " + Global::shellQuote(dest) + " .dump | sqlite3 " + Global::shellQuote(src); + return ::system( cmd ) == 0; +} + void Converter::start_upgrade() { odebug << "Start upgrading" << oendl; - switch( QMessageBox::warning( this, "Pim-Converter", - "Are you really sure that you\n" - "want to convert your database from\n" - "sqlite V2 to sqlite V3?", + switch( QMessageBox::warning( this, tr("Pim-Converter"), + tr("<qt>Are you really sure that you " + "want to convert your database from " + "sqlite V2 to sqlite V3?</qt>"), QMessageBox::Ok | QMessageBox::Default, QMessageBox::Abort | QMessageBox::Escape )) { case QMessageBox::Abort: // Abort clicked or Escape pressed // abort return; break; } odebug << "Checking whether sqlite is installed" << oendl; if ( system( "which sqlite" ) != 0 ){ - QMessageBox::critical( this, "Pim-Converter", - QString("An internal error occurred:\n") + - "sqlite was not accessible!\n"+ - "Please correct the PATH or install \n" + - "this packages!" ); + QMessageBox::critical( this, tr("Pim-Converter"), + tr("<qt>An internal error occurred: " + "sqlite was not accessible! " + "Please correct the PATH or install " + "this packages!</qt>") ); return; } if ( system( "which sqlite3" ) != 0 ){ - QMessageBox::critical( this, "Pim-Converter", - QString("An internal error occurred:\n") + - "sqlite3 was not accessible!\n"+ - "Please correct the PATH or install \n" + - "this packages!" ); + QMessageBox::critical( this, tr("Pim-Converter"), + tr("<qt>An internal error occurred: " + "sqlite3 was not accessible! " + "Please correct the PATH or install " + "this packages!</qt>") ); return; } - if ( QFile::exists( "~/Applications/addressbook/addressbook.db" ) - && !QFile::exists( "~/Applications/addressbook/addressbook.db_v2" ) ){ - odebug << "Executing conversion commands" << oendl; - QString addr_convert_string = "cd ~/Applications/addressbook/;cp addressbook.db addressbook.db_v2;sqlite addressbook.db_v2 .dump | sqlite3 addressbook.db"; - odebug << "1. Addressbook Command:" << addr_convert_string << oendl; - if ( system( addr_convert_string ) != 0 ){ - QMessageBox::critical( this, "Pim-Converter", - QString("An internal error occurred:\n") + - "Converting the addressbook command was impossible!\n"+ - "Executed the following command:\n" + - addr_convert_string ); + + QString src, dest, cmd; + src = Global::applicationFileName("addressbook", "addressbook.db" ); + dest = Global::applicationFileName("addressbook", "addressbook.db_v2" ); + + if(!sqliteCopyAndConvert(src, dest, cmd)) { + QMessageBox::critical( this, tr("Pim-Converter"), + tr("<qt>An internal error occurred: " + "Converting the addressbook command was impossible! " + "Executed the following command: %1</qt>").arg(cmd) ); return; } - } - if ( QFile::exists( "~/Applications/datebook/datebook.db" ) - && !QFile::exists( "~/Applications/datebook/datebook.db_v2" ) ){ - QString dateb_convert_string = "cd ~/Applications/datebook/;cp datebook.db datebook.db_v2;sqlite datebook.db_v2 .dump | sqlite3 datebook.db"; - odebug << "2. Datebook Command" << dateb_convert_string << oendl; - if ( system( dateb_convert_string ) != 0 ){ - QMessageBox::critical( this, "Pim-Converter", - QString("An internal error occurred:\n") + - "Converting the datebook command was impossible!\n"+ - "Executed the following command:\n" + - dateb_convert_string ); + + src = Global::applicationFileName("datebook", "datebook.db" ); + dest = Global::applicationFileName("datebook", "datebook.db_v2" ); + if(!sqliteCopyAndConvert(src, dest, cmd)) { + QMessageBox::critical( this, tr("Pim-Converter"), + tr("<qt>An internal error occurred: " + "Converting the datebook command was impossible! " + "Executed the following command: '%1' </qt>").arg(cmd) + ); return; } - } - if ( QFile::exists( "~/Applications/todolist/todolist.db" ) - && !QFile::exists( "~/Applications/todolist/todolist.db_v2" ) ){ - QString todo_convert_string = "cd ~/Applications/todolist/;cp todolist.db todolist.db_v2;sqlite todolist.db_v2 .dump | sqlite3 todolist.db"; - odebug << "3. Todolist Command:" << todo_convert_string << oendl; - if ( system( todo_convert_string ) != 0 ){ - QMessageBox::critical( this, "Pim-Converter", - QString("An internal error occurred:\n") + - "Converting the todolist command was impossible!\n"+ - "Executed the following command:\n" + - todo_convert_string ); + src = Global::applicationFileName("todolist", "todolist.db" ); + dest = Global::applicationFileName("todolist", "todolist.db_v2" ); + if(!sqliteCopyAndConvert(src, dest, cmd)) { + QMessageBox::critical( this, tr("Pim-Converter"), + tr("<qt>An internal error occurred: " + "Converting the todolist command was impossible! " + "Executed the following command: '%1' </qt>").arg(cmd) ); return; } - } - - QMessageBox::information( this, "Pim-Converter", - "Conversion is finished!", - "&OK", NULL, NULL, - 0, // Enter == button 0 - 0 ); - + QMessageBox::information( this, tr("Pim-Converter"), + tr("Conversion is finished!"), + QMessageBox::Ok ); } diff --git a/noncore/tools/pimconverter/converter.h b/noncore/tools/pimconverter/converter.h index 9d75d79..e9035e7 100755 --- a/noncore/tools/pimconverter/converter.h +++ b/noncore/tools/pimconverter/converter.h @@ -37,10 +37,13 @@ private: int m_selectedDatabase; int m_selectedSourceFormat; int m_selectedDestFormat; bool m_criticalState; + +private: + static bool sqliteCopyAndConvert( const QString &src, const QString &destination, QString &lastCommand ); }; #endif |