Bugfix for calculating data diffs..

1 files changed, 39 insertions, 18 deletions

diff --git a/libopie2/opiepim/backend/ocontactaccessbackend_xml.h b/libopie2/opiepim/backend/ocontactaccessbackend_xml.h
index c6e6cbc..6a1c91d 100644
--- a/libopie2/opiepim/backend/ocontactaccessbackend_xml.h
+++ b/libopie2/opiepim/backend/ocontactaccessbackend_xml.h
@@ -19,2 +19,5 @@
  * $Log$
+ * Revision 1.11  2003/01/03 12:31:28  eilers
+ * Bugfix for calculating data diffs..
+ *
  * Revision 1.10  2003/01/02 14:27:12  eilers
@@ -237,3 +240,3 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                                 // Birthday and anniversary are special nonstring fields and should
-                                // be handled especially
+                                // be handled specially
                                 switch ( i ){
@@ -249,21 +252,39 @@ class OContactAccessBackend_XML : public OContactAccessBackend {
                                         if ( queryDate->isValid() ){
-                                                if ( settings & OContactAccess::DateYear ){
-                                                        if ( queryDate->year() != checkDate->year() )
-                                                                allcorrect = false;
-                                                } 
-                                                if ( settings & OContactAccess::DateMonth ){
-                                                        if ( queryDate->month() != checkDate->month() )
-                                                                allcorrect = false;
-                                                } 
-                                                if ( settings & OContactAccess::DateDay ){
-                                                        if ( queryDate->day() != checkDate->day() )
-                                                                allcorrect = false;
-                                                } 
-                                                if ( settings & OContactAccess::DateDiff ) {
-                                                        QDate current = QDate::currentDate();
-                                                        if ( current.daysTo( *queryDate ) > 0 ){
-                                                                if ( !( ( *checkDate >= current ) && 
-                                                                        ( *checkDate <= *queryDate ) ) )
+                                                if(  checkDate->isValid() ){
+                                                        if ( settings & OContactAccess::DateYear ){
+                                                                if ( queryDate->year() != checkDate->year() )
+                                                                        allcorrect = false;
+                                                        } 
+                                                        if ( settings & OContactAccess::DateMonth ){
+                                                                if ( queryDate->month() != checkDate->month() )
+                                                                        allcorrect = false;
+                                                        } 
+                                                        if ( settings & OContactAccess::DateDay ){
+                                                                if ( queryDate->day() != checkDate->day() )
                                                                         allcorrect = false;
+                                                        } 
+                                                        if ( settings & OContactAccess::DateDiff ) {
+                                                                QDate current = QDate::currentDate();
+                                                                // We have to equalize the year, otherwise
+                                                                // the search will fail..
+                                                                checkDate->setYMD( current.year(), checkDate->month(), checkDate->day() );
+                                                                if ( *checkDate < current )
+                                                                        checkDate->setYMD( current.year()+1, 
+                                                                                           checkDate->month(), 
+                                                                                           checkDate->day() );
+                                                                // qWarning("Checking if %s is between %s and %s ! ", 
+                                                                        // checkDate->toString().latin1(),
+                                                                        // current.toString().latin1(), 
+                                                                        // queryDate->toString().latin1() );
+                                                                if ( current.daysTo( *queryDate ) > 0 ){
+                                                                        if ( !( ( *checkDate >= current ) && 
+                                                                                ( *checkDate <= *queryDate ) ) ){
+                                                                                allcorrect = false;
+                                                                                //qWarning (" Nope!..");
+                                                                        }
+                                                                }
                                                         }
+                                                } else{
+                                                        // checkDate is invalid. Therfore this entry is always rejected
+                                                        allcorrect = false;
                                                 }