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
@@ -1,208 +1,208 @@
 /*
                              This file is part of the Opie Project
                              Copyright (C) Stefan Eilers <eilers.stefan@epost.de>
               =.             Copyright (C) The Opie Team <opie-devel@handhelds.org>
             .=l.
            .>+-=
  _;:,     .>    :=|.         This program is free software; you can
 .> <`_,   >  .   <=          redistribute it and/or  modify it under
 :`=1 )Y*s>-.--   :           the terms of the GNU Library General Public
 .="- .-=="i,     .._         License as published by the Free Software
  - .   .-<_>     .<>         Foundation; either version 2 of the License,
      ._= =}       :          or (at your option) any later version.
     .%`+i>       _;_.
     .i_,=:_.      -<s.       This program is distributed in the hope that
      +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
     : ..    .:,     . . .    without even the implied warranty of
     =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
   _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
 ..}^=.=       =       ;      Library General Public License for more
 ++=   -.     .`     .:       details.
  :     =  ...= . :.=-
  -.   .:....=;==+<;          You should have received a copy of the GNU
   -_. . .   )=.  =           Library General Public License along with
     --        :-=`           this library; see the file COPYING.LIB.
                              If not, write to the Free Software Foundation,
                              Inc., 59 Temple Place - Suite 330,
                              Boston, MA 02111-1307, USA.
 */
 /*
  * =====================================================================
  * ToDo: Use plugins
  * =====================================================================
  */
 #ifndef OPIE_BACKENDFACTORY_H_
 #define OPIE_BACKENDFACTORY_H_
 
 #include <qstring.h>
 #include <qasciidict.h>
 #include <qpe/config.h>
 
 #include <opie2/opimglobal.h>
 #include <opie2/otodoaccessxml.h>
 #include <opie2/ocontactaccessbackend_xml.h>
 #include <opie2/odatebookaccessbackend_xml.h>
 
 #ifdef __USE_SQL
 #include <opie2/otodoaccesssql.h>
 #include <opie2/ocontactaccessbackend_sql.h>
 #include <opie2/odatebookaccessbackend_sql.h>
 #endif
 
 using namespace Opie;
 using namespace Opie::Pim;
 
 namespace Opie {
 
 class OBackendPrivate;
 
 /**
  * This class is our factory. It will give us the default implementations
  * of at least Todolist, Contacts and Datebook. In the future this class will
  * allow users to switch the backend with ( XML->SQLite ) without the need
  * to recompile.#
  * This class - as the whole PIM Api - is making use of templates
  *
  * <pre>
  *   OPimTodoAccessBackend* backend = OBackEndFactory<OPimTodoAccessBackend>::Default("todo", QString::null );
  *   backend->load();
  * </pre>
  *
  * @author Stefan Eilers
  * @version 0.1
  */
 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 );
                 case OPimGlobal::CONTACTLIST:
 #ifdef __USE_SQL
                         if ( used_database == OPimGlobal::SQL )
                                 return (T*) new OPimContactAccessBackend_SQL("");
 #else
                         if ( used_database == OPimGlobal::SQL )
                                 qWarning ("OBackendFactory:: sql Backend for CONTACT not implemented! Using XML instead!");
 #endif
 
                         return (T*) new OPimContactAccessBackend_XML( appName );
                 case OPimGlobal::DATEBOOK:
 #ifdef __USE_SQL
                         if ( used_database == OPimGlobal::SQL )
                                 return (T*) new ODateBookAccessBackend_SQL("");
 #else
                         if ( used_database == OPimGlobal::SQL )
                             qWarning("OBackendFactory:: sql Backend for DATEBOOK not implemented! Using XML instead!");
 #endif
 
                         return (T*) new ODateBookAccessBackend_XML( appName );
                 default:
                         return (T*) NULL;
                 }
 
 
         }
 
         /**
          * Returns the style of the default database which is used to contact PIM data.
          * @param type the type of the backend
          * @see OPimGlobal
          */
         static OPimGlobal::DatabaseStyle defaultDB( OPimGlobal::PimType type ){
                 QString group_name;
                 switch ( type ){
                 case OPimGlobal::TODOLIST:
                         group_name = "todo";
                         break;
                 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;
 
                 return (OPimGlobal::DatabaseStyle) *db_find;
         }
 
 
         /**
          * 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 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(), 
+                qDebug( "OBackendFactory::Default -> Backend is %s, Database is %d", backendName.latin1(),
                         style );
-                
+
                 return create( (OPimGlobal::PimType) *backend_find, style, appName );
 
         }
     private:
         OBackendPrivate* d;
-        
+
 };
 
 }
 
 #endif