Some minor interface updates

7 files changed, 37 insertions, 73 deletions

diff --git a/libopie/pim/DESIGN b/libopie/pim/DESIGN
index 4def7b9..bd92b1b 100644
--- a/libopie/pim/DESIGN
+++ b/libopie/pim/DESIGN
@@ -52,8 +52,2 @@ regards Holger 'zecke' Freyther
 
-Comment by Stefan Eilers:
-
-The opimaccesstemplate defines "SortOrder":
-I think sortorder is the wrong name for the meaning of it and
-it should be defined by the childs of opimaccesstemplate (every 
-implementation may use different values for this..)
 
diff --git a/libopie/pim/ocontactaccess.cpp b/libopie/pim/ocontactaccess.cpp
index b5f358b..e8c0a45 100644
--- a/libopie/pim/ocontactaccess.cpp
+++ b/libopie/pim/ocontactaccess.cpp
@@ -23,2 +23,5 @@
  * $Log$
+ * Revision 1.4  2002/10/14 16:21:54  eilers
+ * Some minor interface updates
+ *
  * Revision 1.3  2002/10/07 17:34:24  eilers
@@ -68,3 +71,2 @@ OContactAccess::OContactAccess ( const QString appname, const QString ,
         if( end == 0 ) {
-                // __asm__("int3");
                 qWarning ("Using BackendFactory !");
diff --git a/libopie/pim/ocontactaccess.h b/libopie/pim/ocontactaccess.h
index 54f7f07..adc66cf 100644
--- a/libopie/pim/ocontactaccess.h
+++ b/libopie/pim/ocontactaccess.h
@@ -19,2 +19,5 @@
  * $Log$
+ * Revision 1.2  2002/10/14 16:21:54  eilers
+ * Some minor interface updates
+ *
  * Revision 1.1  2002/09/27 17:11:44  eilers
@@ -49,34 +52,2 @@ class OContactAccess: public QObject, public OPimAccessTemplate<OContact>
 
-            
-            /*    class Iterator{
-            friend OContactAccess;
-    public:
-            Iterator();
-            Iterator ( const Iterator& copy );
-
-            bool operator== ( const Iterator& it );
-            bool operator!= ( const Iterator& it );
-            Iterator& operator= ( const  Iterator& it );
-            Iterator& operator++ ();      // prefix
-            Iterator operator++ ( int );  // postfix
-            Iterator& operator-- ();      // prefix
-            Iterator operator-- ( int );  // postfix
-            Contact operator*() const;
-            Contact operator->() const;
-
-            Iterator begin();
-            Iterator end();
-
-            uint count() const;
-
-    private:
-            QValueList<int> m_uids;
-            int m_cur_position;
-            bool m_end_reached;
-            OContactAccess *m_db;
-            
-    };
-
-            */
-
  public:
@@ -93,3 +64,3 @@ class OContactAccess: public QObject, public OPimAccessTemplate<OContact>
          * to react itself, it should be disabled by setting it to <b>false</b>
-         * @see OContactBackend
+         * @see OContactAccessBackend
          */
@@ -103,9 +74,9 @@ class OContactAccess: public QObject, public OPimAccessTemplate<OContact>
          * @see queryByExample()
-         * - why not enum - zecke? 
-         *   -> Had some implementation problems .. Will use enum soon ! .. (se)
          */
-        static const int query_WildCards  = 0x0001;
-        static const int query_IgnoreCase = 0x0002;
-        static const int query_RegExp     = 0x0004;
-        static const int query_ExactMatch = 0x0008;
+        enum QuerySettings {
+                WildCards  = 0x0001,
+                IgnoreCase = 0x0002,
+                RegExp     = 0x0004,
+                ExactMatch = 0x0008,
+        };
 
diff --git a/libopie/pim/ocontactaccessbackend_xml.h b/libopie/pim/ocontactaccessbackend_xml.h
index 97ef40f..50ea329 100644
--- a/libopie/pim/ocontactaccessbackend_xml.h
+++ b/libopie/pim/ocontactaccessbackend_xml.h
@@ -19,2 +19,5 @@
  * $Log$
+ * Revision 1.3  2002/10/14 16:21:54  eilers
+ * Some minor interface updates
+ *
  * Revision 1.2  2002/10/07 17:34:24  eilers
@@ -201,6 +204,6 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                                 if ( !query.field(i).isEmpty() ){
-                                        switch ( settings & ~OContactAccess::query_IgnoreCase ){
-                                        case OContactAccess::query_RegExp:{
+                                        switch ( settings & ~OContactAccess::IgnoreCase ){
+                                        case OContactAccess::RegExp:{
                                                 QRegExp expr ( query.field(i),
-                                                               !(settings & OContactAccess::query_IgnoreCase),
+                                                               !(settings & OContactAccess::IgnoreCase),
                                                                false );
@@ -210,5 +213,5 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                                                 break;
-                                        case OContactAccess::query_WildCards:{
+                                        case OContactAccess::WildCards:{
                                                 QRegExp expr ( query.field(i),
-                                                               !(settings & OContactAccess::query_IgnoreCase),
+                                                               !(settings & OContactAccess::IgnoreCase),
                                                                true );
@@ -218,4 +221,4 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                                                 break;
-                                        case OContactAccess::query_ExactMatch:{
-                                                if (settings & OContactAccess::query_IgnoreCase){
+                                        case OContactAccess::ExactMatch:{
+                                                if (settings & OContactAccess::IgnoreCase){
                                                         if ( query.field(i).upper() !=
@@ -245,6 +248,6 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                 {
-                        return ( OContactAccess::query_WildCards
-                                 & OContactAccess::query_IgnoreCase
-                                 & OContactAccess::query_RegExp
-                                 & OContactAccess::query_ExactMatch );
+                        return ( OContactAccess::WildCards
+                                 & OContactAccess::IgnoreCase
+                                 & OContactAccess::RegExp
+                                 & OContactAccess::ExactMatch );
                 }
@@ -253,3 +256,3 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                 {
-                        /* OContactAccess::query_IgnoreCase may be added with one
+                        /* OContactAccess::IgnoreCase may be added with one
                          * of the other settings, but never used alone.
@@ -257,8 +260,8 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                          */
-                        switch ( querySettings & ~OContactAccess::query_IgnoreCase ){
-                        case OContactAccess::query_RegExp:
+                        switch ( querySettings & ~OContactAccess::IgnoreCase ){
+                        case OContactAccess::RegExp:
                                 return ( true );
-                        case OContactAccess::query_WildCards:
+                        case OContactAccess::WildCards:
                                 return ( true );
-                        case OContactAccess::query_ExactMatch:
+                        case OContactAccess::ExactMatch:
                                 return ( true );
diff --git a/libopie/pim/opimaccesstemplate.h b/libopie/pim/opimaccesstemplate.h
index a0d8f63..3e1f393 100644
--- a/libopie/pim/opimaccesstemplate.h
+++ b/libopie/pim/opimaccesstemplate.h
@@ -29,9 +29,2 @@ public:
     /**
-     * our sort order
-     * should be safe explaining
-     */
-    enum SortOrder { WildCards = 0, IgnoreCase = 1,
-                     RegExp = 2, ExactMatch = 4 };
-
-    /**
      * c'tor BackEnd
@@ -68,5 +61,6 @@ public:
     /**
-     * queryByExample
+     * queryByExample)
+     * @see otodoaccess, ocontactaccess
      */
-    virtual List queryByExample( const T& t, int sortOrder );
+    virtual List queryByExample( const T& t, int querySettings );
 
diff --git a/libopie/pim/otodoaccessxml.cpp b/libopie/pim/otodoaccessxml.cpp
index f3b0783..3b4cab2 100644
--- a/libopie/pim/otodoaccessxml.cpp
+++ b/libopie/pim/otodoaccessxml.cpp
@@ -189,3 +189,3 @@ QArray<int> OTodoAccessXML::allRecords()const {
 }
-QArray<int> OTodoAccessXML::queryByExample( const OTodo&, int sort ) {
+QArray<int> OTodoAccessXML::queryByExample( const OTodo&, int ) {
     QArray<int> ids(0);
diff --git a/libopie/pim/otodoaccessxml.h b/libopie/pim/otodoaccessxml.h
index 1e7e371..dc41c32 100644
--- a/libopie/pim/otodoaccessxml.h
+++ b/libopie/pim/otodoaccessxml.h
@@ -26,3 +26,3 @@ public:
     QArray<int> allRecords()const;
-    QArray<int> queryByExample( const OTodo&, int sort );
+    QArray<int> queryByExample( const OTodo&, int querysettings );
     OTodo find( int uid )const;