We can't File.exist?(UploadedFile).

When trying to check if a ActionDispatch::Http::UploadedFile we need to explicitly call #path on it to make it work.
Manfred · Jun 3, 2014 · 5645575 · 5645575
1 parent 7cc67c5
commit 5645575
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/attachment_san/test/helper.rb b/lib/attachment_san/test/helper.rb
@@ -69,6 +69,10 @@ def path #:nodoc:
         @tempfile.path
       end
 
+      def to_s
+        raise TypeError, "no implicit conversion of AttachmentSan::UploadHelpers::FakeUploadFile into String"
+      end
+
       alias local_path path
 
       def method_missing(method_name, *args, &block) #:nodoc:

diff --git a/lib/attachment_san/variant.rb b/lib/attachment_san/variant.rb
@@ -411,7 +411,7 @@ def filename
       # Returns true when there is an uploaded file on disk.
       #
       def has_upload?
-        @record.uploaded_file && File.exist?(@record.uploaded_file)
+        @record.uploaded_file && File.exist?(@record.uploaded_file.path)
       end
 
       ##