move dateless files to na subdir

[photosort.git] / src / misc.py

diff --git a/src/misc.py b/src/misc.py
index 6ca474932d7ba2955262032a5d7cbae7f45d2f9a..08baabb3113b4c1d08583f17f5ad1d0f24db6849 100644 (file)
--- a/src/misc.py
+++ b/src/misc.py
@@ -13,17 +13,11 @@ 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):
@@ -53,13 +47,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],