Changeset 4772 in josm

Timestamp:

Jan 8, 2012 5:14:32 PM (18 months ago)

Author:

bastiK

Message:

fixed #7228 - HTC Sensation uses / as date separator, date parsed wrong when geotagging

File:

• trunk/src/org/openstreetmap/josm/tools/ExifReader.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/ExifReader.java

@@ -22 +22 @@
 
     @SuppressWarnings("unchecked") public static Date readTime(File filename) throws ParseException {
-        Date date = null;
         try {
             Metadata metadata = JpegMetadataReader.readMetadata(filename);
+            String dateStr = null;
+            OUTER:
             for (Iterator<Directory> dirIt = metadata.getDirectoryIterator(); dirIt.hasNext();) {
                 for (Iterator<Tag> tagIt = dirIt.next().getTagIterator(); tagIt.hasNext();) {
                     Tag tag = tagIt.next();
-                    if (tag.getTagType() == 0x9003)
-                        return DateParser.parse(tag.getDescription());
-                    if (tag.getTagType() == 0x132 || tag.getTagType() == 0x9004)
-                        date = DateParser.parse(tag.getDescription());
+                    if (tag.getTagType() == ExifDirectory.TAG_DATETIME_ORIGINAL /* 0x9003 */) {
+                        dateStr = tag.getDescription();
+                        break OUTER; // prefer this tag
+                    }
+                    if (tag.getTagType() == ExifDirectory.TAG_DATETIME /* 0x0132 */ ||
+                        tag.getTagType() == ExifDirectory.TAG_DATETIME_DIGITIZED /* 0x9004 */) {
+                        dateStr = tag.getDescription();
+                    }
                 }
             }
+            dateStr = dateStr.replace('/', ':'); // workaround for HTC Sensation bug, see #7228
+            return DateParser.parse(dateStr);
         } catch (ParseException e) {
             throw e;
@@ -39 +46 @@
             e.printStackTrace();
         }
-        return date;
+        return null;
     }