django-bulk-hooks 0.2.53__py3-none-any.whl → 0.2.55__py3-none-any.whl

django_bulk_hooks/operations/mti_handler.py

@@ -193,8 +193,12 @@ class MTIHandler:
         if existing_pks_map is None:
             existing_pks_map = {}
 
-        # For upsert operations, don't set PKs on source objects - let Django's bulk_create handle it
-        # The PKs will be set on the resulting objects after the bulk operation
+        # Set PKs on existing objects so they can be updated
+        if existing_pks_map:
+            for obj in objs:
+                if id(obj) in existing_pks_map:
+                    obj.pk = existing_pks_map[id(obj)]
+                    obj.id = existing_pks_map[id(obj)]
 
         # Build parent levels
         parent_levels = self._build_parent_levels(
@@ -357,6 +361,13 @@ class MTIHandler:
 
         # Copy field values from source using centralized field extraction
         for field in parent_model._meta.local_fields:
+            # Skip AutoField (primary key) - let Django's bulk_create with update_conflicts
+            # handle PK assignment based on unique_fields. Setting PKs manually here causes
+            # conflicts when parent records exist but update_conflicts isn't properly configured
+            # for this level, leading to IntegrityError on primary key constraint.
+            if isinstance(field, AutoField):
+                continue
+                
             if hasattr(source_obj, field.name):
                 # Use centralized field value extraction for consistent FK handling
                 value = get_field_value_for_db(source_obj, field.name, source_obj.__class__)
@@ -402,7 +413,12 @@ class MTIHandler:
         """
         child_obj = child_model()
 
-        # Copy field values (excluding AutoField and parent links)
+        # Copy field values (excluding AutoField, parent links, and inherited fields)
+        # In MTI, child objects should only have values for fields defined directly on the child model
+        parent_fields = set()
+        for parent_model in child_model._meta.parents.keys():
+            parent_fields.update(f.name for f in parent_model._meta.local_fields)
+
         for field in child_model._meta.local_fields:
             if isinstance(field, AutoField):
                 continue
@@ -413,6 +429,10 @@ class MTIHandler:
                 if child_model._meta.get_ancestor_link(field.related_model) == field:
                     continue
 
+            # Skip inherited fields - these belong to parent models
+            if field.name in parent_fields:
+                continue
+
             if hasattr(source_obj, field.name):
                 # Use centralized field value extraction for consistent FK handling
                 value = get_field_value_for_db(source_obj, field.name, source_obj.__class__)

{django_bulk_hooks-0.2.53.dist-info → django_bulk_hooks-0.2.55.dist-info}/METADATA

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

{django_bulk_hooks-0.2.53.dist-info → django_bulk_hooks-0.2.55.dist-info}/RECORD

@@ -16,12 +16,12 @@ django_bulk_hooks/operations/analyzer.py,sha256=wAG8sAG9NwfwNqG9z81VfGR7AANDzRmM
 django_bulk_hooks/operations/bulk_executor.py,sha256=tMbcwQSN8xNoKL7-KC8m-XhGyqwaAtAQ5CAm3PYHl_Q,24122
 django_bulk_hooks/operations/coordinator.py,sha256=iGavJLqe3eYRqFay8cMn6muwyRYzQo-HFGphsS5hL6g,30799
 django_bulk_hooks/operations/field_utils.py,sha256=Tvr5bcZLG8imH-r2S85oui1Cbw6hGv3VtuIMn4OvsU4,2895
-django_bulk_hooks/operations/mti_handler.py,sha256=SpDe2fKfFgYfRxhimYfu-d1f4g34luXDoGb8oRR0abI,20549
+django_bulk_hooks/operations/mti_handler.py,sha256=ZR4Bps0lPdT9syrk-AMdqsxFN34QRJVqH1peR_E4h0s,21550
 django_bulk_hooks/operations/mti_plans.py,sha256=7STQ2oA2ZT8cEG3-t-6xciRAdf7OeSf0gRLXR_BRG-Q,3363
 django_bulk_hooks/operations/record_classifier.py,sha256=kqML4aO11X9K3SSJ5DUlUukwI172j_Tk12Kr77ee8q8,7065
 django_bulk_hooks/queryset.py,sha256=aQitlbexcVnmeAdc0jtO3hci39p4QEu4srQPEzozy5s,5546
 django_bulk_hooks/registry.py,sha256=uum5jhGI3TPaoiXuA1MdBdu4gbE3rQGGwQ5YDjiMcjk,7949
-django_bulk_hooks-0.2.53.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.2.53.dist-info/METADATA,sha256=0DsfhnU5TezxhZ1z3-it4KNalYb2aAhuaHqsLKqCWaU,9265
-django_bulk_hooks-0.2.53.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.2.53.dist-info/RECORD,,
+django_bulk_hooks-0.2.55.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.2.55.dist-info/METADATA,sha256=2goa-ou7_Q77g-z_GnbCWyhV5Tp7WUhiQFbURMBTOX0,9265
+django_bulk_hooks-0.2.55.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.2.55.dist-info/RECORD,,