Ticket #14797: 14797.diff

tests/regressiontests/generic_views/edit.py

@@ -7 +7 @@
 from regressiontests.generic_views.models import Artist, Author
 from regressiontests.generic_views import views
 
+class ModelFormMixinTests(TestCase):
+    def test_get_form(self):
+        form_class = views.AuthorGetQuerySetFormView().get_form_class()
+        self.assertEqual(form_class.Meta.model, Author)
 
 class CreateViewTests(TestCase):
     urls = 'regressiontests.generic_views.urls'

tests/regressiontests/generic_views/tests.py

@@ -1 +1 @@
 from regressiontests.generic_views.base import ViewTest, TemplateViewTest, RedirectViewTest
 from regressiontests.generic_views.dates import ArchiveIndexViewTests, YearArchiveViewTests, MonthArchiveViewTests, WeekArchiveViewTests, DayArchiveViewTests, DateDetailViewTests
 from regressiontests.generic_views.detail import DetailViewTest
-from regressiontests.generic_views.edit import CreateViewTests, UpdateViewTests, DeleteViewTests
+from regressiontests.generic_views.edit import ModelFormMixinTests, CreateViewTests, UpdateViewTests, DeleteViewTests
 from regressiontests.generic_views.list import ListViewTests

tests/regressiontests/generic_views/views.py

@@ -143 +143 @@
 
 class BookDetail(BookConfig, generic.DateDetailView):
     pass
+
+class AuthorGetQuerySetFormView(generic.edit.ModelFormMixin):
+    def get_queryset(self):
+        return Author.objects.all()

django/views/generic/edit.py

@@ -72 +72 @@
         if self.form_class:
             return self.form_class
         else:
-            if self.model is None:
-                model = self.queryset.model
-            else:
-                model = self.model
+            model = self.get_queryset().model
             return model_forms.modelform_factory(model)
 
     def get_form(self, form_class):