Ticket #7683: 7683.3.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:
+            # 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
+        
 class InMemoryUploadedFile(UploadedFile):
     """
     A file uploaded into memory (i.e. stream-to-memory).