Author: jacob
Date: 2009-04-18 15:17:17 -0500 (Sat, 18 Apr 2009)
New Revision: 10587
Modified:
django/branches/releases/1.0.X/AUTHORS
django/branches/releases/1.0.X/django/contrib/contenttypes/generic.py
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/models.py
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/tests.py
Log:
[1.0.X] Fixed #9122: generic inline formsets now respect exclude and max_num.
Thanks, Alex Robbins. Backport of [10586] from trunk.
Modified: django/branches/releases/1.0.X/AUTHORS
===================================================================
--- django/branches/releases/1.0.X/AUTHORS 2009-04-18 20:12:53 UTC (rev
10586)
+++ django/branches/releases/1.0.X/AUTHORS 2009-04-18 20:17:17 UTC (rev
10587)
@@ -344,6 +344,7 @@
ricardojbarr...@gmail.com
Mike Richardson
Matt Riggott
+ Alex Robbins <alexander.j.robb...@gmail.com>
Henrique Romano <onai...@gmail.com>
Armin Ronacher
Daniel Roseman <http://roseman.org.uk/>
Modified: django/branches/releases/1.0.X/django/contrib/contenttypes/generic.py
===================================================================
--- django/branches/releases/1.0.X/django/contrib/contenttypes/generic.py
2009-04-18 20:12:53 UTC (rev 10586)
+++ django/branches/releases/1.0.X/django/contrib/contenttypes/generic.py
2009-04-18 20:17:17 UTC (rev 10587)
@@ -389,6 +389,8 @@
"can_delete": True,
"can_order": False,
"fields": fields,
+ "max_num": self.max_num,
+ "exclude": self.exclude
}
return generic_inlineformset_factory(self.model, **defaults)
Modified:
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/models.py
===================================================================
---
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/models.py
2009-04-18 20:12:53 UTC (rev 10586)
+++
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/models.py
2009-04-18 20:17:17 UTC (rev 10587)
@@ -20,11 +20,55 @@
class MediaInline(generic.GenericTabularInline):
model = Media
- extra = 1
class EpisodeAdmin(admin.ModelAdmin):
inlines = [
MediaInline,
]
+admin.site.register(Episode, EpisodeAdmin)
-admin.site.register(Episode, EpisodeAdmin)
+#
+# These models let us test the different GenericInline settings at
+# different urls in the admin site.
+#
+
+#
+# Generic inline with extra = 0
+#
+
+class EpisodeExtra(Episode):
+ pass
+
+class MediaExtraInline(generic.GenericTabularInline):
+ model = Media
+ extra = 0
+
+admin.site.register(EpisodeExtra, inlines=[MediaExtraInline])
+
+#
+# Generic inline with extra and max_num
+#
+
+class EpisodeMaxNum(Episode):
+ pass
+
+class MediaMaxNumInline(generic.GenericTabularInline):
+ model = Media
+ extra = 5
+ max_num = 2
+
+admin.site.register(EpisodeMaxNum, inlines=[MediaMaxNumInline])
+
+#
+# Generic inline with exclude
+#
+
+class EpisodeExclude(Episode):
+ pass
+
+class MediaExcludeInline(generic.GenericTabularInline):
+ model = Media
+ exclude = ['url']
+
+admin.site.register(EpisodeExclude, inlines=[MediaExcludeInline])
+
Modified:
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/tests.py
===================================================================
---
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/tests.py
2009-04-18 20:12:53 UTC (rev 10586)
+++
django/branches/releases/1.0.X/tests/regressiontests/generic_inline_admin/tests.py
2009-04-18 20:17:17 UTC (rev 10587)
@@ -5,7 +5,7 @@
from django.contrib.contenttypes.generic import generic_inlineformset_factory
# local test models
-from models import Episode, Media
+from models import Episode, EpisodeExtra, EpisodeMaxNum, EpisodeExclude, Media
class GenericAdminViewTest(TestCase):
fixtures = ['users.xml']
@@ -77,3 +77,62 @@
# Regression test for #10522.
inline_formset = generic_inlineformset_factory(Media,
exclude=('url',))
+
+class GenericInlineAdminParametersTest(TestCase):
+ fixtures = ['users.xml']
+
+ def setUp(self):
+ self.client.login(username='super', password='secret')
+
+ # Can't load content via a fixture (since the GenericForeignKey
+ # relies on content type IDs, which will vary depending on what
+ # other tests have been run), thus we do it here.
+ test_classes = [
+ Episode,
+ EpisodeExtra,
+ EpisodeMaxNum,
+ EpisodeExclude,
+ ]
+ for klass in test_classes:
+ e = klass.objects.create(name='This Week in Django')
+ m = Media(content_object=e, url='http://example.com/podcast.mp3')
+ m.save()
+
+ def tearDown(self):
+ self.client.logout()
+
+ def testNoParam(self):
+ """
+ With one initial form, extra (default) at 3, there should be 4 forms.
+ """
+ response =
self.client.get('/generic_inline_admin/admin/generic_inline_admin/episode/1/')
+ formset = response.context[-1]['inline_admin_formsets'][0].formset
+ self.assertEqual(formset._total_form_count, 4)
+ self.assertEqual(formset._initial_form_count, 1)
+
+ def testExtraParam(self):
+ """
+ With extra=0, there should be one form.
+ """
+ response =
self.client.get('/generic_inline_admin/admin/generic_inline_admin/episodeextra/2/')
+ formset = response.context[-1]['inline_admin_formsets'][0].formset
+ self.assertEqual(formset._total_form_count, 1)
+ self.assertEqual(formset._initial_form_count, 1)
+
+ def testMaxNumParam(self):
+ """
+ With extra=5 and max_num=2, there should be only 2 forms.
+ """
+ inline_form_data = '<input type="hidden"
name="generic_inline_admin-media-content_type-object_id-TOTAL_FORMS" value="2"
id="id_generic_inline_admin-media-content_type-object_id-TOTAL_FORMS" /><input
type="hidden"
name="generic_inline_admin-media-content_type-object_id-INITIAL_FORMS"
value="1"
id="id_generic_inline_admin-media-content_type-object_id-INITIAL_FORMS" />'
+ response =
self.client.get('/generic_inline_admin/admin/generic_inline_admin/episodemaxnum/3/')
+ formset = response.context[-1]['inline_admin_formsets'][0].formset
+ self.assertEqual(formset._total_form_count, 2)
+ self.assertEqual(formset._initial_form_count, 1)
+
+ def testExcludeParam(self):
+ """
+ Generic inline formsets should respect include.
+ """
+ response =
self.client.get('/generic_inline_admin/admin/generic_inline_admin/episodeexclude/4/')
+ formset = response.context[-1]['inline_admin_formsets'][0].formset
+ self.failIf('url' in formset.forms[0], 'The formset has excluded "url"
field.')
\ No newline at end of file
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
