From 9be5ab13fc69eb9aa071ae9d6423df8d599b8414 Mon Sep 17 00:00:00 2001
From: Jaisen Mathai <jaisen@jmathai.com>
Date: Wed, 12 Apr 2017 23:42:44 -0700
Subject: [PATCH] Skip copying file if source is identical to destination. #210
 (#211)

This prevents a zero byte file from being written
---
 elodie/filesystem.py            |  6 ++++++
 elodie/tests/filesystem_test.py | 21 +++++++++++++++++++++
 2 files changed, 27 insertions(+)

diff --git a/elodie/filesystem.py b/elodie/filesystem.py
index 49576b9..fddaa44 100644
--- a/elodie/filesystem.py
+++ b/elodie/filesystem.py
@@ -369,6 +369,12 @@ class FileSystem(object):
                     checksum_file
                 ))
 
+        # If source and destination are identical then
+        #  we should not write the file. gh-210
+        if(_file == dest_path):
+            print('Final source and destination path should not be identical')
+            return
+
         self.create_directory(dest_directory)
 
         if(move is True):
diff --git a/elodie/tests/filesystem_test.py b/elodie/tests/filesystem_test.py
index f6589b2..8c871d1 100644
--- a/elodie/tests/filesystem_test.py
+++ b/elodie/tests/filesystem_test.py
@@ -619,6 +619,27 @@ full_path=%year/%month/%day
     shutil.rmtree(folder)
     shutil.rmtree(os.path.dirname(os.path.dirname(destination)))
 
+def test_process_existing_file_without_changes():
+    # gh-210
+    filesystem = FileSystem()
+    temporary_folder, folder = helper.create_working_folder()
+
+    origin = os.path.join(folder,'plain.jpg')
+    shutil.copyfile(helper.get_file('plain.jpg'), origin)
+
+    media = Photo(origin)
+    destination = filesystem.process_file(origin, temporary_folder, media, allowDuplicate=True)
+
+    assert helper.path_tz_fix(os.path.join('2015-12-Dec', 'Unknown Location', '2015-12-05_00-59-26-plain.jpg')) in destination, destination
+
+    media_second = Photo(destination)
+    destination_second = filesystem.process_file(destination, temporary_folder, media_second, allowDuplicate=True)
+
+    assert destination_second is None, destination_second
+
+    shutil.rmtree(folder)
+    shutil.rmtree(os.path.dirname(os.path.dirname(destination)))
+
 def test_set_utime_with_exif_date():
     filesystem = FileSystem()
     temporary_folder, folder = helper.create_working_folder()