[KPhotoAlbum] Searching on dates

Robert Krawitz rlk at alum.mit.edu
Sun Oct 28 03:16:15 GMT 2018


One thing that has long annoyed me is that the end date of a search is
not inclusive.  So for example if you search with the same start and
end date, you don't get anything at all.

The fix, of course, is trivial; any objections if I push it?

diff --git a/AnnotationDialog/Dialog.cpp b/AnnotationDialog/Dialog.cpp
index e98e989d..b72c52bc 100644
--- a/AnnotationDialog/Dialog.cpp
+++ b/AnnotationDialog/Dialog.cpp
@@ -780,7 +780,7 @@ DB::ImageSearchInfo AnnotationDialog::Dialog::search( DB::ImageSearchInfo* searc
     int ok = exec();
     if ( ok == QDialog::Accepted )  {
         const QDateTime start = m_startDate->date().isNull() ? QDateTime() : QDateTime(m_startDate->date());
-        const QDateTime end = m_endDate->date().isNull() ? QDateTime() : QDateTime( m_endDate->date() );
+        const QDateTime end = m_endDate->date().isNull() ? QDateTime() : QDateTime( m_endDate->date() ).addDays(1).addMSecs(-1);
         m_oldSearch = DB::ImageSearchInfo( DB::ImageDate( start, end ),
                       m_imageLabel->text(), m_description->toPlainText(),
                       m_imageFilePattern->text());

(Incidentally, I note that searching on date is very slow...it will be
interesting to dig into that.)

-- 
Robert Krawitz                                     <rlk at alum.mit.edu>

***  MIT Engineers   A Proud Tradition   http://mitathletics.com  ***
Member of the League for Programming Freedom  --  http://ProgFree.org
Project lead for Gutenprint   --    http://gimp-print.sourceforge.net

"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton



More information about the Kphotoalbum mailing list