diff --git a/django/forms/models.py b/django/forms/models.py index 01bd912063..e6bbb98718 100644 --- a/django/forms/models.py +++ b/django/forms/models.py @@ -493,7 +493,7 @@ class BaseInlineFormSet(BaseModelFormSet): fk_attname = self.fk.get_attname() kwargs = {fk_attname: self.instance.pk} new_obj = self.model(**kwargs) - if fk_attname == self._pk_field.attname: + if fk_attname == self._pk_field.attname or self._pk_field.auto_created: exclude = [self._pk_field.name] else: exclude = [] diff --git a/tests/modeltests/model_formsets/models.py b/tests/modeltests/model_formsets/models.py index f11d4538ef..0503ea2a84 100644 --- a/tests/modeltests/model_formsets/models.py +++ b/tests/modeltests/model_formsets/models.py @@ -36,6 +36,12 @@ class BookWithCustomPK(models.Model): def __unicode__(self): return u'%s: %s' % (self.my_pk, self.title) +class AlternateBook(Book): + notes = models.CharField(max_length=100) + + def __unicode__(self): + return u'%s - %s' % (self.title, self.notes) + class AuthorMeeting(models.Model): name = models.CharField(max_length=100) authors = models.ManyToManyField(Author) @@ -520,6 +526,33 @@ True ... print book.title Les Fleurs du Mal +Test inline formsets where the inline-edited object uses multi-table inheritance, thus +has a non AutoField yet auto-created primary key. + +>>> AuthorBooksFormSet3 = inlineformset_factory(Author, AlternateBook, can_delete=False, extra=1) + +>>> formset = AuthorBooksFormSet3(instance=author) +>>> for form in formset.forms: +... print form.as_p() +
+
+ + +>>> data = { +... 'alternatebook_set-TOTAL_FORMS': '1', # the number of forms rendered +... 'alternatebook_set-INITIAL_FORMS': '0', # the number of forms with initial data +... 'alternatebook_set-0-title': 'Flowers of Evil', +... 'alternatebook_set-0-notes': 'English translation of Les Fleurs du Mal' +... } + +>>> formset = AuthorBooksFormSet3(data, instance=author) +>>> formset.is_valid() +True + +>>> formset.save() +[