kde sync fixes

2 files changed, 16 insertions, 3 deletions

diff --git a/kde2file/abdump/main.cpp b/kde2file/abdump/main.cpp
index 9ad78e5..b359cfe 100644
--- a/kde2file/abdump/main.cpp
+++ b/kde2file/abdump/main.cpp
@@ -163,31 +163,34 @@ int main( int argc, char *argv[] )
                     if ( (*itL).uid() == (*itA).uid() ) {
                         found = true;
                         break;
                     }
                 }
                 if ( !found ) {
                     it2 = itA;
                     ++itA;
                     standardAddressBook->removeAddressee( it2 );
                     ++deleted;
                 } else  {
                     ++itA;
                 }
             }
 
             //standardAddressBook->saveAll();
             standardAddressBook->setAutomaticSave( true );
             qDebug("************************************* ");
             qDebug("*************kdeABdump*************** ");
             qDebug("************************************* ");
             qDebug("Addressbook entries\nchanged %d\ndeleted %d\nadded %d\nfrom file %s", changedC,deleted, added, fileName.latin1());
         } else
             qDebug("error open file ");
     }
+    standardAddressBook->close();
+    // line not needed by KDE 3.4:
+    //    delete standardAddressBook;
+
 
-    delete standardAddressBook;
     //KABC::StdAddressBook::close();
     //StdAddressBook::mSelf = 0;
     qDebug("ente ");
     return 0;
 }
diff --git a/libkdepim/ksyncmanager.cpp b/libkdepim/ksyncmanager.cpp
index d59f4a4..7319285 100644
--- a/libkdepim/ksyncmanager.cpp
+++ b/libkdepim/ksyncmanager.cpp
@@ -901,69 +901,79 @@ void KSyncManager::confSync()
     mLocalMachineName = sp->getLocalMachineName ();
     int ii;
     for ( ii = 0; ii < oldSyncProfileNames.count(); ++ii ) {
         if ( ! mSyncProfileNames.contains( oldSyncProfileNames[ii] ) )
             mImplementation->removeSyncInfo( oldSyncProfileNames[ii]  );
     }
     QTimer::singleShot ( 1, this, SLOT ( fillSyncMenu() ) ); 
 }
 void KSyncManager::syncKDE()
 {
     mSyncWithDesktop = true;
     emit save();
     switch(mTargetApp)
         {
         case (KAPI):
             {
 #ifdef DESKTOP_VERSION
                 QString command = qApp->applicationDirPath () + "/kdeabdump"; 
 #else
                 QString command = "kdeabdump"; 
 #endif
                 if ( ! QFile::exists ( command ) )
                     command = "kdeabdump";  
                 QString fileName = QDir::homeDirPath ()+"/.kdeaddressbookdump.vcf";
-                system (  command.latin1());
+                int result = system (  command.latin1());
+                qDebug("AB dump command call result: %d ", result);
+                if ( result != 0 ) {
+                    KMessageBox::error( 0, i18n("Error accessing KDE addressbook data.\nMake sure the file\n/opt/kdepimpi/kdeabdump\nexists.The standard version of this file\nis for syncing with KDE 3.4.x.\nIf you are running KDE 3.3.x please\ndownload the KDE 3.3.x version of this file\nat http://sourceforge.net/projects/kdepimpi/\nsection: general files for KDE/Pim."));
+                    return;
+                }
                     if ( syncWithFile( fileName,true ) ) {
                         if ( mWriteBackFile ) {
                             command += " --read";
                             system (  command.latin1());
                         }
                     }
                 
             }
             break;
         case (KOPI):
             {
 #ifdef DESKTOP_VERSION
                 QString command = qApp->applicationDirPath () + "/kdecaldump"; 
 #else
                 QString command = "kdecaldump"; 
 #endif
                 if ( ! QFile::exists ( command ) )
                     command = "kdecaldump";  
                 QString fileName = QDir::homeDirPath ()+"/.kdecalendardump.ics";
-                system (  command.latin1());
+                int result = system (  command.latin1());
+                qDebug("Cal dump command call result result: %d ", result);
+                if ( result != 0 ) {
+                    KMessageBox::error( 0, i18n("Error accessing KDE calendar data.\nMake sure the file\n/opt/kdepimpi/kdecaldump\nexists.The standard version of this file\nis for syncing with KDE 3.4.x.\nIf you are running KDE 3.3.x please\ndownload the KDE 3.3.x version of this file\nat http://sourceforge.net/projects/kdepimpi/\nsection: general files for KDE/Pim."));
+                    return;
+                }
                     if ( syncWithFile( fileName,true ) ) {
                         if ( mWriteBackFile ) {
                             command += " --read";
                             system (  command.latin1());
                         }
                     }
                 
             }
             break;
         case (PWMPI):
 
             break;
         default:
             qDebug("KSM::slotSyncMenu: invalid apptype selected");
             break;
 	      
         }
 }
 
 void KSyncManager::syncSharp()
 {
 
     if ( ! syncExternalApplication("sharp") )
         qDebug("KSM::ERROR sync sharp  ");