fixed DTM sync cat handling

4 files changed, 109 insertions, 61 deletions

diff --git a/bin/kdepim/WhatsNew.txt b/bin/kdepim/WhatsNew.txt
index 756ccc1..f38dd70 100644
--- a/bin/kdepim/WhatsNew.txt
+++ b/bin/kdepim/WhatsNew.txt
@@ -4,8 +4,10 @@ Info about the changes in new versions of KDE-Pim/Pi
 
 Fixed several problems in PWM/Pi, like
 asking the user, if unsaved changed are pending
 when closing the app.
+And PwM/Pi handles now different texts for the
+fields Description, Username, Password, configurable per category.
 
 Fixed a crash in KO/Pi , when importing/loading vcs files
 which have an entry with an attendee with state:
 NEEDS ACTION
@@ -44,11 +46,16 @@ non-latin1 characters.
 
 Rearranged package contents of Sharp-ipks and made all
 packages installable on SD again.
 
-PwM/Pi handles now different texts for the
-fields Description, Username, Password, configurable per category.
+Fixed the writing of addressbook data in DTM sync.
+Empty fields in KA/Pi were not removed. 
 
+Added better category handling in KA/Pi:
+Menu "change selected has now" 
+Set Categories and
+Add Categories option.
+Possible to configure a view to display categories.
 
 
 ********** VERSION 1.9.10 ************
 
diff --git a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
index 7069f5d..6413b42 100644
--- a/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
+++ b/kabc/plugins/sharpdtm/resourcesharpdtm.cpp
@@ -199,9 +199,10 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                     {
                         res = mConverter->addresseeToSharp( (*it), mAccess, id  );
                         if (res == true)
                             {
-                                res = mAccess->finishEditCard(&id);; 
+                                res = mAccess->finishEditCard(&id);;
+                                mConverter->setCategories( (*it), mAccess, id  );
                                 map.insert(id,(*it).uid());                           
                                 if (res == false)
                                     qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());
                              
@@ -227,8 +228,9 @@ bool ResourceSharpDTM::save( Ticket *ticket )
                         res = mConverter->addresseeToSharp( (*it), mAccess, id  );
                         if (res == true)
                             {
                                 res = mAccess->finishEditCard(&id);
+                                mConverter->setCategories( (*it), mAccess, id  );
                                 map.insert(id,(*it).uid());
                                 if (res == false)
                                     qDebug("Unable to append Contact: %s", addressee.formattedName().latin1());         
                             }
diff --git a/kabc/plugins/sharpdtm/sharpdtmconverter.cpp b/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
index 396be0a..ae4cfbb 100644
--- a/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
+++ b/kabc/plugins/sharpdtm/sharpdtmconverter.cpp
@@ -306,61 +306,45 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
     QString emails = addr.emails().join(" ");
     database->writeField(ZdbAdrs::Emails, emails );
 
     database->writeField(ZdbAdrs::DefaultEmail, addr.preferredEmail() );
-
-
     // home
     const Address homeaddress = addr.address(Address::Home);
-    if (!homeaddress.isEmpty()) {
-        //qDebug("write home address ");
-      database->writeField(ZdbAdrs::HomeStreet, homeaddress.street());
-      database->writeField(ZdbAdrs::HomeCity, homeaddress.locality());
-      database->writeField(ZdbAdrs::HomeState, homeaddress.region());
-      database->writeField(ZdbAdrs::HomeZip, homeaddress.postalCode());
-      database->writeField(ZdbAdrs::HomeCountry, homeaddress.country());
-    }
-
+    database->writeField(ZdbAdrs::HomeStreet, homeaddress.street());
+    database->writeField(ZdbAdrs::HomeCity, homeaddress.locality());
+    database->writeField(ZdbAdrs::HomeState, homeaddress.region());
+    database->writeField(ZdbAdrs::HomeZip, homeaddress.postalCode());
+    database->writeField(ZdbAdrs::HomeCountry, homeaddress.country());
+    
     PhoneNumber homephone = addr.phoneNumber( PhoneNumber::Home );
-    if (!homephone.number().isEmpty())
-      database->writeField(ZdbAdrs::HomePhone, homephone.number());
-
+    database->writeField(ZdbAdrs::HomePhone, homephone.number());
     PhoneNumber homefax = addr.phoneNumber( PhoneNumber::Home | PhoneNumber::Fax );
-    if (!homefax.number().isEmpty())
-      database->writeField(ZdbAdrs::HomeFax, homefax.number());
-
+    database->writeField(ZdbAdrs::HomeFax, homefax.number());
     PhoneNumber homemobile = addr.phoneNumber( PhoneNumber::Home | PhoneNumber::Cell );
+    database->writeField(ZdbAdrs::HomeMobile, homemobile.number());
     if (!homemobile.number().isEmpty()) {
-      database->writeField(ZdbAdrs::HomeMobile, homemobile.number());
       cellHome = true;
     }
-
     database->writeField(ZdbAdrs::HomeWebPage, addr.url().url());
-
-
     // business
     const Address businessaddress = addr.address(Address::Work);
-    if (!businessaddress.isEmpty()) {
         //qDebug("write business address ");
-      database->writeField(ZdbAdrs::BusinessStreet, businessaddress.street());
-      database->writeField(ZdbAdrs::BusinessCity, businessaddress.locality());
-      database->writeField(ZdbAdrs::BusinessState, businessaddress.region());
-      database->writeField(ZdbAdrs::BusinessZip, businessaddress.postalCode());
-      database->writeField(ZdbAdrs::BusinessCountry, businessaddress.country());
-    }
-
+    database->writeField(ZdbAdrs::BusinessStreet, businessaddress.street());
+    database->writeField(ZdbAdrs::BusinessCity, businessaddress.locality());
+    database->writeField(ZdbAdrs::BusinessState, businessaddress.region());
+    database->writeField(ZdbAdrs::BusinessZip, businessaddress.postalCode());
+    database->writeField(ZdbAdrs::BusinessCountry, businessaddress.country());
+    
     PhoneNumber businessphone = addr.phoneNumber( PhoneNumber::Work );
-    if (!businessphone.number().isEmpty())
-      database->writeField(ZdbAdrs::BusinessPhone, businessphone.number());
+    database->writeField(ZdbAdrs::BusinessPhone, businessphone.number());
 
     PhoneNumber businessfax = addr.phoneNumber( PhoneNumber::Work | PhoneNumber::Fax );
-    if (!businessfax.number().isEmpty())
-      database->writeField(ZdbAdrs::BusinessFax, businessfax.number());
+    database->writeField(ZdbAdrs::BusinessFax, businessfax.number());
 
     PhoneNumber businessmobile = addr.phoneNumber( PhoneNumber::Work | PhoneNumber::Cell );
+    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 (! mobile.number().isEmpty()) {
         if ( ! cellHome ) 
@@ -369,10 +353,9 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
             database->writeField(ZdbAdrs::BusinessMobile, mobile.number());
     }
 
     PhoneNumber businesspager = addr.phoneNumber( PhoneNumber::Work | PhoneNumber::Pager );
-    if (!businesspager.number().isEmpty())
-      database->writeField(ZdbAdrs::BusinessPager, businesspager.number());
+    database->writeField(ZdbAdrs::BusinessPager, businesspager.number());
 
     database->writeField(ZdbAdrs::JobTitle, addr.role());
     database->writeField(ZdbAdrs::Company, addr.organization());
 
@@ -395,48 +378,103 @@ bool SharpDTMConverter::addresseeToSharp( const Addressee &addr, SlZDataBase* da
       database->writeField(ZdbAdrs::Gender, "female");
     else
       database->writeField(ZdbAdrs::Gender, "");
 
-
-    QDate dt = KGlobal::locale()->readDate(
-            addr.custom("KADDRESSBOOK", "X-Anniversary" ), "%Y-%m-%d"); // = Qt::ISODate
+    QString dateS ;
+    QDate dt = KGlobal::locale()->readDate(addr.custom("KADDRESSBOOK", "X-Anniversary" ), "%Y-%m-%d"); // = Qt::ISODate
     if ( dt.isValid() ) {
-        QString dateS =  KGlobal::locale()->formatDate(dt, true  );
-        database->writeField(ZdbAdrs::Anniversary, dateS );
+        dateS =  KGlobal::locale()->formatDate(dt, true  );
     }
+    database->writeField(ZdbAdrs::Anniversary, dateS );
     database->writeField(ZdbAdrs::Children, addr.custom( "KADDRESSBOOK", "X-Children" ));
-    dt = addr.birthday().date();
+    dt = addr.birthday().date(); 
+    dateS = ""; 
     if ( dt.isValid() ) {
-        QString dateS =  KGlobal::locale()->formatDate(dt, true );
-        database->writeField(ZdbAdrs::Birthday, dateS);
+        dateS =  KGlobal::locale()->formatDate(dt, true );
     }
+    database->writeField(ZdbAdrs::Birthday, dateS);
     database->writeField(ZdbAdrs::Nickname, addr.nickName());
 
     // other
     database->writeField(ZdbAdrs::Notes, addr.note());
 
 //US    QString groups() const { return find( Qtopia::Groups ); }
-//US    QStringList groupList() const;
+//US    QStringList groupList() const; 
 
 
 	//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);
-    }
-
     return true;
 }
 
+
+bool SharpDTMConverter::setCategories( const Addressee &addr, SlZDataBase* database , const CardId &contact )
+{  
+    QStringList list = addr.categories();
+
+	QArray<int> ids(list.count());
+	uint index = 0;
+	for(uint i=0; i<ids.size(); i++){
+		if(catDB->exists(list[i])){
+			ids[index] = catDB->id(list[i]);
+            qDebug("set ex cat %d %s ",ids[index] , list[i].latin1());
+			index++;
+		} else {
+			ids[index] = catDB->addCategory(list[i]);
+            qDebug("add new cat %d %s ",ids[index] , list[i].latin1());
+			index++;
+		}
+	}
+    bool res ;
+    if ( res = database->updateCategories(contact, ids) )
+        qDebug("cat updated ");
+    else
+        qDebug("error cat updating ");
+
+    return res;
+}
+
+#if 0
+	m_pCategories = new SlCategories;
+catDB
+
+	if(categories == QString::null) return true;
+	QStringList list = CSVParser::parse(categories);
+	QArray<int> ids(list.count());
+	uint index = 0;
+	for(uint i=0; i<ids.size(); i++){
+#if 0
+		bool ok;
+		ids[index] = list[i].toInt(&ok);
+		if(ok){
+			index++;
+		} else {
+			if(m_pCategories->exists(list[i])){
+				ids[index] = m_pCategories->id(list[i]);
+				index++;
+			}
+		}
+#else
+		if(m_pCategories->exists(list[i])){
+			ids[index] = m_pCategories->id(list[i]);
+			index++;
+		} else if(m_addCategory){
+			ids[index] = m_pCategories->addCategory(list[i]);
+			index++;
+		}
+#endif
+	}
+	if(ids.size() > 0 && index == 0){
+		return true;
+	} else if(index != ids.size()){
+		ids.resize(index);
+	}
+	if(!m_pZdb->updateCategories(id, ids)){
+		return false;
+	}
+	return true;
+#endif
+
 QDate SharpDTMConverter::convertDate( QString  s)
 {
     QDate dt = KGlobal::locale()->readDate( s ); 
     return dt;
diff --git a/kabc/plugins/sharpdtm/sharpdtmconverter.h b/kabc/plugins/sharpdtm/sharpdtmconverter.h
index a04083e..db1bf2d 100644
--- a/kabc/plugins/sharpdtm/sharpdtmconverter.h
+++ b/kabc/plugins/sharpdtm/sharpdtmconverter.h
@@ -74,8 +74,9 @@ public:
    * @param database The sharp database pointer we use for the conversion
    * @param contact The sharp card id, eventhough it might be 0, if the contact is a new one.
    */
   bool addresseeToSharp( const Addressee &addr, SlZDataBase* database , const CardId &contact );
+  bool setCategories( const Addressee &addr, SlZDataBase* database , const CardId &contact );
 
   private:
     SlCategory::SlCategories* catDB;
     QDate convertDate( QString );