django-bulk-hooks 0.1.275__tar.gz → 0.1.277__tar.gz

{django_bulk_hooks-0.1.275 → django_bulk_hooks-0.1.277}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-bulk-hooks
-Version: 0.1.275
+Version: 0.1.277
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 Home-page: https://github.com/AugendLimited/django-bulk-hooks
 License: MIT

{django_bulk_hooks-0.1.275 → django_bulk_hooks-0.1.277}/django_bulk_hooks/queryset.py

@@ -586,13 +586,9 @@ class HookQuerySetMixin:
                         converted_record = []
                         for i, field_name in enumerate(unique_fields):
                             db_value = existing_record[i]
-                            # Check if this field uses _id suffix in the query
-                            query_field_name = query_fields[field_name]
-                            if query_field_name.endswith("_id"):
-                                # Convert to string to match how we extract from objects
-                                converted_record.append(str(db_value))
-                            else:
-                                converted_record.append(db_value)
+                            # Convert all values to strings for consistent comparison
+                            # This ensures all database values are strings like object values
+                            converted_record.append(str(db_value))
                         converted_tuple = tuple(converted_record)
                         existing_records_lookup.add(converted_tuple)
 
@@ -629,7 +625,9 @@ class HookQuerySetMixin:
                                     # Convert to string to match how we convert DB values
                                     obj_unique_tuple.append(str(value))
                                 else:
-                                    obj_unique_tuple.append(value)
+                                    # For non-_id fields, also convert to string for consistency
+                                    # This ensures all values are strings like in the database lookup
+                                    obj_unique_tuple.append(str(value))
                             obj_unique_tuple = tuple(obj_unique_tuple)
 
                             logger.debug(
@@ -700,9 +698,16 @@ class HookQuerySetMixin:
                             if key in existing_db_map:
                                 db_record = existing_db_map[key]
                                 # Copy all fields from the database record to ensure completeness
+                                # but exclude auto_now_add fields which should never be updated
                                 populated_fields = []
                                 for field in model_cls._meta.local_fields:
                                     if field.name != "id":  # Don't overwrite the ID
+                                        # Skip auto_now_add fields for existing records
+                                        if (
+                                            hasattr(field, "auto_now_add")
+                                            and field.auto_now_add
+                                        ):
+                                            continue
                                         db_value = getattr(db_record, field.name)
                                         if (
                                             db_value is not None

{django_bulk_hooks-0.1.275 → django_bulk_hooks-0.1.277}/pyproject.toml

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