django-bulk-hooks 0.1.271__tar.gz → 0.1.273__tar.gz

{django_bulk_hooks-0.1.271 → django_bulk_hooks-0.1.273}/PKG-INFO

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

@@ -534,13 +534,14 @@ class HookQuerySetMixin:
                     unique_value = {}
                     query_fields = {}  # Track which database field to use for each unique field
                     for field_name in unique_fields:
-                        if hasattr(obj, field_name):
-                            unique_value[field_name] = getattr(obj, field_name)
-                            query_fields[field_name] = field_name
-                        elif hasattr(obj, field_name + '_id'):
+                        # First check for _id field (more reliable for ForeignKeys)
+                        if hasattr(obj, field_name + '_id'):
                             # Handle ForeignKey fields where _id suffix is used
                             unique_value[field_name] = getattr(obj, field_name + '_id')
                             query_fields[field_name] = field_name + '_id'  # Use _id field for query
+                        elif hasattr(obj, field_name):
+                            unique_value[field_name] = getattr(obj, field_name)
+                            query_fields[field_name] = field_name
                     if unique_value:
                         unique_values.append((unique_value, query_fields))
 
@@ -566,19 +567,29 @@ class HookQuerySetMixin:
                     existing_query = model_cls.objects.filter(existing_filters)
                     logger.debug(f"DEBUG: Existence check SQL: {existing_query.query}")
 
-                    for existing_record in existing_query.values_list(*unique_fields):
+                    # Also get the raw database values for debugging
+                    raw_existing = list(existing_query.values_list(*unique_fields))
+                    logger.debug(f"DEBUG: Raw existing records from DB: {raw_existing}")
+
+                    existing_records_found = []
+                    for existing_record in raw_existing:
                         # Convert tuple to a hashable key for lookup
                         existing_records_lookup.add(existing_record)
+                        existing_records_found.append(existing_record)
+
+                    logger.debug(f"DEBUG: Found {len(existing_records_found)} existing records: {existing_records_found}")
+                    logger.debug(f"DEBUG: Existing records lookup set: {existing_records_lookup}")
 
                     # Separate records based on whether they already exist
                     for obj in objs:
                         obj_unique_value = {}
                         for field_name in unique_fields:
-                            if hasattr(obj, field_name):
-                                obj_unique_value[field_name] = getattr(obj, field_name)
-                            elif hasattr(obj, field_name + '_id'):
+                            # First check for _id field (more reliable for ForeignKeys)
+                            if hasattr(obj, field_name + '_id'):
                                 # Handle ForeignKey fields where _id suffix is used
                                 obj_unique_value[field_name] = getattr(obj, field_name + '_id')
+                            elif hasattr(obj, field_name):
+                                obj_unique_value[field_name] = getattr(obj, field_name)
 
                         # Check if this record already exists using our bulk lookup
                         if obj_unique_value:
@@ -587,10 +598,14 @@ class HookQuerySetMixin:
                                 obj_unique_value[field_name]
                                 for field_name in unique_fields
                             )
+                            logger.debug(f"DEBUG: Object unique tuple: {obj_unique_tuple}")
+                            logger.debug(f"DEBUG: Object unique value: {obj_unique_value}")
                             if obj_unique_tuple in existing_records_lookup:
                                 existing_records.append(obj)
+                                logger.debug(f"DEBUG: Found existing record for tuple: {obj_unique_tuple}")
                             else:
                                 new_records.append(obj)
+                                logger.debug(f"DEBUG: No existing record found for tuple: {obj_unique_tuple}")
                         else:
                             # If we can't determine uniqueness, treat as new
                             new_records.append(obj)

{django_bulk_hooks-0.1.271 → django_bulk_hooks-0.1.273}/pyproject.toml

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