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
@@ -37,11 +37,14 @@
 #include <qstring.h>
 #include <qasciidict.h>
 #include <qpe/config.h>
 
+#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>
 
 #ifdef __USE_SQL
 #include <opie2/otodoaccesssql.h>
@@ -81,55 +84,83 @@ class OBackendFactory
          * Returns a selected backend implementation
          * @param type the type of the backend
          * @param database the type of the used database
          * @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:
                         return (T*) NULL;
                 }
 
-
         }
 
         /**
          * Returns the style of the default database which is used to contact PIM data.
@@ -174,30 +205,13 @@ class OBackendFactory
 
         /**
          * Returns the default backend implementation for backendName. Which one is used, is defined
          * 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 );
         }
     private:
         OBackendPrivate* d;
 
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
@@ -52,8 +52,9 @@
 #include <stdlib.h>
 
 #include <opie2/ocontactaccessbackend_xml.h>
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
 namespace Opie {
 
 OPimContactAccess::OPimContactAccess ( const QString appname, const QString ,
@@ -64,9 +65,9 @@ OPimContactAccess::OPimContactAccess ( const QString appname, const QString ,
          * will use the XML-Backend as default (until we have a cute SQL-Backend..).
          */
         if( end == 0 ) {
                 qWarning ("Using BackendFactory !");
-                end = OBackendFactory<OPimContactAccessBackend>::defaultBackend( "contact", appname );
+                end = OBackendFactory<OPimContactAccessBackend>::defaultBackend( OPimGlobal::CONTACTLIST, appname );
         }
         // Set backend locally and in template
         m_backEnd = end;
         OPimAccessTemplate<OPimContact>::setBackEnd (end);
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
@@ -28,8 +28,9 @@
 */
 #include <opie2/obackendfactory.h>
 #include <opie2/odatebookaccess.h>
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
 namespace Opie {
 /**
  * Simple constructor
@@ -41,9 +42,9 @@ namespace Opie {
 ODateBookAccess::ODateBookAccess( ODateBookAccessBackend* back, enum Access ac )
     : OPimAccessTemplate<OPimEvent>( back )
 {
     if (!back )
-        back = OBackendFactory<ODateBookAccessBackend>::defaultBackend("datebook", QString::null );
+        back = OBackendFactory<ODateBookAccessBackend>::defaultBackend( OPimGlobal::DATEBOOK, QString::null );
 
     m_backEnd = back;
     setBackEnd( m_backEnd );
 }
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
@@ -49,9 +49,9 @@ class OPimGlobal{
                 DEFAULT,      // Use default Database
                 UNKNOWN,      // Unknown database style
                 XML,
                 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
@@ -33,8 +33,9 @@
 // #include "otodoaccesssql.h"
 #include <opie2/otodoaccess.h>
 #include <opie2/obackendfactory.h>
 #include <opie2/opimresolver.h>
+#include <opie2/opimglobal.h>
 
 namespace Opie {
 OPimTodoAccess::OPimTodoAccess( OPimTodoAccessBackend* end, enum Access )
     : QObject(), OPimAccessTemplate<OPimTodo>( end ),  m_todoBackEnd( end )
@@ -43,9 +44,9 @@ OPimTodoAccess::OPimTodoAccess( OPimTodoAccessBackend* end, enum Access )
 //         m_todoBackEnd = new OPimTodoAccessBackendSQL( QString::null);
 
         // Zecke: Du musst hier noch fr das XML-Backend einen Appnamen bergeben !
         if (end == 0l )
-                m_todoBackEnd = OBackendFactory<OPimTodoAccessBackend>::defaultBackend ("todo", QString::null);
+                m_todoBackEnd = OBackendFactory<OPimTodoAccessBackend>::defaultBackend (OPimGlobal::TODOLIST, QString::null);
 
     setBackEnd( m_todoBackEnd );
 }
 OPimTodoAccess::~OPimTodoAccess() {