django-bulk-hooks 0.1.136__tar.gz → 0.1.138__tar.gz

{django_bulk_hooks-0.1.136 → django_bulk_hooks-0.1.138}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.136
+Version: 0.1.138
 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.136 → django_bulk_hooks-0.1.138}/django_bulk_hooks/queryset.py

@@ -393,7 +393,14 @@ class HookQuerySet(models.QuerySet):
             
             # 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]
+            print(f"DEBUG: Child model fields to insert: {[f.name for f in fields]}")
+            
+            # Debug: Check what fields are actually set on the child objects
+            for i, child_obj in enumerate(all_child_objects[:3]):  # Check first 3 objects
+                print(f"DEBUG: Child object {i} fields: {[f.name for f in child_model._meta.local_fields if hasattr(child_obj, f.name) and getattr(child_obj, f.name) is not None]}")
             
             with transaction.atomic(using=self.db, savepoint=False):
                  if objs_with_pk:
@@ -413,7 +420,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,
@@ -473,6 +481,7 @@ class HookQuerySet(models.QuerySet):
 
     def _create_child_instance(self, source_obj, child_model, parent_instances):
         child_obj = child_model()
+        # Only copy fields that exist in the child model's local fields
         for field in child_model._meta.local_fields:
             if isinstance(field, AutoField):
                 continue
@@ -480,6 +489,8 @@ class HookQuerySet(models.QuerySet):
                 value = getattr(source_obj, field.name, None)
                 if value is not None:
                     setattr(child_obj, field.name, value)
+        
+        # Set parent links for MTI
         for parent_model, parent_instance in parent_instances.items():
             parent_link = child_model._meta.get_ancestor_link(parent_model)
             if parent_link:

{django_bulk_hooks-0.1.136 → django_bulk_hooks-0.1.138}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.136"
+version = "0.1.138"
 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"