[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 113645] New: "Use modification date if EXIF not present" does
From:       Charles Samuels <charles () kde ! org>
Date:       2005-09-30 19:44:59
Message-ID: 20050930214457.113645.charles () kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=113645         
           Summary: "Use modification date if EXIF not present" does nothing
           Product: kimdaba
           Version: unspecified
          Platform: unspecified
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: blackie blackie dk
        ReportedBy: charles kde org


Version:           SVN (using KDE 3.4.91 (beta1, >= 20050910), compiled sources)
Compiler:          gcc version 3.3.6 (Debian 1:3.3.6-10)
OS:                Linux (i686) release 2.6.11.11

So, I have images without EXIF data.  Oh well.  I try to load dates from the exif \
data, click on "Use modification date if EXIF not present" and they still have \
invalid dates.

So I applied this patch:


Index: imageinfo.cpp
===================================================================
--- imageinfo.cpp       (revision 465667)
+++ imageinfo.cpp       (working copy)
 @ -331,7 +331,7  @
         if ( (mode & EXIFMODE_FORCE) || Options::instance()->trustTimeStamps() ) {
             QDate date = exifInfo.date( &foundDateInExif );
             QTime time = exifInfo.time( &foundTimeInExif );
-            if ( date.isValid() && foundDateInExif ) {
+            if ( date.isValid() && ( foundDateInExif || (mode & EXIFMODE_FORCE_DATE) \
) ) {  if ( time.isValid() && foundTimeInExif )
                     _date = QDateTime( date, time );
                 else


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic