django-bulk-hooks 0.1.135__tar.gz → 0.1.137__tar.gz

{django_bulk_hooks-0.1.135 → django_bulk_hooks-0.1.137}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.135
+Version: 0.1.137
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 License: MIT
 Keywords: django,bulk,hooks

{django_bulk_hooks-0.1.135 → django_bulk_hooks-0.1.137}/django_bulk_hooks/queryset.py

@@ -381,17 +381,21 @@ class HookQuerySet(models.QuerySet):
             # Use Django's exact approach: call _prepare_for_bulk_create then partition
             base_qs._prepare_for_bulk_create(all_child_objects)
             
-            # Use partition like Django does to separate objects with PKs from those without
-            from itertools import partition
-            objs_without_pk, objs_with_pk = partition(lambda o: o._is_pk_set(), all_child_objects)
-            objs_without_pk = list(objs_without_pk)
-            objs_with_pk = list(objs_with_pk)
+            # Implement our own partition since itertools.partition might not be available
+            objs_without_pk, objs_with_pk = [], []
+            for obj in all_child_objects:
+                if obj._is_pk_set():
+                    objs_with_pk.append(obj)
+                else:
+                    objs_without_pk.append(obj)
             
             print(f"DEBUG: Prepared {len(objs_with_pk)} objects with PK, {len(objs_without_pk)} objects without PK")
             
             # Use Django's internal _batched_insert method
             opts = child_model._meta
-            fields = [f for f in opts.concrete_fields if not f.generated]
+            # For child models in MTI, we need to exclude the parent's primary key
+            # since it's inherited and not stored in the child table
+            fields = [f for f in opts.concrete_fields if not f.generated and not f.primary_key]
             
             with transaction.atomic(using=self.db, savepoint=False):
                  if objs_with_pk:
@@ -411,7 +415,8 @@ class HookQuerySet(models.QuerySet):
                  
                  if objs_without_pk:
                      print(f"DEBUG: Inserting {len(objs_without_pk)} objects without PK")
-                     fields = [f for f in fields if not isinstance(f, AutoField)]
+                     # For objects without PK, we still need to exclude primary key fields
+                     fields = [f for f in fields if not isinstance(f, AutoField) and not f.primary_key]
                      returned_columns = base_qs._batched_insert(
                          objs_without_pk,
                          fields,

{django_bulk_hooks-0.1.135 → django_bulk_hooks-0.1.137}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.135"
+version = "0.1.137"
 description = "Hook-style hooks for Django bulk operations like bulk_create and bulk_update."
 authors = ["Konrad Beck <konrad.beck@merchantcapital.co.za>"]
 readme = "README.md"