django-bulk-hooks 0.1.270__py3-none-any.whl → 0.1.272__py3-none-any.whl

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))
 
@@ -557,12 +558,16 @@ class HookQuerySetMixin:
                             filter_kwargs[db_field_name] = value
                         existing_filters |= Q(**filter_kwargs)
 
+                    logger.debug(f"DEBUG: Existence check query filters: {existing_filters}")
+                    logger.debug(f"DEBUG: Unique fields for values_list: {unique_fields}")
+
                     # Get all existing records in one query and create a lookup set
                     # We need to use the original unique_fields for values_list to maintain consistency
                     existing_records_lookup = set()
-                    for existing_record in model_cls.objects.filter(
-                        existing_filters
-                    ).values_list(*unique_fields):
+                    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):
                         # Convert tuple to a hashable key for lookup
                         existing_records_lookup.add(existing_record)
 
@@ -570,11 +575,12 @@ class HookQuerySetMixin:
                     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:

{django_bulk_hooks-0.1.270.dist-info → django_bulk_hooks-0.1.272.dist-info}/METADATA

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

@@ -9,9 +9,9 @@ django_bulk_hooks/handler.py,sha256=Bx-W6yyiciKMyy-BRxUt3CmRPCrX9_LhQgU-5LaJTjg,
 django_bulk_hooks/manager.py,sha256=3jNWL-EkvGScsliNc7mW-ozQCG6HyaEevI1u1BFS4AA,3836
 django_bulk_hooks/models.py,sha256=WtSfc4GBOG_oOt8n37cVvid0MtFIGze9JYKSixil2y0,4370
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=GggX-BoBm9hKcO5Bs6N2oQQ01yjhhGCBdz6YK4gkuHQ,90329
+django_bulk_hooks/queryset.py,sha256=7J1kU8trTSXKatdu2cE7sN39KTL5179YYPjjjzbBOe4,90788
 django_bulk_hooks/registry.py,sha256=GRUTGVQEO2sdkC9OaZ9Q3U7mM-3Ix83uTyvrlTtpatw,1317
-django_bulk_hooks-0.1.270.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.270.dist-info/METADATA,sha256=fCm_gvOH4hJioiQe3eRm-74Eg_nqhuxUu85oE9lLwm4,9115
-django_bulk_hooks-0.1.270.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.270.dist-info/RECORD,,
+django_bulk_hooks-0.1.272.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.272.dist-info/METADATA,sha256=rVznllpFeHYu1UyNYJPYM90biupnGzzpX2eQCEvJ5XM,9115
+django_bulk_hooks-0.1.272.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.272.dist-info/RECORD,,