Changes in Backendfactory: Now providing VCARD/VCAL backends if available

Changing API: Default now gets a OPimGlobal::PimType as first param instead of QString.

5 files changed, 63 insertions, 46 deletions

diff --git a/libopie2/opiepim/backend/obackendfactory.h b/libopie2/opiepim/backend/obackendfactory.h
index 4bedbef..c8a4627 100644
--- a/libopie2/opiepim/backend/obackendfactory.h
+++ b/libopie2/opiepim/backend/obackendfactory.h
@@ -40,5 +40,8 @@
 
+#include <opie2/opimaccessbackend.h>
 #include <opie2/opimglobal.h>
 #include <opie2/otodoaccessxml.h>
+#include <opie2/otodoaccessvcal.h>
 #include <opie2/ocontactaccessbackend_xml.h>
+#include <opie2/ocontactaccessbackend_vcard.h>
 #include <opie2/odatebookaccessbackend_xml.h>
@@ -84,44 +87,73 @@ class OBackendFactory
          * @param appName The name of your application. It will be passed on to the backend.
+         * @param filename Filename of the database file if you don't want to access the default 
          */
         static T* create( OPimGlobal::PimType type, OPimGlobal::DatabaseStyle database,
-                          const QString& appName ){
+                          const QString& appName, const QString& filename = QString::null ){
                 qWarning("Selected backend for %d is: %d", type, database );
                 // If we should use the dafult database style, we have to request it
-                OPimGlobal::DatabaseStyle used_database = database;
-                if ( database == OPimGlobal::DEFAULT ){
-                        used_database = defaultDB( type );
+                OPimGlobal::DatabaseStyle use_database = database;
+                if ( use_database == OPimGlobal::DEFAULT ){
+                        use_database = defaultDB( type );
                 }
 
-
                 switch ( type ){
                 case OPimGlobal::TODOLIST:
+
+                        switch ( use_database ){
+                        default: // Use SQL if something weird is given.
+                                // Fall through !!
+                        case OPimGlobal::SQL:
 #ifdef __USE_SQL
-                        if ( used_database == OPimGlobal::SQL )
-                                return (T*) new OPimTodoAccessBackendSQL("");
+                                return (T*) new OPimTodoAccessBackendSQL( filename );
+                                break;
 #else
-                        if ( used_database == OPimGlobal::SQL )
                                 qWarning ("OBackendFactory:: sql Backend for TODO not implemented! Using XML instead!");
+                                // Fall through !!
 #endif
-
-                        return (T*) new OPimTodoAccessXML( appName );
+                        case OPimGlobal::XML:
+                                return (T*) new OPimTodoAccessXML( appName, filename );
+                                break;
+                        case OPimGlobal::VCARD:
+                                return (T*) new OPimTodoAccessVCal( filename );
+                                break;
+                        }
                 case OPimGlobal::CONTACTLIST:
+                        switch ( use_database ){
+                        default: // Use SQL if something weird is given.
+                                // Fall through !!
+                        case OPimGlobal::SQL:
 #ifdef __USE_SQL
-                        if ( used_database == OPimGlobal::SQL )
-                                return (T*) new OPimContactAccessBackend_SQL("");
+                                return (T*) new OPimContactAccessBackend_SQL( appName, filename );
+                                break;
 #else
-                        if ( used_database == OPimGlobal::SQL )
                                 qWarning ("OBackendFactory:: sql Backend for CONTACT not implemented! Using XML instead!");
+                                // Fall through !!
 #endif
-
-                        return (T*) new OPimContactAccessBackend_XML( appName );
+                        case OPimGlobal::XML:
+                                return (T*) new OPimContactAccessBackend_XML( appName, filename );
+                                break;
+                        case OPimGlobal::VCARD:
+                                return (T*) new OPimContactAccessBackend_VCard( appName, filename );
+                                break;
+                        }
                 case OPimGlobal::DATEBOOK:
+                        switch ( use_database ){
+                        default: // Use SQL if something weird is given.
+                                // Fall through !!
+                        case OPimGlobal::SQL:
 #ifdef __USE_SQL
-                        if ( used_database == OPimGlobal::SQL )
-                                return (T*) new ODateBookAccessBackend_SQL("");
+                                return (T*) new ODateBookAccessBackend_SQL( appName, filename );
+                                break;
 #else
-                        if ( used_database == OPimGlobal::SQL )
-                            qWarning("OBackendFactory:: sql Backend for DATEBOOK not implemented! Using XML instead!");
+                                qWarning("OBackendFactory:: sql Backend for DATEBOOK not implemented! Using XML instead!");
+                                // Fall through !!
 #endif
-
-                        return (T*) new ODateBookAccessBackend_XML( appName );
+                        case OPimGlobal::XML:
+                                return (T*) new ODateBookAccessBackend_XML( appName, filename );
+                                break;
+                        case OPimGlobal::VCARD:
+                                qWarning("OBackendFactory:: VCal Backend for DATEBOOK not implemented!");
+                                return (T*) NULL;
+                                break;
+                        }
                 default:
@@ -130,3 +162,2 @@ class OBackendFactory
 
-
         }
@@ -177,24 +208,7 @@ class OBackendFactory
          * by the configfile "pimaccess.conf".
-         * @param backendName the type of the backend (use "todo", "contact" or "datebook" )
+         * @param type The type of the backend (@see OPimGlobal())
          * @param appName The name of your application. It will be passed on to the backend
          */
-        static T* defaultBackend( const QString backendName, const QString& appName ){
-
-                QAsciiDict<int> dictBackends( OPimGlobal::_END_PimType );
-                dictBackends.setAutoDelete ( TRUE );
-
-                dictBackends.insert( "todo", new int (OPimGlobal::TODOLIST) );
-                dictBackends.insert( "contact", new int (OPimGlobal::CONTACTLIST) );
-                dictBackends.insert( "datebook", new int(OPimGlobal::DATEBOOK) );
-
-                int* backend_find = dictBackends[ backendName ];
-                if ( !backend_find ) return NULL;
-
-                OPimGlobal::DatabaseStyle style = defaultDB( static_cast<OPimGlobal::PimType>( *backend_find ) );
-
-                qDebug( "OBackendFactory::Default -> Backend is %s, Database is %d", backendName.latin1(),
-                        style );
-
-                return create( (OPimGlobal::PimType) *backend_find, style, appName );
-
+        static T* defaultBackend( OPimGlobal::PimType type, const QString& appName ){
+                return create( type, OPimGlobal::DEFAULT, appName );
         }
diff --git a/libopie2/opiepim/core/ocontactaccess.cpp b/libopie2/opiepim/core/ocontactaccess.cpp
index 417a954..a372267 100644
--- a/libopie2/opiepim/core/ocontactaccess.cpp
+++ b/libopie2/opiepim/core/ocontactaccess.cpp
@@ -55,2 +55,3 @@
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
@@ -67,3 +68,3 @@ OPimContactAccess::OPimContactAccess ( const QString appname, const QString ,
                 qWarning ("Using BackendFactory !");
-                end = OBackendFactory<OPimContactAccessBackend>::defaultBackend( "contact", appname );
+                end = OBackendFactory<OPimContactAccessBackend>::defaultBackend( OPimGlobal::CONTACTLIST, appname );
         }
diff --git a/libopie2/opiepim/core/odatebookaccess.cpp b/libopie2/opiepim/core/odatebookaccess.cpp
index 5f471cf..440ee0a 100644
--- a/libopie2/opiepim/core/odatebookaccess.cpp
+++ b/libopie2/opiepim/core/odatebookaccess.cpp
@@ -31,2 +31,3 @@
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
@@ -44,3 +45,3 @@ ODateBookAccess::ODateBookAccess( ODateBookAccessBackend* back, enum Access ac )
     if (!back )
-        back = OBackendFactory<ODateBookAccessBackend>::defaultBackend("datebook", QString::null );
+        back = OBackendFactory<ODateBookAccessBackend>::defaultBackend( OPimGlobal::DATEBOOK, QString::null );
 
diff --git a/libopie2/opiepim/core/opimglobal.h b/libopie2/opiepim/core/opimglobal.h
index d56c9be..efd455c 100644
--- a/libopie2/opiepim/core/opimglobal.h
+++ b/libopie2/opiepim/core/opimglobal.h
@@ -52,3 +52,3 @@ class OPimGlobal{
                 SQL,
-                VCARD,
+                VCARD,        // Also used for VCARD !
                 _END_DatabaseStyle
diff --git a/libopie2/opiepim/core/otodoaccess.cpp b/libopie2/opiepim/core/otodoaccess.cpp
index 87a211f..c4adbbd 100644
--- a/libopie2/opiepim/core/otodoaccess.cpp
+++ b/libopie2/opiepim/core/otodoaccess.cpp
@@ -36,2 +36,3 @@
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
@@ -46,3 +47,3 @@ OPimTodoAccess::OPimTodoAccess( OPimTodoAccessBackend* end, enum Access )
         if (end == 0l )
-                m_todoBackEnd = OBackendFactory<OPimTodoAccessBackend>::defaultBackend ("todo", QString::null);
+                m_todoBackEnd = OBackendFactory<OPimTodoAccessBackend>::defaultBackend (OPimGlobal::TODOLIST, QString::null);