Kaydet (Commit) cd217de6 authored tarafından Tim Graham's avatar Tim Graham

Reverted "Fixed #26644 -- Allowed wrapping NamedTemporaryFile with File."

This reverts commit 1b407050 as it
introduces a regression in the test for refs #26772.
üst 625b8e92
...@@ -18,10 +18,6 @@ class File(FileProxyMixin): ...@@ -18,10 +18,6 @@ class File(FileProxyMixin):
self.file = file self.file = file
if name is None: if name is None:
name = getattr(file, 'name', None) name = getattr(file, 'name', None)
# Use only the basename from a file's name if it's an absolute path,
# e.g. from NamedTemporaryFile.
if isinstance(name, six.string_types) and os.path.isabs(name):
name = os.path.basename(name)
self.name = name self.name = name
if hasattr(file, 'mode'): if hasattr(file, 'mode'):
self.mode = file.mode self.mode = file.mode
......
...@@ -10,7 +10,6 @@ import threading ...@@ -10,7 +10,6 @@ import threading
import time import time
import unittest import unittest
from datetime import datetime, timedelta from datetime import datetime, timedelta
from tempfile import NamedTemporaryFile
from django.core.cache import cache from django.core.cache import cache
from django.core.exceptions import SuspiciousFileOperation, SuspiciousOperation from django.core.exceptions import SuspiciousFileOperation, SuspiciousOperation
...@@ -860,13 +859,6 @@ class FileFieldStorageTests(TestCase): ...@@ -860,13 +859,6 @@ class FileFieldStorageTests(TestCase):
with temp_storage.open('tests/stringio') as f: with temp_storage.open('tests/stringio') as f:
self.assertEqual(f.read(), b'content') self.assertEqual(f.read(), b'content')
def test_save_temporary_file(self):
storage = Storage()
with NamedTemporaryFile() as f:
f.write(b'content')
storage.normal = File(f)
storage.save() # no crash
# Tests for a race condition on file saving (#4948). # Tests for a race condition on file saving (#4948).
# This is written in such a way that it'll always pass on platforms # This is written in such a way that it'll always pass on platforms
......
...@@ -26,12 +26,6 @@ else: ...@@ -26,12 +26,6 @@ else:
class FileTests(unittest.TestCase): class FileTests(unittest.TestCase):
def test_file_truncates_namedtemporaryfile_name(self):
named_file = NamedTemporaryFile()
f = File(named_file)
self.assertEqual(f.name, os.path.basename(named_file.name))
def test_unicode_uploadedfile_name(self): def test_unicode_uploadedfile_name(self):
uf = UploadedFile(name='¿Cómo?', content_type='text') uf = UploadedFile(name='¿Cómo?', content_type='text')
self.assertIs(type(repr(uf)), str) self.assertIs(type(repr(uf)), str)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment