fixed dtm resource

1 files changed, 41 insertions, 25 deletions

diff --git a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
index 48b7d91..7069f5d 100644
--- a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
+++ b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
@@ -80,8 +80,18 @@ ResourceSharpDTM::ResourceSharpDTM( const QString &fileName  )
 }
 
 void ResourceSharpDTM::init( const QString &fileName )
 {
+    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);
+                return;
+            }
+    } 
   setFileName( fileName );
 }
 
 ResourceSharpDTM::~ResourceSharpDTM()
@@ -109,32 +119,10 @@ Ticket *ResourceSharpDTM::requestSaveTicket()
 
 
 bool ResourceSharpDTM::doOpen()
 {
-    qDebug("%xResourceSharpDTM::doOpen: %s", this,fileName().latin1());
-
-    if ( ! mAccess  ) {
-        // the last parameter in the SlZDataBase constructor means "readonly"
-        mAccess = new SlZDataBase(fileName(),
-                                  SlZDataBase::addressbookItems(),
-                                  NULL, false);
-    }
-    if ( !mAccess ) {
-        qDebug("Unable to load file() %s", fileName().latin1());
+    if (!mConverter)
         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;
 }
 
 void ResourceSharpDTM::doClose()
@@ -143,8 +131,19 @@ void ResourceSharpDTM::doClose()
 }
 
 bool ResourceSharpDTM::load()
 {
+    if (!mConverter)
+        return false;
+    QString fileN = SlZDataBase::addressbookFileName();
+    if ( ! mAccess  ) {
+        mAccess = new SlZDataBase(fileN,
+                                  SlZDataBase::addressbookItems(),
+                                  0, false);
+    }
+    if(mAccess == 0)
+        return false;
+
     qDebug("%xResourceSharpDTM::load: %s",this, fileName().latin1());
     bool res = false;
     CardId id;
     for (bool res=mAccess->first(); res == true; res=mAccess->next())
@@ -157,13 +156,27 @@ bool ResourceSharpDTM::load()
                     addressee.setResource( this );
                     addressBook()->insertAddressee( addressee );
                 }
         }
+    if(mAccess != 0)
+        delete mAccess;
+    mAccess = 0;
+
     return true;
 }
 
 bool ResourceSharpDTM::save( Ticket *ticket )
 {
+    if (!mConverter)
+        return false;
+    QString fileN = SlZDataBase::addressbookFileName();
+    if ( ! mAccess  ) {
+        mAccess = new SlZDataBase(fileN,
+                                  SlZDataBase::addressbookItems(),
+                                  0, false);
+    }
+    if(mAccess == 0)
+        return false;
   qDebug("ResourceSharpDTM::save: %s", fileName().latin1());
   KABC::AddressBook::Iterator it;
   bool res;
   KABC::Addressee::List changedAddressees;
@@ -248,14 +261,17 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                   }
           }
       }
   delete ticket;
-
+    if(mAccess != 0)
+        delete mAccess;
+    mAccess = 0;
   return true;
 }
 
 bool ResourceSharpDTM::lock( const QString &lockfileName )
 {
+    return true;
 }
 
 void ResourceSharpDTM::unlock( const QString &fileName )
 {
@@ -277,9 +293,9 @@ void ResourceSharpDTM::removeAddressee( const Addressee &addr )
 }
 
 void ResourceSharpDTM::cleanUp()
 {
-  unlock( fileName() );
+ 
 }