django-bulk-hooks 0.2.53__tar.gz → 0.2.54__tar.gz

{django_bulk_hooks-0.2.53 → django_bulk_hooks-0.2.54}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.2.53
+Version: 0.2.54
 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 → django_bulk_hooks-0.2.54}/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(
@@ -402,7 +406,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 +422,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 → django_bulk_hooks-0.2.54}/pyproject.toml

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