X-Git-Url: http://git.treefish.org/~alex/photosort.git/blobdiff_plain/d615316bb498b3a60dee4ad4ee3bd9247a82182e..797a8228d05a1080d6928721d96a53f8fc91ec06:/src/misc.py

diff --git a/src/misc.py b/src/misc.py
index 6ca4749..acd5c94 100644
--- a/src/misc.py
+++ b/src/misc.py
@@ -1,4 +1,5 @@
 import datetime
+import logging
 import mimetypes
 import os
 import PIL.Image
@@ -13,27 +14,23 @@ def walk_media_files(dir_path):
             if is_media_file(file_path):
                 yield (f, file_path)
 
-def extract_timestamp(file_path, use_meta=False):
-    time = None
-    if use_meta:
-        if is_media_file(file_path, types=['image']):
-            time = _extract_image_timestamp(file_path)
-        elif is_media_file(file_path, types=['video']):
-            time = _extract_video_timestamp(file_path)
-    if time:
-        return time
-    else:
-        return os.path.getmtime(file_path)
+def extract_meta_time(file_path):
+    if is_media_file(file_path, types=['image']):
+        return _extract_image_timestamp(file_path)
+    elif is_media_file(file_path, types=['video']):
+        return _extract_video_timestamp(file_path)
 
-def find_file(dir_path, file_name, file_size, exclude_dir):
-    for root, dirs, files in os.walk(dir_path):
+def find_file(base_dir, name, size, meta_time, exclude_dir):
+    for root, dirs, files in os.walk(base_dir):
         if root.startswith(exclude_dir):
             continue
-        for f in files:
-            if f == file_name:
-                full_path = os.path.join(root, f)
-                if os.path.getsize(full_path) == file_size:
-                    return root
+        for other_name in files:
+            if other_name == name:
+                full_path = os.path.join(root, other_name)
+                if os.path.getsize(full_path) == size:
+                    other_meta_time = extract_meta_time(full_path)
+                    if meta_time == other_meta_time:
+                        return root
     return None
 
 def import_file(src_file_path, dst_file_path):
@@ -53,13 +50,17 @@ def is_media_file(file_path, types=['image', 'video']):
     return True
 
 def _extract_image_timestamp(file_path):
-    with PIL.Image.open(file_path) as image:
-        exif = image._getexif()
-        if exif and 36867 in exif:
-            return datetime.datetime\
-                           .strptime(exif[36867], '%Y:%m:%d %H:%M:%S')\
-                           .timestamp()
-    return None
+    time = None
+    try:
+        with PIL.Image.open(file_path) as image:
+            exif = image._getexif()
+            if exif and 36867 in exif:
+                time = datetime.datetime\
+                               .strptime(exif[36867], '%Y:%m:%d %H:%M:%S')\
+                               .timestamp()
+    except Exception as e:
+        logging.warn("Error extracting exif for %s: %s", file_path, str(e))
+    return time
 
 def _extract_video_timestamp(file_path):
     p = subprocess.run(['ffmpeg', '-i', file_path],