From 5c03ec2b2d0d6b10f7581b8b447cbcb1935e7e67 Mon Sep 17 00:00:00 2001 From: paule Date: Sat, 13 Jan 2007 09:00:15 +0000 Subject: Implement default sort functions for contact last name and birthday/anniversary. The latter is required for sorting birthdays/anniversaries correctly on the today screen. Fixes bug #1760. --- diff --git a/libopie2/opiepim/private/opimcontactsortvector.cpp b/libopie2/opiepim/private/opimcontactsortvector.cpp index 8c7d5ca..c8de4d1 100644 --- a/libopie2/opiepim/private/opimcontactsortvector.cpp +++ b/libopie2/opiepim/private/opimcontactsortvector.cpp @@ -64,6 +64,9 @@ int OPimContactSortVector::compareItems( const OPimContact& left, ret = testString( left.middleName(), right.middleName() ); soMiddleName = true; break; + case OPimContactAccess::SortLastName: + ret = testString( left.lastName(), right.lastName() ); + break; case OPimContactAccess::SortSuffix: ret = testString( left.suffix(), right.suffix() ); soSuffix = true; @@ -93,6 +96,17 @@ int OPimContactSortVector::compareItems( const OPimContact& left, ret = testString( left.gender(), right.gender() ); soGender = true; break; + case OPimContactAccess::SortBirthdayWithoutYear: + // This doesn't actually just sort by the date without year, + // it actually works out the days until the next occurrence, + // which is more useful since it will work correctly when + // crossing year boundaries. - Paul Eggleton Dec 2006 + ret = testDaysUntilNextDate( left.birthday(), right.birthday() ); + break; + case OPimContactAccess::SortAnniversaryWithoutYear: + // (as above) + ret = testDaysUntilNextDate( left.anniversary(), right.anniversary() ); + break; } /* twist to honor ascending/descending setting as QVector only sorts ascending*/ diff --git a/libopie2/opiepim/private/opimsortvector.h b/libopie2/opiepim/private/opimsortvector.h index 11a40ac..9684ca1 100644 --- a/libopie2/opiepim/private/opimsortvector.h +++ b/libopie2/opiepim/private/opimsortvector.h @@ -56,6 +56,8 @@ protected: int testTime( const QTime&, const QTime& )const; int testDateTime( const QDateTime& left, const QDateTime& right )const; + int testDaysUntilNextDate( const QDate& left, + const QDate& right )const; protected: bool sortAscending()const; int sortOrder()const; @@ -166,6 +168,28 @@ inline int OPimSortVector::testDateTime( const QDateTime& left, } +template +inline int OPimSortVector::testDaysUntilNextDate( const QDate& left, + const QDate& right )const { + int ret = 0; + if ( !left .isValid() ) ret++; + if ( !right.isValid() ) ret--; + + if ( left.isValid() && right.isValid() ){ + int currentYear = QDate::currentDate().year(); + QDate nextLeft( currentYear, left.month(), left.day() ); + if ( QDate::currentDate().daysTo(nextLeft) < 0 ) + nextLeft.setYMD( currentYear+1, left.month(), left.day() ); + QDate nextRight( currentYear, right.month(), right.day() ); + if ( QDate::currentDate().daysTo(nextRight) < 0 ) + nextRight.setYMD( currentYear+1, right.month(), right.day() ); + + ret += QDate::currentDate().daysTo(nextLeft) < QDate::currentDate().daysTo(nextRight) ? -1 : 1; + } + + return ret; +} + } } -- cgit v0.9.0.2