Handling the file command, added after the initial rails3 branch.

lib/paperclip/upfile.rb

@@ -16,7 +16,10 @@ module Paperclip
       when "js"                      then "application/js"
       when "js"                      then "application/js"
       when "csv", "xml", "css"       then "text/#{type}"
       when "csv", "xml", "css"       then "text/#{type}"
       else
       else
-        Paperclip.run("file", "--mime-type #{self.path}").split(':').last.strip rescue "application/x-#{type}"
+        # On BSDs, `file` doesn't give a result code of 1 if the file doesn't exist.
+        content_type = (Paperclip.run("file", "--mime-type #{self.path}").split(':').last.strip rescue "application/x-#{type}")
+        content_type = "application/x-#{type}" if content_type.match(/\(.*?\)/)
+        content_type
       end
       end
     end
     end
 
 

test/upfile_test.rb

@@ -23,14 +23,14 @@ class UpfileTest < Test::Unit::TestCase
 
 
         assert_equal content_type, file.content_type
         assert_equal content_type, file.content_type
       end
       end
+    end
+  end
 
 
-      should "return a content_type of text/plain on a real file whose content_type is determined with the file command" do
-        file = File.new(File.join(File.dirname(__FILE__), "..", "LICENSE"))
-        class << file
-          include Paperclip::Upfile
-        end
-        assert_equal 'text/plain', file.content_type
-      end
+  should "return a content_type of text/plain on a real file whose content_type is determined with the file command" do
+    file = File.new(File.join(File.dirname(__FILE__), "..", "LICENSE"))
+    class << file
+      include Paperclip::Upfile
     end
     end
+    assert_equal 'text/plain', file.content_type
   end
   end
 end
 end