DTM sync fixes

5 files changed, 117 insertions, 96 deletions

diff --git a/kabc/addressbook.cpp b/kabc/addressbook.cpp
index adb451f..5fb49eb 100644
--- a/kabc/addressbook.cpp
+++ b/kabc/addressbook.cpp
@@ -238,7 +238,5 @@ void AddressBook::init(const QString &config, const QString &family )
   d = new AddressBookData;
   QString fami = family;
-  qDebug("new ab ");
   if (config != 0) {
-      qDebug("config != 0 ");
       if ( family == "syncContact" ) {
           qDebug("creating sync config ");
@@ -607,5 +605,5 @@ Addressee AddressBook::findByUid( const QString &uid )
 void AddressBook::preExternSync( AddressBook* aBook, const QString& csd )
 {
-    qDebug("AddressBook::preExternSync ");
+    //qDebug("AddressBook::preExternSync ");
     AddressBook::Iterator it;  
     for ( it = begin(); it != end(); ++it ) {
@@ -617,8 +615,8 @@ void AddressBook::preExternSync( AddressBook* aBook, const QString& csd )
 void AddressBook::postExternSync( AddressBook* aBook , const QString& csd)
 {
-    qDebug("AddressBook::postExternSync ");
+    //qDebug("AddressBook::postExternSync ");
     AddressBook::Iterator it;  
     for ( it = begin(); it != end(); ++it ) {
-        qDebug("check uid %s ", (*it).uid().latin1() );
+        // qDebug("check uid %s ", (*it).uid().latin1() );
         if ( (*it).tempSyncStat() == SYNC_TEMPSTATE_NEW_ID ||
              (*it).tempSyncStat() == SYNC_TEMPSTATE_NEW_CSUM ) {
diff --git a/kabc/addressee.cpp b/kabc/addressee.cpp
index e241281..31ce1d6 100644
--- a/kabc/addressee.cpp
+++ b/kabc/addressee.cpp
@@ -44,4 +44,6 @@ using namespace KABC;
 
 static bool matchBinaryPattern( int value, int pattern );
+static bool matchBinaryPatternA( int value, int pattern );
+static bool matchBinaryPatternP( int value, int pattern );
 
 struct Addressee::AddresseeData : public KShared
@@ -382,5 +384,5 @@ void Addressee::setOriginalExternalUID( const QString &id )
   detach();
   mData->empty = false;
-  qDebug("*******Set orig uid %s ", id.latin1());
+  //qDebug("*******Set orig uid %s ", id.latin1());
   mData->originalExternalUID = id;
 }
@@ -395,5 +397,5 @@ void Addressee::setUid( const QString &id )
   if ( id == mData->uid ) return;
   detach();
-  qDebug("****setuid %s ", id.latin1());
+  //qDebug("****setuid %s ", id.latin1());
   mData->empty = false;
   mData->uid = id;
@@ -1291,5 +1293,5 @@ PhoneNumber Addressee::phoneNumber( int type ) const
   PhoneNumber::List::ConstIterator it;
   for( it = mData->phoneNumbers.begin(); it != mData->phoneNumbers.end(); ++it ) {
-    if ( matchBinaryPattern( (*it).type(), type ) ) {
+    if ( matchBinaryPatternP( (*it).type(), type ) ) {
       if ( (*it).type() & PhoneNumber::Pref )
         return (*it);
@@ -1528,5 +1530,5 @@ Address Addressee::address( int type ) const
   Address::List::ConstIterator it;
   for( it = mData->addresses.begin(); it != mData->addresses.end(); ++it ) {
-    if ( matchBinaryPattern( (*it).type(), type ) ) {
+    if ( matchBinaryPatternA( (*it).type(), type ) ) {
       if ( (*it).type() & Address::Pref )
         return (*it);
@@ -1836,5 +1838,4 @@ QDataStream &KABC::operator>>( QDataStream &s, Addressee &a )
   return s;
 }
-
 bool matchBinaryPattern( int value, int pattern )
 {
@@ -1850,2 +1851,19 @@ bool matchBinaryPattern( int value, int pattern )
     return ( pattern == ( pattern & value ) );
 }
+
+bool matchBinaryPatternP( int value, int pattern )
+{
+  
+  if ( pattern == 0 )
+    return ( value == 0 );
+  else
+      return ( (pattern |PhoneNumber::Pref ) == ( value |PhoneNumber::Pref ) );
+}
+bool matchBinaryPatternA( int value, int pattern )
+{
+ 
+  if ( pattern == 0 )
+    return ( value == 0 );
+  else
+      return ( (pattern | Address::Pref) == ( value | Address::Pref ) );
+}
diff --git a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
index bba5f9a..48d3985 100644
--- a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
+++ b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
@@ -130,5 +130,5 @@ bool ResourceSharpDTM::doOpen()
   mAccess = new SlZDataBase(fileName(),
                         SlZDataBase::addressbookItems(),
-                        NULL, true);
+                        NULL, false);
 
   if ( !mAccess ) {
@@ -140,5 +140,5 @@ bool ResourceSharpDTM::doOpen()
   {
     mConverter = new SharpDTMConverter();
-    bool res = mConverter->init();
+    bool res = mConverter->init(); 
     if ( !res )
     {
@@ -225,17 +225,16 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                           {
                               KABC::Addressee addressee;
-                              qDebug("1NEW id is %d", id);
                               res = mConverter->sharpToAddressee( id, mAccess, addressee );
                               addressee.setUid((*it).uid() );
-                              addressee.setTempSyncStat( SYNC_TEMPSTATE_NEW_CSUM );
-                              changedAddressees.append( addressee );
+                              addressee.setTempSyncStat( SYNC_TEMPSTATE_NEW_ID );
                               res = mAccess->finishEditCard(&id);
-                              qDebug("2NEW id is %d", id);
+                              //qDebug("2NEW id is %d", id);
+                              addressee.setExternalUID( QString::number( id ) );
+                              addressee.setOriginalExternalUID( QString::number( id ) ); 
+                              changedAddressees.append( addressee );
                               
                               if (res == false)
                                   qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());
-                              else {
-                                  qDebug("added card success ");
-                              }
+                             
                           }
                       else
@@ -256,4 +255,5 @@ bool ResourceSharpDTM::save( Ticket *ticket )
           } else if ( (*it).tempSyncStat() != SYNC_TEMPSTATE_NEW_EXTERNAL ) {
               //changed
+              qDebug("changed ****");
               res = mAccess->startEditCard(id);
               if (res == true)
@@ -270,7 +270,5 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                               if (res == false)
                                   qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());
-                              else {
-                                  qDebug("change card succes ");
-                              }
+                              
                           }
                       else
@@ -281,40 +279,5 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                   }
           }
-
-
-
-
-
-
-
-
-      }
-
-#if 0 
-      // old code
-            CardId id = 0;// 0 means new card
-    KABC::Addressee addressee = (*it);
-
-    res = mAccess->startEditCard(id);
-    if (res == true)
-    {
-      res = mConverter->addresseeToSharp( *it, mAccess, id  );
-      if (res == true)
-      {
-        res = mAccess->finishEditCard(&id);
-        if (res == false)
-          qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());
       }
-      else
-      {
-        qDebug("Unable to convert Addressee: %s", addressee.formattedName().latin1());
-        mAccess->cancelEditCard();
-      }
-    }
-    else
-    {
-      qDebug("Unable to add new card. Addressee: %s", addressee.formattedName().latin1());
-    }
-#endif
 
   }
diff --git a/kabc/plugins/sharpdtm/sharpdtmconverter.cpp b/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
index b065bf3..f3bf2c5 100644
--- a/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
+++ b/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
@@ -27,4 +27,5 @@ $Id$
 
 #include "kglobal.h"
+#include <qregexp.h>
 
 
@@ -32,4 +33,5 @@ $Id$
 
 #include <sl/slcategories.h>
+#include <libkdepim/ksyncprofile.h>
 //US #include <qpe/categoryselect.h>
 
@@ -74,9 +76,10 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
     addr.setExternalUID( QString::number( contact ) );
     addr.setOriginalExternalUID( QString::number( contact ) ); 
+    addr.setTempSyncStat(  SYNC_TEMPSTATE_NEW_EXTERNAL );
 
 
     // name
-    qDebug("SharpDTMConverter::sharpToAddressee check if the fileAs transformation works!!");
-    addr.setFormattedName(db->readField(ZdbAdrs::FileAs));
+    //qDebug("SharpDTMConverter::sharpToAddressee check if the fileAs transformation works!!");
+    addr.setFormattedName(db->readField(ZdbAdrs::FileAs)); // needs fix
 
     addr.setFamilyName( db->readField(ZdbAdrs::LastName) );
@@ -87,15 +90,22 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
 
 
-    // email
     QString emailstr = db->readField(ZdbAdrs::Emails);
-    qDebug("SharpDTMConverter::sharpToAddressee whats the character to seperate the emailadresses? %s ", emailstr.latin1());
-    QStringList emails = QStringList::split(",", emailstr);
-
+    emailstr.replace( QRegExp(","), " " ); 
+    emailstr.replace( QRegExp(";"), " " ); 
+    emailstr.replace( QRegExp(":"), " " ); 
+    //qDebug("SharpDTMConverter::sharpToAddressee whats the character to seperate the emailadresses? %s ", emailstr.latin1());
+    QStringList emails = QStringList::split(" ", emailstr.simplifyWhiteSpace());
+    bool defE = false;
+    bool found = false;
     for ( QStringList::Iterator it = emails.begin(); it != emails.end(); ++it ) {
-      addr.insertEmail( *it, ((*it) == db->readField(ZdbAdrs::DefaultEmail)) );
+        if (found )
+            defE = false;
+        else
+            found = defE = ((*it).lower() == db->readField(ZdbAdrs::DefaultEmail).lower());
+      addr.insertEmail( *it, defE );
     }
-
-    if (!db->readField(ZdbAdrs::DefaultEmail).isEmpty())
-      addr.insertEmail(db->readField(ZdbAdrs::DefaultEmail), true);
+    if ( ! found )
+        if (!db->readField(ZdbAdrs::DefaultEmail).isEmpty())
+            addr.insertEmail(db->readField(ZdbAdrs::DefaultEmail), true);
 
     // home
@@ -199,5 +209,5 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
     }
 
-    addr.setRole( db->readField(ZdbAdrs::JobTitle) ); //?
+    addr.setRole( db->readField(ZdbAdrs::JobTitle) );
     addr.setOrganization( db->readField(ZdbAdrs::Company) );
     addr.insertCustom( "KADDRESSBOOK", "X-Profession", db->readField(ZdbAdrs::Profession) );
@@ -211,6 +221,6 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
 
         QString gen = db->readField(ZdbAdrs::Gender);
-        qDebug("SharpDTMConverter::sharpToAddressee pleas check that gender works!! : Gender: %s", gen.latin1());
-        qDebug("SharpDTMConverter::sharpToAddressee: may be int db->readUshortField(\"ZdbAdrs::Gender\") is here better suited");
+        //qDebug("SharpDTMConverter::sharpToAddressee pleas check that gender works!! : Gender: %s", gen.latin1());
+        //qDebug("SharpDTMConverter::sharpToAddressee: may be int db->readUshortField(\"ZdbAdrs::Gender\") is here better suited");
 
     if (gen == "male")
@@ -220,17 +230,17 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
 
 
-        QDate ann = db->readDateField(ZdbAdrs::Anniversary);
-
+    QDate ann = KGlobal::locale()->readDate( db->readField(ZdbAdrs::Anniversary) ); 
     if (ann.isValid()) {
       QString dt = KGlobal::locale()->formatDate(ann, true, KLocale::ISODate);
-//US
-//      qDebug("QtopiaConverter::qtopiaToAddressee found:%s", dt.latin1());
+      //qDebug("qtopiaToAddressee annyversary found:%s", dt.latin1());
       addr.insertCustom( "KADDRESSBOOK", "X-Anniversary", dt);
     }
+  
 
     addr.insertCustom( "KADDRESSBOOK", "X-Children", db->readField(ZdbAdrs::Children) );
 
 
-        QDate birthd = db->readDateField(ZdbAdrs::Birthday);
+    QDate birthd = KGlobal::locale()->readDate( db->readField(ZdbAdrs::Birthday) );
+    //qDebug("birtd %s ", birthd.toString().latin1());
     if (birthd.isValid())
       addr.setBirthday( birthd );
@@ -241,6 +251,7 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
     //US I put opies BusinessWebPage into Ka/Pi's notes block, because no other native field is available.
     QString notes = db->readField(ZdbAdrs::Notes);
-    notes += "\nBusinessWebPage: " + db->readField(ZdbAdrs::BusinessWebPage) + "\n";
-
+    //notes += "\nBusinessWebPage: " + db->readField(ZdbAdrs::BusinessWebPage) + "\n";
+    if ( addr.url().isEmpty() )
+        addr.setUrl( db->readField(ZdbAdrs::BusinessWebPage) );
     addr.setNote( db->readField(ZdbAdrs::Notes) );
 
@@ -250,5 +261,5 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
 //US    QStringList groupList() const;
 
-        qDebug("SharpDTMConverter::sharpToAddressee please check that the categories will be converted" );
+        //qDebug("SharpDTMConverter::sharpToAddressee please check that the categories will be converted" );
     QArray<int> catArray = db->readCategories();
     QString cat;
@@ -267,4 +278,5 @@ bool SharpDTMConverter::sharpToAddressee( const CardId &contact, const SlZDataBa
 bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* database , const CardId &contact )
 {
+    bool cellHome = false, cellWork = false;
     // name
     database->writeField(ZdbAdrs::LastName, addr.familyName());
@@ -274,12 +286,12 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
     database->writeField(ZdbAdrs::Suffix, addr.suffix());
 
-    qDebug("SharpDTMConverter::addresseeToSharp check if the fileAs transformation works!!");
+    //qDebug("SharpDTMConverter::addresseeToSharp check if the fileAs transformation works!!\n%s",addr.formattedName().latin1() );
     database->writeField(ZdbAdrs::FileAs, addr.formattedName());
-
+    database->writeField(ZdbAdrs::FullName, addr.formattedName());
 
     // email
-    qDebug("SharpDTMConverter::addresseeToSharp check which seperator we need here for the emails!!");
-    qDebug("SharpDTMConverter::addresseeToSharp its probably the same from sharpToAddressee");
-    QString emails = addr.emails().join(",");
+    //qDebug("SharpDTMConverter::addresseeToSharp check which seperator we need here for the emails!!");
+    //qDebug("SharpDTMConverter::addresseeToSharp its probably the same from sharpToAddressee");
+    QString emails = addr.emails().join(" ");
     database->writeField(ZdbAdrs::Emails, emails );
 
@@ -306,6 +318,8 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
 
     PhoneNumber homemobile = addr.phoneNumber( PhoneNumber::Home | PhoneNumber::Cell );
-    if (!homemobile.number().isEmpty())
+    if (!homemobile.number().isEmpty()) {
       database->writeField(ZdbAdrs::HomeMobile, homemobile.number());
+      cellHome = true;
+    }
 
     database->writeField(ZdbAdrs::HomeWebPage, addr.url().url());
@@ -331,6 +345,15 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
 
     PhoneNumber businessmobile = addr.phoneNumber( PhoneNumber::Work | PhoneNumber::Cell );
-    if (!businessmobile.number().isEmpty())
-      database->writeField(ZdbAdrs::BusinessMobile, businessmobile.number());
+    if (!businessmobile.number().isEmpty()) {
+        cellWork = true;
+        database->writeField(ZdbAdrs::BusinessMobile, businessmobile.number());
+    }
+    PhoneNumber mobile = addr.phoneNumber( PhoneNumber::Cell );
+    if (!businessmobile.number().isEmpty()) {
+        if ( ! cellHome ) 
+            database->writeField(ZdbAdrs::HomeMobile, mobile.number());
+        else if (! cellWork )
+            database->writeField(ZdbAdrs::BusinessMobile, mobile.number());
+    }
 
     PhoneNumber businesspager = addr.phoneNumber( PhoneNumber::Work | PhoneNumber::Pager );
@@ -351,10 +374,10 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
 
     QString gt = addr.custom( "KADDRESSBOOK", "X-Gender" );
-        qDebug("SharpDTMConverter::addresseeToSharp please check that gender works!! : Gender: %s", gt.latin1());
-        qDebug("SharpDTMConverter::addresseeToSharp: may be writeField(\"ZdbAdrs::Gender\", ushort) is here better suited?");
-        qDebug("SharpDTMConverter::addresseeToSharp: check also the reverse functionality in sharpToAddressee");
-    if (gt = "male")
+        //qDebug("SharpDTMConverter::addresseeToSharp please check that gender works!! : Gender: %s", gt.latin1());
+        //qDebug("SharpDTMConverter::addresseeToSharp: may be writeField(\"ZdbAdrs::Gender\", ushort) is here better suited?");
+        //qDebug("SharpDTMConverter::addresseeToSharp: check also the reverse functionality in sharpToAddressee");
+    if (gt == "male")
       database->writeField(ZdbAdrs::Gender, "male");
-    else if (gt = "female")
+    else if (gt == "female")
       database->writeField(ZdbAdrs::Gender, "female");
     else
@@ -364,9 +387,14 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
     QDate dt = KGlobal::locale()->readDate(
             addr.custom("KADDRESSBOOK", "X-Anniversary" ), "%Y-%m-%d"); // = Qt::ISODate
-    database->writeField(ZdbAdrs::Anniversary, dt );
-
+    if ( dt.isValid() ) {
+        QString dateS =  KGlobal::locale()->formatDate(dt, true  );
+        database->writeField(ZdbAdrs::Anniversary, dateS );
+    }
     database->writeField(ZdbAdrs::Children, addr.custom( "KADDRESSBOOK", "X-Children" ));
-
-    database->writeField(ZdbAdrs::Birthday, addr.birthday().date());
+    dt = addr.birthday().date();
+    if ( dt.isValid() ) {
+        QString dateS =  KGlobal::locale()->formatDate(dt, true );
+        database->writeField(ZdbAdrs::Birthday, dateS);
+    }
     database->writeField(ZdbAdrs::Nickname, addr.nickName());
 
@@ -378,10 +406,16 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
 
 
-        qDebug("SharpDTMConverter::addresseeToSharp please check if category transformation works");
+        //qDebug("SharpDTMConverter::addresseeToSharp please check if category transformation works");
 
     QStringList cats = addr.categories();
+    int ii;
+    for ( ii = 0; ii < cats.count() ;++ii ) {
+        if ( !catDB->exists( cats[ii]  ))
+            catDB->addCategory(cats[ii]);
+    }
     QArray<int> iar;
     if ( !cats.isEmpty() ) {
       QArray<int> iar = catDB->ids(cats);
+      qDebug("***cat count is %d ", iar.count());
       database->updateCategories(contact, iar);
     }
@@ -389,2 +423,9 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
     return true;
 }
+
+QDate SharpDTMConverter::convertDate( QString  s)
+{
+    QDate dt = KGlobal::locale()->readDate( s ); 
+    return dt;
+    return QDate ();
+}
diff --git a/kabc/plugins/sharpdtm/sharpdtmconverter.h b/kabc/plugins/sharpdtm/sharpdtmconverter.h
index 99e0413..a04083e 100644
--- a/kabc/plugins/sharpdtm/sharpdtmconverter.h
+++ b/kabc/plugins/sharpdtm/sharpdtmconverter.h
@@ -79,4 +79,5 @@ public:
   private:
     SlCategory::SlCategories* catDB;
+    QDate convertDate( QString );