summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Kestenholz <mk@feinheit.ch>2020-07-23 01:38:31 +0200
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2020-07-24 11:06:57 +0200
commitc1f8d87bb0ac72b213bf31e672bd34c93bda6b18 (patch)
tree75cdd10eb1d7840202003eef2300dca2b79c2e18
parent41065cfed56d5408dd8f267b9e70089471a7f1be (diff)
Fixed #31812 -- Fixed FileField.model for fields defined in abstract models.
Regression in a93425a37f4defdb31d4ca96bb3bf6da21f0b5ce.
-rw-r--r--django/db/models/fields/files.py4
-rw-r--r--tests/model_fields/test_filefield.py21
2 files changed, 24 insertions, 1 deletions
diff --git a/django/db/models/fields/files.py b/django/db/models/fields/files.py
index b064202ee3..e10a5bb6d9 100644
--- a/django/db/models/fields/files.py
+++ b/django/db/models/fields/files.py
@@ -299,6 +299,10 @@ class FileField(Field):
file.save(file.name, file.file, save=False)
return file
+ def contribute_to_class(self, cls, name, **kwargs):
+ super().contribute_to_class(cls, name, **kwargs)
+ setattr(cls, self.attname, self.descriptor_class(self))
+
def generate_filename(self, instance, filename):
"""
Apply (if callable) or prepend (if a string) upload_to to the filename,
diff --git a/tests/model_fields/test_filefield.py b/tests/model_fields/test_filefield.py
index d4e70d6041..51e29f6d25 100644
--- a/tests/model_fields/test_filefield.py
+++ b/tests/model_fields/test_filefield.py
@@ -8,8 +8,9 @@ from pathlib import Path
from django.core.files import File, temp
from django.core.files.base import ContentFile
from django.core.files.uploadedfile import TemporaryUploadedFile
-from django.db import IntegrityError
+from django.db import IntegrityError, models
from django.test import TestCase, override_settings
+from django.test.utils import isolate_apps
from .models import Document
@@ -147,3 +148,21 @@ class FileFieldTests(TestCase):
self.assertEqual(document.myfile.field, loaded_myfile.field)
finally:
document.myfile.delete()
+
+ @isolate_apps('model_fields')
+ def test_abstract_filefield_model(self):
+ """
+ FileField.model returns the concrete model for fields defined in an
+ abstract model.
+ """
+ class AbstractMyDocument(models.Model):
+ myfile = models.FileField(upload_to='unused')
+
+ class Meta:
+ abstract = True
+
+ class MyDocument(AbstractMyDocument):
+ pass
+
+ document = MyDocument(myfile='test_file.py')
+ self.assertEqual(document.myfile.field.model, MyDocument)