First attempt at handling a local destination directory that already …

…exists.
dvjones89 · May 21, 2018 · 95f842b · 95f842b
1 parent 478b2be
commit 95f842b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/sheldon/brain.rb b/lib/sheldon/brain.rb
@@ -45,9 +45,11 @@ def recall(recall_cue)
     entry = memory.recall(recall_cue)
     destination_path = add_home(entry[:filepath])
     destination_dir = File.dirname(destination_path)
+    raise DestinationNotEmptyException, "#{destination_path} is not empty." if File.exist?(destination_path)
+
     FileUtils.mkdir_p(destination_dir) unless File.directory?(destination_dir)
     brain_path = brain_path_for_cue(recall_cue)
-    FileUtils.ln_s(get_content(brain_path), destination_path, force: true)
+    FileUtils.ln_s(get_content(brain_path), destination_path)
     return true
   end
 

diff --git a/lib/sheldon/exceptions.rb b/lib/sheldon/exceptions.rb
@@ -1,2 +1,5 @@
 class MissingDataDirectoryException < StandardError
 end
+
+class DestinationNotEmptyException < StandardError
+end