From 189ee9c3b26de3dcbd2bd1931844e7173151ece1 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Sun, 19 Feb 2012 17:39:52 +0000 Subject: [PATCH] Clear the fingerprint when an attachment is removed --- lib/paperclip/attachment.rb | 1 + test/attachment_test.rb | 6 ++++++ 2 files changed, 7 insertions(+) diff --git a/lib/paperclip/attachment.rb b/lib/paperclip/attachment.rb index 41f848aa6..40e98d091 100644 --- a/lib/paperclip/attachment.rb +++ b/lib/paperclip/attachment.rb @@ -462,6 +462,7 @@ def queue_existing_for_delete #:nodoc: instance_write(:file_name, nil) instance_write(:content_type, nil) instance_write(:file_size, nil) + instance_write(:fingerprint, nil) instance_write(:updated_at, nil) end diff --git a/test/attachment_test.rb b/test/attachment_test.rb index 5ff83e7dd..cf3243126 100644 --- a/test/attachment_test.rb +++ b/test/attachment_test.rb @@ -925,6 +925,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.assign nil @attachment.save @@ -935,6 +936,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.clear @attachment.save @@ -945,6 +947,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.destroy @existing_names.each{|f| assert ! File.exists?(f) } @@ -959,6 +962,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.assign nil @attachment.save @@ -969,6 +973,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.clear @attachment.save @@ -979,6 +984,7 @@ def do_after_all; end @attachment.expects(:instance_write).with(:file_name, nil) @attachment.expects(:instance_write).with(:content_type, nil) @attachment.expects(:instance_write).with(:file_size, nil) + @attachment.expects(:instance_write).with(:fingerprint, nil) @attachment.expects(:instance_write).with(:updated_at, nil) @attachment.destroy @existing_names.each{|f| assert File.exists?(f) }