django-bulk-hooks 0.2.48__py3-none-any.whl → 0.2.49__py3-none-any.whl

django_bulk_hooks/operations/mti_handler.py

@@ -78,8 +78,7 @@ class MTIHandler:
                 chain.append(current_model)
 
             # Get concrete parent models (not abstract, not proxy)
-            parents = [parent for parent in current_model._meta.parents.keys() 
-                      if not parent._meta.proxy and not parent._meta.abstract]
+            parents = [parent for parent in current_model._meta.parents.keys() if not parent._meta.proxy and not parent._meta.abstract]
 
             current_model = parents[0] if parents else None
 
@@ -113,31 +112,33 @@ class MTIHandler:
 
     def find_model_with_unique_fields(self, unique_fields):
         """
-        Find which model in the inheritance chain contains the unique fields.
-        
-        This is critical for MTI upserts: we need to query the model that has
-        the unique constraint, not necessarily the child model.
-        
+        Find which model in the inheritance chain to query for existing records.
+
+        For MTI upsert operations, we need to determine if the parent record exists
+        to properly fire AFTER_CREATE vs AFTER_UPDATE hooks. This is critical because:
+        - If parent exists but child doesn't: creating child for existing parent → AFTER_UPDATE
+        - If neither exists: creating both parent and child → AFTER_CREATE
+
+        Therefore, we return the root parent model to check if the parent record exists,
+        regardless of where the unique fields are defined.
+
         Args:
             unique_fields: List of field names forming the unique constraint
-        
+
         Returns:
-            Model class that contains all the unique fields (closest to root)
+            Model class to query for existing records (root parent for MTI)
         """
         if not unique_fields:
             return self.model_cls
-        
+
         inheritance_chain = self.get_inheritance_chain()
-        
-        # Start from root and find the first model that has all unique fields
-        for model_cls in inheritance_chain:
-            model_field_names = {f.name for f in model_cls._meta.local_fields}
-            
-            # Check if this model has all the unique fields
-            if all(field in model_field_names for field in unique_fields):
-                return model_cls
-        
-        # Fallback to child model (shouldn't happen if unique_fields are valid)
+
+        # For MTI models with multiple levels, return the root parent model
+        # This ensures we check if the parent exists, which determines create vs update hooks
+        if len(inheritance_chain) > 1:
+            return inheritance_chain[0]  # Root parent model
+
+        # For non-MTI models (shouldn't happen, but safe fallback)
         return self.model_cls
 
     # ==================== MTI BULK CREATE PLANNING ====================

{django_bulk_hooks-0.2.48.dist-info → django_bulk_hooks-0.2.49.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.2.48
+Version: 0.2.49
 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.48.dist-info → django_bulk_hooks-0.2.49.dist-info}/RECORD

@@ -15,12 +15,12 @@ django_bulk_hooks/operations/__init__.py,sha256=BtJYjmRhe_sScivLsniDaZmBkm0ZLvcm
 django_bulk_hooks/operations/analyzer.py,sha256=wAG8sAG9NwfwNqG9z81VfGR7AANDzRmMGE_o82MWji4,10689
 django_bulk_hooks/operations/bulk_executor.py,sha256=RbvmYaZw94H31D3HMlK_hwWaHSy8Y3l8juxIzegtS7k,23527
 django_bulk_hooks/operations/coordinator.py,sha256=iGavJLqe3eYRqFay8cMn6muwyRYzQo-HFGphsS5hL6g,30799
-django_bulk_hooks/operations/mti_handler.py,sha256=GIpTzzoo_hx1iI5osx2COk3cVQgqTsD-o41d9S7aiqo,20748
+django_bulk_hooks/operations/mti_handler.py,sha256=uDrfE29e80v4YjPF4fPw2x0bntRnFGyfEh-grMQUGu0,20907
 django_bulk_hooks/operations/mti_plans.py,sha256=7STQ2oA2ZT8cEG3-t-6xciRAdf7OeSf0gRLXR_BRG-Q,3363
 django_bulk_hooks/operations/record_classifier.py,sha256=vNi0WSNiPAVb8pTZZJ26b81oX59snk7LIxWMzyenDCk,6694
 django_bulk_hooks/queryset.py,sha256=aQitlbexcVnmeAdc0jtO3hci39p4QEu4srQPEzozy5s,5546
 django_bulk_hooks/registry.py,sha256=uum5jhGI3TPaoiXuA1MdBdu4gbE3rQGGwQ5YDjiMcjk,7949
-django_bulk_hooks-0.2.48.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.2.48.dist-info/METADATA,sha256=R6l5_Y49oVdJK6YSLF6zRuRUVIuVSkUILudpFQKCumI,9265
-django_bulk_hooks-0.2.48.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.2.48.dist-info/RECORD,,
+django_bulk_hooks-0.2.49.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.2.49.dist-info/METADATA,sha256=ROH2Js_T0cGMRLaxUQnx9ZOQr9Rteii1n5MF0EJHqh8,9265
+django_bulk_hooks-0.2.49.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.2.49.dist-info/RECORD,,