django-bulk-hooks 0.1.256__tar.gz → 0.1.258__tar.gz

{django_bulk_hooks-0.1.256 → django_bulk_hooks-0.1.258}/PKG-INFO

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

@@ -482,6 +482,11 @@ class HookQuerySetMixin:
         passed through to the correct logic. For MTI, only a subset of options may be supported.
         """
         model_cls = self.model
+        
+        print(f"DEBUG: bulk_create called for {model_cls.__name__} with {len(objs)} objects")
+        print(f"DEBUG: update_conflicts={update_conflicts}, unique_fields={unique_fields}, update_fields={update_fields}")
+        logger.debug(f"bulk_create called for {model_cls.__name__} with {len(objs)} objects")
+        logger.debug(f"update_conflicts={update_conflicts}, unique_fields={unique_fields}, update_fields={update_fields}")
 
         # When you bulk insert you don't get the primary keys back (if it's an
         # autoincrement, except if can_return_rows_from_bulk_insert=True), so
@@ -624,6 +629,20 @@ class HookQuerySetMixin:
                                         # Preserve the original updated_at timestamp
                                         setattr(obj, field.name, getattr(db_record, field.name))
                 
+                # CRITICAL: For existing records, we need to set updated_at to current time
+                # since they are being updated, not just preserved
+                if existing_records:
+                    from django.utils import timezone
+                    current_time = timezone.now()
+                    print(f"DEBUG: Setting updated_at to current time for {len(existing_records)} existing records")
+                    logger.debug(f"Setting updated_at to current time for {len(existing_records)} existing records")
+                    for obj in existing_records:
+                        for field in model_cls._meta.local_fields:
+                            if hasattr(field, "auto_now") and field.auto_now:
+                                setattr(obj, field.name, current_time)
+                                print(f"DEBUG: Set {field.name} to {current_time} for existing record {obj.pk}")
+                                logger.debug(f"Set {field.name} to {current_time} for existing record {obj.pk}")
+                
                 # CRITICAL: Exclude auto_now fields from update_fields for existing records
                 # This prevents Django from including them in the ON CONFLICT DO UPDATE clause
                 if existing_records and update_fields:
@@ -817,6 +836,7 @@ class HookQuerySetMixin:
         logger.debug(
             f"bulk_update {model_cls.__name__} bypass_hooks={bypass_hooks} objs={len(objs)} fields={fields}"
         )
+        print(f"DEBUG: bulk_update {model_cls.__name__} bypass_hooks={bypass_hooks} objs={len(objs)} fields={fields}")
 
         # Check for MTI
         is_mti = False
@@ -846,28 +866,34 @@ class HookQuerySetMixin:
             # Don't include auto_now_add fields (like created_at) as they should only be set on creation
             if hasattr(field, "auto_now") and field.auto_now:
                 logger.debug(f"Found auto_now field: {field.name}")
+                print(f"DEBUG: Found auto_now field: {field.name}")
                 if field.name not in fields_set and field.name not in pk_fields:
                     fields_set.add(field.name)
                     if field.name != field.attname:
                         fields_set.add(field.attname)
                     auto_now_fields.append(field.name)
                     logger.debug(f"Added auto_now field {field.name} to fields list")
+                    print(f"DEBUG: Added auto_now field {field.name} to fields list")
                 else:
                     logger.debug(f"Auto_now field {field.name} already in fields list or is PK")
+                    print(f"DEBUG: Auto_now field {field.name} already in fields list or is PK")
             elif hasattr(field, "auto_now_add") and field.auto_now_add:
                 logger.debug(f"Found auto_now_add field: {field.name} (skipping)")
         
         logger.debug(f"Auto_now fields detected: {auto_now_fields}")
+        print(f"DEBUG: Auto_now fields detected: {auto_now_fields}")
         fields = list(fields_set)
         
         # Set auto_now field values to current timestamp
         if auto_now_fields:
             from django.utils import timezone
             current_time = timezone.now()
+            print(f"DEBUG: Setting auto_now fields {auto_now_fields} to current time: {current_time}")
             logger.debug(f"Setting auto_now fields {auto_now_fields} to current time: {current_time}")
             for obj in objs:
                 for field_name in auto_now_fields:
                     setattr(obj, field_name, current_time)
+                    print(f"DEBUG: Set {field_name} to {current_time} for object {obj.pk}")
 
         # Handle MTI models differently
         if is_mti:
@@ -880,20 +906,21 @@ class HookQuerySetMixin:
                 if k not in ["bypass_hooks", "bypass_validation"]
             }
             logger.debug("Calling Django bulk_update")
-                    # Build a per-object concrete value map to avoid leaking expressions into hooks
-        value_map = {}
-        logger.debug(f"Building value map for {len(objs)} objects with fields: {fields}")
-        for obj in objs:
-            if obj.pk is None:
-                continue
-            field_values = {}
-            for field_name in fields:
-                # Capture raw values assigned on the object (not expressions)
-                field_values[field_name] = getattr(obj, field_name)
-                if field_name in auto_now_fields:
-                    logger.debug(f"Object {obj.pk} {field_name}: {field_values[field_name]}")
-            if field_values:
-                value_map[obj.pk] = field_values
+            print("DEBUG: Calling Django bulk_update")
+            # Build a per-object concrete value map to avoid leaking expressions into hooks
+            value_map = {}
+            logger.debug(f"Building value map for {len(objs)} objects with fields: {fields}")
+            for obj in objs:
+                if obj.pk is None:
+                    continue
+                field_values = {}
+                for field_name in fields:
+                    # Capture raw values assigned on the object (not expressions)
+                    field_values[field_name] = getattr(obj, field_name)
+                    if field_name in auto_now_fields:
+                        logger.debug(f"Object {obj.pk} {field_name}: {field_values[field_name]}")
+                if field_values:
+                    value_map[obj.pk] = field_values
 
             # Make the value map available to the subsequent update() call
             if value_map:

{django_bulk_hooks-0.1.256 → django_bulk_hooks-0.1.258}/pyproject.toml

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