Creation of models with ForeignObject fails with bulk_create() function

[rogerhu]

The bulk_create() function uses local_fields instead of concrete_fields.

[rhu@…]

--- a/django/db/models/query.py
+++ b/django/db/models/query.py
@@ -389,7 +389,7 @@ class QuerySet(object):
             return objs
         self._for_write = True
         connection = connections[self.db]
-        fields = self.model._meta.local_fields
+        fields = self.model._meta.concrete_fields
         with transaction.commit_on_success_unless_managed(using=self.db):
             if (connection.features.can_combine_inserts_with_and_without_auto_increment_pk
                     and self.model._meta.has_auto_field):
diff --git a/tests/foreign_object/tests.py b/tests/foreign_object/tests.py
index 66f57b6..b1502c3 100644
--- a/tests/foreign_object/tests.py
+++ b/tests/foreign_object/tests.py
@@ -4,7 +4,7 @@ from operator import attrgetter
 from .models import (
     Country, Person, Group, Membership, Friendship, Article,
     ArticleTranslation, ArticleTag, ArticleIdea, NewsArticle)
-from django.test import TestCase
+from django.test import TestCase, skipUnlessDBFeature
 from django.utils.translation import activate
 from django.core.exceptions import FieldError
 from django import forms
@@ -380,6 +380,13 @@ class MultiColumnFKTests(TestCase):
                     'active_translation')[0].active_translation.title,
                 "foo")
 
+    @skipUnlessDBFeature('has_bulk_insert')
+    def test_batch_create_foreign_object(self):
+        self.bob.person_country = self.usa
+
+        objs = [Person(name=i, person_country=self.usa) for i in range(0, 5)]
+        Person.objects.bulk_create(objs, 10)
+

[rhu@…]

PR at ​https://github.com/django/django/pull/2038

[Baptiste Mispelon]

Hi,

I can reproduce the issue being reported and the provided patch does seem to fix the problem.

I left a few comments on the pull request. Once those are fixed I think this should be good to go.

Thanks.

[rogerhu]

Thanks. I fixed against your comments. Also, since bulk creation is disabled for inherited models, local_concrete_fields vs. concrete_fields may not matter too much, but I've changed. Thanks again for the fast review!

[Baptiste Mispelon <bmispelon@…>]

In bbc73e6a12227f5ed52fd38bc37f56f434a0a72c:

Fixed #21566 -- Fixed AttributeError when using bulk_create with ForeignObject.

[Tim Graham <timograham@…>]

In 9a446211bd13998aa6f0b6a3e6e6cdd1c69df704:

[1.6.x] Fixed #21566 -- Fixed AttributeError when using bulk_create with ForeignObject.

Backport of bbc73e6a12 from master.

[Tim Graham <timograham@…>]

In 865392c47899d70229d2450ec398582f31e3cf1d:

Added 1.6.3 release note for refs #21566.

[Tim Graham <timograham@…>]

In a6f05af016b5488b42b9334ec915d6dfd2702c00:

[1.6.x] Added 1.6.3 release note for refs #21566.

Backport of 865392c478 from master