django-bulk-hooks 0.2.57__py3-none-any.whl → 0.2.58__py3-none-any.whl

django_bulk_hooks/operations/mti_handler.py

@@ -306,6 +306,14 @@ class MTIHandler:
                     if not filtered_updates and normalized_unique:
                         filtered_updates = [normalized_unique[0]]
 
+                    # CRITICAL FIX: Always include auto_now fields in updates to ensure timestamps are updated.
+                    # During MTI upsert, parent tables need auto_now fields updated even when only child fields change.
+                    # This ensures parent-level timestamps (e.g., updated_at) refresh correctly on upsert.
+                    auto_now_fields = self._get_auto_now_fields_for_model(model_class, model_fields_by_name)
+                    if auto_now_fields:
+                        # Convert to set to avoid duplicates, then back to list for consistency
+                        filtered_updates = list(set(filtered_updates) | set(auto_now_fields))
+
                     # Only enable upsert if we have fields to update (real or dummy)
                     if filtered_updates:
                         level_update_conflicts = True
@@ -334,6 +342,14 @@ class MTIHandler:
                         ]
                         level_update_fields = available_fields[:1] if available_fields else [pk_field.name]
 
+                        # CRITICAL FIX: Include auto_now fields in update_fields to ensure timestamps are updated.
+                        # During MTI upsert, parent tables need auto_now fields updated even when using dummy fields.
+                        # This ensures parent-level timestamps (e.g., updated_at) refresh correctly on upsert.
+                        auto_now_fields = self._get_auto_now_fields_for_model(model_class, model_fields_by_name)
+                        if auto_now_fields:
+                            # Convert to set to avoid duplicates, then back to list for consistency
+                            level_update_fields = list(set(level_update_fields) | set(auto_now_fields))
+
             # Create parent level
             parent_level = ParentLevel(
                 model_class=model_class,
@@ -347,6 +363,29 @@ class MTIHandler:
 
         return parent_levels
 
+    def _get_auto_now_fields_for_model(self, model_class, model_fields_by_name):
+        """
+        Get auto_now (not auto_now_add) fields for a specific model.
+        
+        Only includes fields that exist in model_fields_by_name to ensure
+        they're valid local fields for this model level.
+        
+        Args:
+            model_class: Model class to get fields for
+            model_fields_by_name: Dict of valid field names for this model level
+            
+        Returns:
+            List of auto_now field names (excluding auto_now_add)
+        """
+        auto_now_fields = []
+        for field in model_class._meta.local_fields:
+            # Only include auto_now (not auto_now_add) since auto_now_add should only be set on creation
+            if getattr(field, "auto_now", False) and not getattr(field, "auto_now_add", False):
+                # Double-check field exists in model_fields_by_name for safety
+                if field.name in model_fields_by_name:
+                    auto_now_fields.append(field.name)
+        return auto_now_fields
+
     def _has_matching_constraint(self, model_class, normalized_unique):
         """Check if model has a unique constraint matching the given fields."""
         try:

{django_bulk_hooks-0.2.57.dist-info → django_bulk_hooks-0.2.58.dist-info}/METADATA

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

@@ -16,12 +16,12 @@ django_bulk_hooks/operations/analyzer.py,sha256=wAG8sAG9NwfwNqG9z81VfGR7AANDzRmM
 django_bulk_hooks/operations/bulk_executor.py,sha256=Y-wkvuV_X-SZmI965JVrrtwbzPZVggUfy8mR1pzP9d0,27048
 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=MMDxuE6jzwkhMwVE-6qJUdp4MNKgAIFPMjwdN_umrqs,23503
+django_bulk_hooks/operations/mti_handler.py,sha256=173jghcxCE5UEZxM1QJRS-lWg0-KJxCQCbWHVKppIEM,26000
 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.57.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.2.57.dist-info/METADATA,sha256=SWZOMPOgiBDCLuSxHX9xTeJMqcd3CgvDZP-JuoY4wRs,9265
-django_bulk_hooks-0.2.57.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.2.57.dist-info/RECORD,,
+django_bulk_hooks-0.2.58.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.2.58.dist-info/METADATA,sha256=kVna-_dN9xVNaMMZHZUzm-Ab4IjhVzA1L8XXPms5T7o,9265
+django_bulk_hooks-0.2.58.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.2.58.dist-info/RECORD,,