2 files changed, 81 insertions, 35 deletions

diff --git a/kabc/converter/opie/opieconverter.cpp b/kabc/converter/opie/opieconverter.cpp
index d785fbc..b4e9f1c 100644
--- a/kabc/converter/opie/opieconverter.cpp
+++ b/kabc/converter/opie/opieconverter.cpp
@@ -33,9 +33,10 @@ $Id$
 
 #include <qpe/categories.h>
+#include <qpe/categoryselect.h>
 
 
 using namespace KABC;
 
-OpieConverter::OpieConverter()
+OpieConverter::OpieConverter() : catDB(0)
 {
 }
@@ -43,4 +44,29 @@ OpieConverter::OpieConverter()
 OpieConverter::~OpieConverter()
 {
+  deinit();
+}
+
+bool OpieConverter::init()
+{
+  catDB = new Categories();
+  if (catDB) 
+  {
+    catDB->load( categoryFileName() );
+    return true;
+  }
+  else 
+  {
+    return false;
+  }
+
+}
+
+void OpieConverter::deinit()
+{
+  if (catDB) 
+  {
+    delete catDB;
+    catDB = 0;
+  }
 }
   
@@ -62,18 +88,26 @@ bool OpieConverter::opieToAddressee( OContact &contact, Addressee &addr )
     }
 
-    addr.insertEmail(contact.defaultEmail(), true);
+    if (!contact.defaultEmail().isEmpty()) 
+      addr.insertEmail(contact.defaultEmail(), true);
     
     // home
-    Address homeaddress;
-    homeaddress.setType(Address::Home);
+    if ((!contact.homeStreet().isEmpty()) ||
+        (!contact.homeCity().isEmpty()) ||
+        (!contact.homeState().isEmpty()) ||
+        (!contact.homeZip().isEmpty()) ||
+        (!contact.homeCountry().isEmpty()))
+    {
+      Address homeaddress;
+      homeaddress.setType(Address::Home);
 //US    homeaddress.setPostOfficeBox( "" );
 //US    homeaddress.setExtended( "" );
-    homeaddress.setStreet( contact.homeStreet() );
-    homeaddress.setLocality( contact.homeCity() );
-    homeaddress.setRegion( contact.homeState() );
-    homeaddress.setPostalCode( contact.homeZip() );
-    homeaddress.setCountry( contact.homeCountry() );
+      homeaddress.setStreet( contact.homeStreet() );
+      homeaddress.setLocality( contact.homeCity() );
+      homeaddress.setRegion( contact.homeState() );
+      homeaddress.setPostalCode( contact.homeZip() );
+      homeaddress.setCountry( contact.homeCountry() );
     
-    addr.insertAddress( homeaddress );
+      addr.insertAddress( homeaddress );
+    }
     
     if (!contact.homePhone().isEmpty())
@@ -105,16 +139,24 @@ bool OpieConverter::opieToAddressee( OContact &contact, Addressee &addr )
 
     // business
-    Address businessaddress;
-    businessaddress.setType(Address::Work);
+    if ((!contact.businessStreet().isEmpty()) ||
+        (!contact.businessCity().isEmpty()) ||
+        (!contact.businessState().isEmpty()) ||
+        (!contact.businessZip().isEmpty()) ||
+        (!contact.businessCountry().isEmpty()))
+    {
+      Address businessaddress;
+      businessaddress.setType(Address::Work);
 //US    businessaddress.setPostOfficeBox( "" );
 //US    businessaddress.setExtended( "" );
-    businessaddress.setStreet( contact.businessStreet() );
-    businessaddress.setLocality( contact.businessCity() );
-    businessaddress.setRegion( contact.businessState() );
-    businessaddress.setPostalCode( contact.businessZip() );
-    businessaddress.setCountry( contact.businessCountry() );
+      businessaddress.setStreet( contact.businessStreet() );
+      businessaddress.setLocality( contact.businessCity() );
+      businessaddress.setRegion( contact.businessState() );
+      businessaddress.setPostalCode( contact.businessZip() );
+      businessaddress.setCountry( contact.businessCountry() );
+    
+      addr.insertAddress( businessaddress );
+    }
+    
     
-    addr.insertAddress( businessaddress );
-
     if (!contact.businessPhone().isEmpty())
     {
@@ -162,10 +204,7 @@ bool OpieConverter::opieToAddressee( OContact &contact, Addressee &addr )
     
     if (contact.anniversary().isValid()) {
-      int orgformat = KGlobal::locale()->getIntDateFormat();
-      KGlobal::locale()->setIntDateFormat( 2 ); // = Qt::ISODate
-      QString dt = KGlobal::locale()->formatDate(contact.anniversary(), true);
+      QString dt = KGlobal::locale()->formatDate(contact.anniversary(), true, KLocale::ISODate);
 //US      
       qDebug("OpieConverter::opieToAddressee found:%s", dt.latin1());
-      KGlobal::locale()->setIntDateFormat(orgformat );
       addr.insertCustom( "KADDRESSBOOK", "X-Anniversary", dt);
     }
@@ -191,7 +230,7 @@ bool OpieConverter::opieToAddressee( OContact &contact, Addressee &addr )
     
     QStringList cats = contact.categoryNames("Contacts");
-    for ( QStringList::Iterator it = cats.begin(); it != cats.end(); ++it ) {
-      qDebug("Cat: %s", (*it).latin1());
-    }
+//    for ( QStringList::Iterator it = cats.begin(); it != cats.end(); ++it ) {
+//      qDebug("Cat: %s", (*it).latin1());
+//    }
 
         
@@ -324,14 +363,12 @@ bool OpieConverter::addresseeToOpie( const Addressee &addr, OContact &contact )
 
     Categories catDB;
-    catDB.dump();
-//US    catDB.load( contact.categoryFileName() );
-    catDB.load( "contact" );
-
+    catDB.load( categoryFileName() );
     QStringList cats = addr.categories();
-    for ( QStringList::Iterator it = cats.begin(); it != cats.end(); ++it ) {
-      qDebug("categories: %s", (*it).latin1());
-//      contact.setCategories(catDB.id(*it));
-    }
     
+    QArray<int> iar;
+    if ( !cats.isEmpty() ) {
+      QArray<int> iar = catDB.ids("contact", cats);
+      contact.setCategories(iar);
+    }
 
     return true;
diff --git a/kabc/converter/opie/opieconverter.h b/kabc/converter/opie/opieconverter.h
index ef7300b..7d2ad85 100644
--- a/kabc/converter/opie/opieconverter.h
+++ b/kabc/converter/opie/opieconverter.h
@@ -34,4 +34,6 @@ $Id$
 #include <opie/ocontact.h>
 
+class Categories;
+
 namespace KABC {
 
@@ -48,5 +50,8 @@ public:
    * Destructor.
    */
-  ~OpieConverter();
+  virtual ~OpieConverter();
+  
+  virtual bool init();
+  virtual void deinit();
   
   /**
@@ -65,5 +70,9 @@ public:
    */
   bool addresseeToOpie( const Addressee &addr, OContact &contact );
+
+  private:
+    Categories* catDB;
   
+    
 };