6 files changed, 194 insertions, 119 deletions

diff --git a/libopie/pim/opimaccessbackend.h b/libopie/pim/opimaccessbackend.h
index d9af589..8e744e7 100644
--- a/libopie/pim/opimaccessbackend.h
+++ b/libopie/pim/opimaccessbackend.h
@@ -10,6 +10,4 @@ class OPimAccessBackend {
 public:
-    OPimAccessBackend() {
-    }
-    ~OPimAccessBackend() {
-    }
+    OPimAccessBackend();
+    virtual ~OPimAccessBackend();
     virtual void load() = 0;
@@ -17,3 +15,3 @@ public:
     virtual void save() = 0;
-    virtual QArray<int> allRecords() = 0;
+    virtual QArray<int> allRecords()const = 0;
     virtual QArray<int> queryByExample( const T& t, int sort ) = 0;
@@ -23,3 +21,3 @@ public:
     virtual bool remove( int uid ) = 0;
-    virtual void replace( const T& t ) = 0;
+    virtual bool replace( const T& t ) = 0;
 
@@ -28,2 +26,11 @@ public:
 
+template <class T>
+OPimAccessBackend<T>::OPimAccessBackend() {
+
+}
+template <class T>
+OPimAccessBackend<T>::~OPimAccessBackend() {
+
+}
+
 #endif
diff --git a/libopie/pim/opimaccesstemplate.h b/libopie/pim/opimaccesstemplate.h
index f2a241d..e0708e1 100644
--- a/libopie/pim/opimaccesstemplate.h
+++ b/libopie/pim/opimaccesstemplate.h
@@ -9,4 +9,6 @@
 
+#include "otemplatebase.h"
+
 template <class T = OPimRecord >
-class OPimAccessTemplate {
+class OPimAccessTemplate : public OTemplateBase<T> {
 public:
@@ -14,17 +16,7 @@ public:
     typedef OPimAccessBackend<T> BackEnd;
-    OPimAccessTemplate( BackEnd* end)
-        : m_backEnd( end ) {
-    }
-    ~OPimAccessTemplate() {
-        delete m_backEnd;
-    }
-    virtual void load() {
-        m_backEnd->load();
-    }
-    virtual void reload() {
-        m_backEnd->reload();
-    }
-    virtual void save() {
-        m_backEnd->save();
-    }
+    OPimAccessTemplate( BackEnd* end);
+    virtual ~OPimAccessTemplate();
+    virtual void load();
+    virtual void reload();
+    virtual void save();
 
@@ -34,45 +26,17 @@ public:
      */
-    virtual List allRecords()const {
-        QArray<int> ints = m_backEnd->allRecords();
-
-        List lis( ints,  this );
-        return lis;
-    }
-    virtual List queryByExample( const T& t, int sortOrder ) {
-        QArray<int> ints = m_backEnd->query( t, sortOrder );
-        List lis( ints, this );
-
-        return lis;
-    }
-    /* implement QCache here */
-    virtual T find( int uid ) {
-        T t = m_backEnd->find( uid );
-        return t;
-    }
+    virtual List allRecords()const;
+    virtual List queryByExample( const T& t, int sortOrder );
+    virtual T find( int uid );
 
     /* invalidate cache here */
-    virtual void clear() {
-        invalidateCache();
-        m_backEnd->clear();
-    }
-    virtual bool add( const T& t ) {
-        return m_backEnd->add( t );
-    }
+    virtual void clear() ;
+    virtual bool add( const T& t ) ;
 
     /* only the uid matters */
-    virtual bool remove( const T& t ) {
-        /* remove from cache */
-        return m_backEnd->remove( t.uid() );
-    }
-    virtual bool remove( int uid ) {
-        /* remove from cache */
-        return m_backEnd->remove(uid);
-    }
-    virtual bool replace( const T& t) {
-        return m_backEnd->replace( t );
-    }
+    virtual bool remove( const T& t );
+    virtual bool remove( int uid );
+    virtual bool replace( const T& t) ;
 protected:
-    void invalidateCache() {
-
-    }
+    void invalidateCache();
+    BackEnd* backEnd();
     BackEnd* m_backEnd;
@@ -81,2 +45,74 @@ protected:
 
+template <class T>
+OPimAccessTemplate<T>::OPimAccessTemplate( BackEnd* end )
+    : OTemplateBase<T>(), m_backEnd( end )
+{
+
+}
+template <class T>
+OPimAccessTemplate<T>::~OPimAccessTemplate() {
+    qWarning("~OPimAccessTemplate<T>");
+    delete m_backEnd;
+}
+template <class T>
+void OPimAccessTemplate<T>::load() {
+    m_backEnd->load();
+}
+template <class T>
+void OPimAccessTemplate<T>::reload() {
+    m_backEnd->reload();
+}
+template <class T>
+void OPimAccessTemplate<T>::save() {
+    m_backEnd->save();
+}
+template <class T>
+OPimAccessTemplate<T>::List OPimAccessTemplate<T>::allRecords()const {
+    QArray<int> ints = m_backEnd->allRecords();
+    List lis(ints, this );
+    return lis;
+}
+template <class T>
+OPimAccessTemplate<T>::List
+OPimAccessTemplate<T>::queryByExample( const T& t, int sortOrder ) {
+    QArray<int> ints = m_backEnd->queryByExample( t, sortOrder );
+
+    List lis(ints, this );
+    return lis;
+}
+template <class T>
+T OPimAccessTemplate<T>::find( int uid ) {
+    T t = m_backEnd->find( uid );
+    return t;
+}
+template <class T>
+void OPimAccessTemplate<T>::clear() {
+    invalidateCache();
+    m_backEnd->clear();
+}
+template <class T>
+bool OPimAccessTemplate<T>::add( const T& t ) {
+    return m_backEnd->add( t );
+}
+template <class T>
+bool OPimAccessTemplate<T>::remove( const T& t ) {
+    return m_backEnd->remove( t.uid() );
+}
+template <class T>
+bool OPimAccessTemplate<T>::remove( int uid ) {
+    return m_backEnd->remove( uid );
+}
+template <class T>
+bool OPimAccessTemplate<T>::replace( const T& t ) {
+    return m_backEnd->replace( t );
+}
+template <class T>
+void OPimAccessTemplate<T>::invalidateCache() {
+
+}
+template <class T>
+OPimAccessTemplate<T>::BackEnd* OPimAccessTemplate<T>::backEnd() {
+    return m_backEnd;
+}
+
 #endif
diff --git a/libopie/pim/opimrecord.cpp b/libopie/pim/opimrecord.cpp
index 88b6fde..60cdbf3 100644
--- a/libopie/pim/opimrecord.cpp
+++ b/libopie/pim/opimrecord.cpp
@@ -5,2 +5,5 @@
 
+Qtopia::UidGen OPimRecord::m_uidGen( Qtopia::UidGen::Qtopia );
+
+
 OPimRecord::OPimRecord( int uid )
@@ -127 +130,4 @@ void OPimRecord::setUid( int uid ) {
 };
+Qtopia::UidGen &OPimRecord::uidGen() {
+    return m_uidGen;
+}
diff --git a/libopie/pim/opimrecord.h b/libopie/pim/opimrecord.h
index 297ff12..e4d33d6 100644
--- a/libopie/pim/opimrecord.h
+++ b/libopie/pim/opimrecord.h
@@ -106,2 +106,3 @@ public:
 protected:
+    Qtopia::UidGen &uidGen();
     QString crossToString()const;
@@ -112,2 +113,3 @@ private:
     QMap<QString, QArray<int> > m_relations;
+    static Qtopia::UidGen m_uidGen;
 
diff --git a/libopie/pim/orecordlist.h b/libopie/pim/orecordlist.h
index 65c2169..a3955b0 100644
--- a/libopie/pim/orecordlist.h
+++ b/libopie/pim/orecordlist.h
@@ -4,4 +4,35 @@
 
-#include <opie/opimaccesstemplate.h>
-#include <opie/opimrecord.h>
+#include <qarray.h>
+
+#include "otemplatebase.h"
+#include "opimrecord.h"
+
+
+template <class T = OPimRecord>
+class ORecordListIterator {
+public:
+    typedef OTemplateBase<T> Base;
+    ORecordListIterator( const QArray<int>, const Base* );
+    ORecordListIterator();
+    ~ORecordListIterator();
+    ORecordListIterator( const ORecordListIterator& );
+    ORecordListIterator &operator=(const ORecordListIterator& );
+    T &operator*();
+    ORecordListIterator &operator++();
+    ORecordListIterator &operator--();
+
+    bool operator==( const ORecordListIterator& it );
+    bool operator!=( const ORecordListIterator& it );
+
+private:
+    QArray<int> m_uids;
+    int m_current;
+    const Base* m_temp;
+    bool m_end : 1;
+    T m_record;
+
+    /* d pointer for future versions */
+    class IteratorPrivate;
+    IteratorPrivate *d;
+};
 
@@ -10,32 +41,6 @@ class ORecordList {
 public:
-    class Iterator {
-        friend class ORecordList;
-    public:
-        Iterator();
-        ~Iterator();
-        Iterator(const Iterator& );
-        Iterator &operator=(const Iterator& );
-        T &operator*();
-        Iterator &operator++();
-        Iterator &operator--();
-
-        bool operator==( const Iterator& it );
-        bool operator!=( const Iterator& it );
-
-    private:
-        Iterator( const QArray<int>,
-                  OPimAccessTemplate<T>* );
-
-        QArray<int> m_uids;
-        int m_current;
-        OPimAccessTemplate* m_temp;
-        bool m_end : 1;
-        T m_record;
-
-        /* d pointer for future versions */
-        class IteratorPrivate;
-        IteratorPrivate *d;
-    };
+    typedef OTemplateBase<T> Base;
+    typedef ORecordListIterator<T> Iterator;
     ORecordList( const QArray<int>& ids,
-                 OPimAccessTemplate<T>* acc );
+                 const Base* );
     ~ORecordList();
@@ -49,3 +54,3 @@ private:
     QArray<int> m_ids;
-    OPimAccessTemplate<T>* m_acc;
+    const Base* m_acc;
 };
@@ -53,4 +58,4 @@ private:
 /* ok now implement it  */
-template <class T= OPimRecord>
-ORecordList<T>::Iterator::Iterator() {
+template <class T>
+ORecordListIterator<T>::ORecordListIterator() {
     m_current = 0;
@@ -59,4 +64,4 @@ ORecordList<T>::Iterator::Iterator() {
 }
-template <class T= OPimRecord>
-ORecordList<T>::Iterator::~Iterator() {
+template <class T>
+ORecordListIterator<T>::~ORecordListIterator() {
 /* nothing to delete */
@@ -64,4 +69,4 @@ ORecordList<T>::Iterator::~Iterator() {
 
-template <class T = OPimRecord>
-ORecordList<T>::Iterator::Iterator( const ORecordList<T>::Iterator& it) {
+template <class T>
+ORecordListIterator<T>::ORecordListIterator( const ORecordListIterator<T>& it) {
     m_uids = it.m_uids;
@@ -73,4 +78,4 @@ ORecordList<T>::Iterator::Iterator( const ORecordList<T>::Iterator& it) {
 
-template <class T = OPimRecord>
-ORecordList<T>::Iterator &ORecordList::Iterator::operator=( const ORecordList<T>::Iterator& it) {
+template <class T>
+ORecordListIterator<T> &ORecordListIterator<T>::operator=( const ORecordListIterator<T>& it) {
     m_uids = it.m_uids;
@@ -84,4 +89,4 @@ ORecordList<T>::Iterator &ORecordList::Iterator::operator=( const ORecordList<T>
 
-template <class T = OPimRecord>
-T &ORecordList<T>::Iterator::operator*() {
+template <class T>
+T &ORecordListIterator<T>::operator*() {
     if (!m_end )
@@ -89,3 +94,3 @@ T &ORecordList<T>::Iterator::operator*() {
     else
-        m_record = T;
+        m_record = T();
 
@@ -94,4 +99,4 @@ T &ORecordList<T>::Iterator::operator*() {
 
-template <class T = OPimRecord>
-ORecordList<T>::Iterator &ORecordList<T>::Iterator::operator++() {
+template <class T>
+ORecordListIterator<T> &ORecordListIterator<T>::operator++() {
     if (m_current < m_uids.count() ) {
@@ -104,4 +109,4 @@ ORecordList<T>::Iterator &ORecordList<T>::Iterator::operator++() {
 }
-template <class T = OPimRecord>
-ORecordList<T>::Iterator &ORecordList<T>::Iterator::operator--() {
+template <class T>
+ORecordListIterator<T> &ORecordListIterator<T>::operator--() {
     if ( m_current > 0 ) {
@@ -115,4 +120,4 @@ ORecordList<T>::Iterator &ORecordList<T>::Iterator::operator--() {
 
-template <class T = OPimRecord>
-bool ORecordList<T>::Iterator::operator==( const ORecordList<T>::Iterator& it ) {
+template <class T>
+bool ORecordListIterator<T>::operator==( const ORecordListIterator<T>& it ) {
 
@@ -127,9 +132,9 @@ bool ORecordList<T>::Iterator::operator==( const ORecordList<T>::Iterator& it )
 }
-template <class T = ORecordList>
-bool ORecordList<T>::Iterator::operator!=( const ORecordList<T>::Iterator it ) {
+template <class T>
+bool ORecordListIterator<T>::operator!=( const ORecordListIterator<T>& it ) {
     return !(*this == it );
 }
-template <class T = ORecordList>
-ORecordList<T>::Iterator::Iterator( const QArray<int> uids,
-                                    OPimAccessTemplate<T>* t )
+template <class T>
+ORecordListIterator<T>::ORecordListIterator( const QArray<int> uids,
+                                  const Base* t )
     : m_uids( uids ), m_current( 0 ),  m_temp( t ), m_end( false )
@@ -137,5 +142,6 @@ ORecordList<T>::Iterator::Iterator( const QArray<int> uids,
 }
-template <class T = ORecordList>
+
+template <class T>
 ORecordList<T>::ORecordList( const QArray<int>& ids,
-                             OPimAccessTemplate<T>* acc )
+                             const Base* acc )
     : m_ids( ids ), m_acc( acc )
@@ -143,3 +149,3 @@ ORecordList<T>::ORecordList( const QArray<int>& ids,
 }
-template <class T = ORecordList>
+template <class T>
 ORecordList<T>::~ORecordList() {
@@ -147,3 +153,3 @@ ORecordList<T>::~ORecordList() {
 }
-template <class T = ORecordList>
+template <class T>
 ORecordList<T>::Iterator ORecordList<T>::begin() {
@@ -152,3 +158,3 @@ ORecordList<T>::Iterator ORecordList<T>::begin() {
 }
-template <class T = ORecordList>
+template <class T>
 ORecordList<T>::Iterator ORecordList<T>::end() {
diff --git a/libopie/pim/otemplatebase.h b/libopie/pim/otemplatebase.h
new file mode 100644
index 0000000..41cc934
--- a/dev/null
+++ b/libopie/pim/otemplatebase.h
@@ -0,0 +1,18 @@
+#ifndef OPIE_TEMPLATE_BASE_H
+#define OPIE_TEMPLATE_BASE_H
+
+#include "opimrecord.h"
+
+template <class T = OPimRecord>
+class OTemplateBase {
+public:
+    OTemplateBase() {
+    };
+    virtual ~OTemplateBase() {
+    }
+    virtual T find( int uid ) = 0;
+
+};
+
+
+#endif