Remove unnecessary mv/unlink from Filesystem#flush_writes, complicates some failure modes

lib/paperclip/storage/filesystem.rb

@@ -38,12 +38,7 @@ module Paperclip
           file.close
           file.close
           FileUtils.mkdir_p(File.dirname(path(style_name)))
           FileUtils.mkdir_p(File.dirname(path(style_name)))
           log("saving #{path(style_name)}")
           log("saving #{path(style_name)}")
-          begin
-            FileUtils.mv(file.path, path(style_name))
-          rescue SystemCallError
-            FileUtils.cp(file.path, path(style_name))
-            FileUtils.rm(file.path)
-          end
+          FileUtils.cp(file.path, path(style_name))
           FileUtils.chmod(0666&~File.umask, path(style_name))
           FileUtils.chmod(0666&~File.umask, path(style_name))
         end
         end
 
 

test/integration_test.rb

@@ -81,8 +81,6 @@ class IntegrationTest < Test::Unit::TestCase
         original.close
         original.close
         tf.rewind
         tf.rewind
 
 
-        File.expects(:unlink).with(tf.instance_variable_get(:@tmpname))
-
         @d2.avatar.expects(:to_file).with(:original).returns(tf)
         @d2.avatar.expects(:to_file).with(:original).returns(tf)
 
 
         @d2.avatar.reprocess!
         @d2.avatar.reprocess!