add a 'default' backend for readEntry or you could/will crash

because QString::null is not in the dictionary of available backends.
I'm conversative so it is 'xml'

Eilers/BradC change it on your will

1 files changed, 7 insertions, 7 deletions

diff --git a/libopie2/opiepim/backend/obackendfactory.h b/libopie2/opiepim/backend/obackendfactory.h
index 4daf861..4bedbef 100644
--- a/libopie2/opiepim/backend/obackendfactory.h
+++ b/libopie2/opiepim/backend/obackendfactory.h
@@ -74,33 +74,33 @@ class OBackendPrivate;
 template<class T>
 class OBackendFactory
 {
  public:
         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.
          */
-        static T* create( OPimGlobal::PimType type, OPimGlobal::DatabaseStyle database, 
+        static T* create( OPimGlobal::PimType type, OPimGlobal::DatabaseStyle database,
                           const QString& appName ){
                 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 );
                 }
- 
+
 
                 switch ( type ){
                 case OPimGlobal::TODOLIST:
 #ifdef __USE_SQL
                         if ( used_database == OPimGlobal::SQL )
                                 return (T*) new OPimTodoAccessBackendSQL("");
 #else
                         if ( used_database == OPimGlobal::SQL )
                                 qWarning ("OBackendFactory:: sql Backend for TODO not implemented! Using XML instead!");
 #endif
 
                         return (T*) new OPimTodoAccessXML( appName );
@@ -145,26 +145,26 @@ class OBackendFactory
                 case OPimGlobal::CONTACTLIST:
                         group_name = "contact";
                         break;
                 case OPimGlobal::DATEBOOK:
                         group_name = "datebook";
                         break;
                 default:
                         group_name = "unknown";
                 }
 
                 Config config( "pimaccess" );
                 config.setGroup ( group_name );
-                QString db_String = config.readEntry( "usebackend" );
-                
+                QString db_String = config.readEntry( "usebackend", "xml" );
+
                 QAsciiDict<int> dictDbTypes( OPimGlobal::_END_DatabaseStyle );
                 dictDbTypes.setAutoDelete( TRUE );
 
                 dictDbTypes.insert( "xml", new int (OPimGlobal::XML) );
                 dictDbTypes.insert( "sql", new int (OPimGlobal::SQL) );
                 dictDbTypes.insert( "vcard", new int (OPimGlobal::VCARD) );
 
                 int* db_find = dictDbTypes[ db_String ];
 
                 if ( !db_find )
                         return OPimGlobal::UNKNOWN;
 
@@ -183,26 +183,26 @@ class OBackendFactory
                 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(), 
+                qDebug( "OBackendFactory::Default -> Backend is %s, Database is %d", backendName.latin1(),
                         style );
-                
+
                 return create( (OPimGlobal::PimType) *backend_find, style, appName );
 
         }
     private:
         OBackendPrivate* d;
-        
+
 };
 
 }
 
 #endif