5 files changed, 61 insertions, 177 deletions

diff --git a/kabc/addressbook.cpp b/kabc/addressbook.cpp
index 4c4ae09..8487ff3 100644
--- a/kabc/addressbook.cpp
+++ b/kabc/addressbook.cpp
@@ -1162,2 +1162,13 @@ bool AddressBook::addResource( Resource *resource )
 }
+void AddressBook::removeResources()
+{
+    //remove all possible resources. This should cleanup the configfile.
+    QPtrList<KABC::Resource> mResources = resources();
+    
+    QPtrListIterator<KABC::Resource> it(mResources);
+    for ( ; it.current(); ++it ) {
+        KABC::Resource *res = it.current();
+        removeResource(res);
+    }
+}
 
diff --git a/kabc/addressbook.h b/kabc/addressbook.h
index 75f8b51..2351add 100644
--- a/kabc/addressbook.h
+++ b/kabc/addressbook.h
@@ -276,2 +276,3 @@ class AddressBook : public QObject
     */
+    void removeResources();
     bool removeResource( Resource * );
diff --git a/kabc/addressee.cpp b/kabc/addressee.cpp
index 3d4992c..028d3bb 100644
--- a/kabc/addressee.cpp
+++ b/kabc/addressee.cpp
@@ -274,3 +274,3 @@ void Addressee::computeCsum(const QString &dev)
     uint cs = getCsum4List(l);
-#if 1
+#if 0
     for ( iii = 0; iii < l.count(); ++iii)
diff --git a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
index ba17c50..48b7d91 100644
--- a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
+++ b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
@@ -69,5 +69,5 @@ ResourceSharpDTM::ResourceSharpDTM( const KConfig *config  )
   // we can not choose the filename. Therefore use the default to display
-
-  QString fileName = SlZDataBase::addressbookFileName();
-  init( fileName );
+    mAccess = 0;
+    QString fileName = SlZDataBase::addressbookFileName();
+    init( fileName );
 }
@@ -77,3 +77,4 @@ ResourceSharpDTM::ResourceSharpDTM( const QString &fileName  )
 {
-  init( fileName );
+    mAccess = 0;
+    init( fileName );
 }
@@ -82,7 +83,2 @@ void ResourceSharpDTM::init( const QString &fileName )
 {
-
-  connect( &mDirWatch, SIGNAL( dirty(const QString&) ), SLOT( fileChanged() ) );
-  connect( &mDirWatch, SIGNAL( created(const QString&) ), SLOT( fileChanged() ) );
-  connect( &mDirWatch, SIGNAL( deleted(const QString&) ), SLOT( fileChanged() ) );
-
   setFileName( fileName );
@@ -106,3 +102,2 @@ Ticket *ResourceSharpDTM::requestSaveTicket()
 {
- 
 
@@ -111,9 +106,2 @@ Ticket *ResourceSharpDTM::requestSaveTicket()
   if ( !addressBook() ) return 0;
-
-#ifdef ALLOW_LOCKING
-  if ( !lock( fileName() ) ) {
-      qDebug("ResourceSharpDTM::requestSaveTicket(): Unable to lock file ");
-    return 0;
-  }
-#endif
   return createTicket( this );
@@ -124,30 +112,27 @@ bool ResourceSharpDTM::doOpen()
 {
-  qDebug("ResourceSharpDTM::doOpen: %s", fileName().latin1());
-
-  // the last parameter in the SlZDataBase constructor means "readonly"
-  mAccess = new SlZDataBase(fileName(),
-                        SlZDataBase::addressbookItems(),
-                        NULL, false);
+    qDebug("%xResourceSharpDTM::doOpen: %s", this,fileName().latin1());
 
-  if ( !mAccess ) {
-    qDebug("Unable to load file() %s", fileName().latin1());
-    return false;
-  }
-
-  if (mConverter == 0)
-  {
-    mConverter = new SharpDTMConverter();
-    bool res = mConverter->init(); 
-    if ( !res )
-    {
-          QString msg("Unable to initialize sharp converter. Most likely a problem with the category file");
-
-      qDebug(msg);
-      delete mAccess;
-      mAccess = 0;
-      return false;
+    if ( ! mAccess  ) {
+        // the last parameter in the SlZDataBase constructor means "readonly"
+        mAccess = new SlZDataBase(fileName(),
+                                  SlZDataBase::addressbookItems(),
+                                  NULL, false);
     }
-  }
-
- return true;
+    if ( !mAccess ) {
+        qDebug("Unable to load file() %s", fileName().latin1());
+        return false;
+    }
+    if (mConverter == 0) {
+        mConverter = new SharpDTMConverter();
+        bool res = mConverter->init(); 
+        if ( !res )
+            {
+                QString msg("Unable to initialize sharp converter. Most likely a problem with the category file");
+                qDebug(msg);
+                delete mAccess;
+                mAccess = 0;
+                return false;
+            }
+    }
+    return true;
 }
@@ -156,12 +141,2 @@ void ResourceSharpDTM::doClose()
 {
-  qDebug("ResourceSharpDTM::doClose: %s", fileName().latin1());
-
-  if(mAccess)
-  {
-    delete mAccess;
-    mAccess = 0;
-  }
-  // it seems so, that deletion of access deletes backend as well
-  //delete backend;
-
   return;
@@ -171,24 +146,17 @@ bool ResourceSharpDTM::load()
 {
-  qDebug("ResourceSharpDTM::load: %s", fileName().latin1());
-
-  bool res = false;
-
-  CardId id;
-
-  for (bool res=mAccess->first(); res == true; res=mAccess->next())
-  {
-    id = mAccess->cardId();
-
-    KABC::Addressee addressee;
-
-    res = mConverter->sharpToAddressee( id, mAccess, addressee );
-
-    if ( !addressee.isEmpty() && res )
-    {
-      addressee.setResource( this );
-      addressBook()->insertAddressee( addressee );
-    }
-  }
-
-  return true;
+    qDebug("%xResourceSharpDTM::load: %s",this, fileName().latin1());
+    bool res = false;
+    CardId id;
+    for (bool res=mAccess->first(); res == true; res=mAccess->next())
+        {
+            id = mAccess->cardId();
+            KABC::Addressee addressee;
+            res = mConverter->sharpToAddressee( id, mAccess, addressee );
+            if ( !addressee.isEmpty() && res )
+                {
+                    addressee.setResource( this );
+                    addressBook()->insertAddressee( addressee );
+                }
+        }
+    return true;
 }
@@ -198,5 +166,2 @@ bool ResourceSharpDTM::save( Ticket *ticket )
   qDebug("ResourceSharpDTM::save: %s", fileName().latin1());
-
-  mDirWatch.stopScan();
-
   KABC::AddressBook::Iterator it;
@@ -207,3 +172,2 @@ bool ResourceSharpDTM::save( Ticket *ticket )
   CardId id ;
-
   for ( it = addressBook()->begin(); it != addressBook()->end(); ++it ) {
@@ -225,6 +189,3 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                           {
-                              res = mAccess->finishEditCard(&id);
-                              //(*it).setTempSyncStat( SYNC_TEMPSTATE_NEW_ID );
-                              //(*it).setExternalUID( QString::number( id ) );
-                              //(*it).setOriginalExternalUID( QString::number( id ) ); 
+                              res = mAccess->finishEditCard(&id);; 
                               map.insert(id,(*it).uid());                           
@@ -256,7 +217,5 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                               res = mAccess->finishEditCard(&id);
-                              //(*it).setTempSyncStat( SYNC_TEMPSTATE_NEW_CSUM );
                               map.insert(id,(*it).uid());
                               if (res == false)
-                                  qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());
-                              
+                                  qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());         
                           }
@@ -291,8 +250,3 @@ bool ResourceSharpDTM::save( Ticket *ticket )
       }
-
-  //US  mAccess->save();
-
-  mDirWatch.startScan();
   delete ticket;
-  unlock( fileName() );
 
@@ -303,51 +257,2 @@ bool ResourceSharpDTM::lock( const QString &lockfileName )
 {
-#ifdef ALLOW_LOCKING
-  qDebug("ResourceSharpDTM::lock: %s", fileName().latin1());
-
-  kdDebug(5700) << "ResourceSharpDTM::lock()" << endl;
-
-  QString fn = lockfileName;
-
-  KURL url(fn);
-  QString lockName = locateLocal( "data", "kabc/lock/" + url.fileName() + ".lock" );
-
-  kdDebug(5700) << "-- lock name: " << lockName << endl;
-
-  if (QFile::exists( lockName ))
-  {
-    qDebug("ResourceSharpDTM::lock: lockfile %s already exists. (Delete it before continuing if nobody else is accessing the resourcefile %s)", lockName.latin1(), fileName());
-    return false;
-  }
-
-
-  QString lockUniqueName;
-  lockUniqueName = fn + KApplication::randomString( 8 );
-
-  url = lockUniqueName;
-//US  mLockUniqueName = locateLocal( "data", "kabc/lock/" + lockUniqueName );
-  mLockUniqueName = locateLocal( "data", "kabc/lock/" + url.fileName() );
-  kdDebug(5700) << "-- lock unique name: " << mLockUniqueName << endl;
-
-  // Create unique file
-  QFile file( mLockUniqueName );
-  file.open( IO_WriteOnly );
-  file.close();
-
-  // Create lock file
-  int result = 0;
-#ifndef _WIN32_
-  result = ::link( QFile::encodeName( mLockUniqueName ),
-                       QFile::encodeName( lockName ) );
-#endif
-  if ( result == 0 ) {
-    addressBook()->emitAddressBookLocked();
-    return true;
-  }
-
-  // TODO: check stat
-
-  return false;
-#else
-  return true;
-#endif 
 }
@@ -356,13 +261,3 @@ void ResourceSharpDTM::unlock( const QString &fileName )
 {
-#ifdef ALLOW_LOCKING
-  qDebug("ResourceSharpDTM::unlock() %s", fileName.latin1());
 
-  QString fn = fileName;
-  KURL url(fn);
-  QString lockName = locateLocal( "data", "kabc/lock/" + url.fileName() + ".lock" );
-
-  QFile::remove( lockName );
-  QFile::remove( mLockUniqueName );
-  addressBook()->emitAddressBookUnlocked();
-#endif 
 }
@@ -371,10 +266,3 @@ void ResourceSharpDTM::setFileName( const QString &newFileName )
 {
-  mDirWatch.stopScan();
-  mDirWatch.removeFile( fileName() );
-
-  Resource::setFileName( newFileName );
-
-  mDirWatch.addFile( fileName() );
-  mDirWatch.startScan();
-
+    Resource::setFileName( newFileName );
 }
@@ -383,12 +271,3 @@ void ResourceSharpDTM::fileChanged()
 {
-  // There is a small theoretical chance that KDirWatch calls us before
-  // we are fully constructed
-  if (!addressBook())
-    return;
-
-  QString text( i18n( "Sharp DTM resource '%1'<br> has been changed by third party.<br>Do you want to reload?").arg( fileName() ) );
-  if ( readOnly() || KMessageBox::questionYesNo( 0, text ) == KMessageBox::Yes ) {
-    load();
-    addressBook()->emitAddressBookChanged();
-  }
+ 
 }
diff --git a/kabc/tmpaddressbook.cpp b/kabc/tmpaddressbook.cpp
index 6e24302..cfa57e3 100644
--- a/kabc/tmpaddressbook.cpp
+++ b/kabc/tmpaddressbook.cpp
@@ -46,10 +46,3 @@ TmpAddressBook::~TmpAddressBook()
 {
-  //remove all possible resources. This should cleanup the configfile.
-  QPtrList<KABC::Resource> mResources = resources();
-
-  QPtrListIterator<KABC::Resource> it(mResources);
-  for ( ; it.current(); ++it ) {
-    KABC::Resource *res = it.current();
-    removeResource(res);
-  }
+    removeResources();
 }