delegate to the @tempfile instance variable

1 files changed, 5 insertions, 13 deletions

diff --git a/actionpack/lib/action_dispatch/http/upload.rb b/actionpack/lib/action_dispatch/http/upload.rb
index 49465d820e..bfbe7c5305 100644
--- a/actionpack/lib/action_dispatch/http/upload.rb
+++ b/actionpack/lib/action_dispatch/http/upload.rb
@@ -2,27 +2,19 @@ require 'active_support/core_ext/object/blank'
 
 module ActionDispatch
   module Http
-    class UploadedFile < Tempfile
+    class UploadedFile
       attr_accessor :original_filename, :content_type, :tempfile, :headers
 
       def initialize(hash)
         @original_filename = hash[:filename]
         @content_type      = hash[:type]
         @headers           = hash[:head]
-
-        # To the untrained eye, this may appear as insanity. Given the alternatives,
-        # such as busting the method cache on every request or breaking backwards
-        # compatibility with is_a?(Tempfile), this solution is the best available
-        # option.
-        #
-        # TODO: Deprecate is_a?(Tempfile) and define a real API for this parameter
-        tempfile = hash[:tempfile]
-        tempfile.instance_variables.each do |ivar|
-          instance_variable_set(ivar, tempfile.instance_variable_get(ivar))
-        end
+        @tempfile          = hash[:tempfile]
       end
 
-      alias local_path path
+      def method_missing(name, *args, &block)
+        @tempfile.send(name, *args, &block)
+      end
     end
 
     module Upload