Context Navigation

Ticket #13750: 13750.diff

File 13750.diff, 1.8 KB (added by Aksel, 10 years ago)
Documentation for behavior and updated regression test

diff --git a/docs/topics/files.txt b/docs/topics/files.txt
index 88f2bc9..80455a8 100644

                it has all the methods and attributes described below.
     file name used on disk cannot be relied on until after the model has been
     saved.
+.. note::
+    Accessing an image field (or other file-like field) on a model returns
+    a closed file reference.
 The ``File`` object
 ===================

diff --git a/tests/model_fields/models.py b/tests/model_fields/models.py
index 4208454..daf9ea8 100644

                class AbstractForeignFieldsModel(models.Model):
     class Meta:
         abstract = True
+class Profile(models.Model):
+    image = models.ImageField()

diff --git a/tests/model_fields/test_imagefield.py b/tests/model_fields/test_imagefield.py
index 7389573..a131a49 100644

                class ImageFieldTests(ImageFieldTestMixin, TestCase):
         loaded_p = pickle.loads(dump)
         self.assertEqual(p.mugshot, loaded_p.mugshot)
+    def test_image_field_io_closed_file(self):
+        """
+        Opening 'image' property as Image object of a model's ImageField will fail
+        with an 'I/O operation on closed file' error. Regression for #13750
+        Accessing the image after saving it should return a closed file reference.
+        """
+        from .models import Profile
+        profile = Profile()
+        profile.image = self.file1
+        profile.save()
+        self.assertTrue(profile.image.closed)
 @skipIf(Image is None, "Pillow is required to test ImageField")
 class ImageFieldTwoDimensionsTests(ImageFieldTestMixin, TestCase):