Ticket #7683: 7683.5.diff

django/db/models/base.py

@@ -527 +527 @@
         full_filename = self._get_FIELD_filename(field)
         if hasattr(raw_field, 'temporary_file_path'):
             # This file has a file path that we can move.
+            file_move_safe(raw_field.temporary_file_path(), full_filename)
             raw_field.close()
-            file_move_safe(raw_field.temporary_file_path(), full_filename)
         else:
             # This is a normal uploadedfile that we can stream.
             fp = open(full_filename, 'wb')

django/core/files/uploadedfile.py

@@ -201 +201 @@
     def read(self, *args):          return self._file.read(*args)
     def seek(self, offset):         return self._file.seek(offset)
     def write(self, s):             return self._file.write(s)
-    def close(self):                return self._file.close()
     def __iter__(self):             return iter(self._file)
     def readlines(self, size=None): return self._file.readlines(size)
     def xreadlines(self):           return self._file.xreadlines()
-
+    def close(self):
+        try:
+            return self._file.close()
+        except OSError, e:
+            if e.errno == 2:
+                # Means the file was moved or deleted before the tempfile could unlink it.
+                # Still sets self._file.close_called and calls self._file.file.close()
+                # before the exception
+                return
+            else: 
+                raise e
+        
 class InMemoryUploadedFile(UploadedFile):
     """
     A file uploaded into memory (i.e. stream-to-memory).