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

django_bulk_hooks/queryset.py

@@ -567,9 +567,30 @@ 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):
-                        # Convert tuple to a hashable key for lookup
-                        existing_records_lookup.add(existing_record)
+                    # 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}")
+
+                    # Convert database values to match object types for comparison
+                    # This handles cases where object values are strings but DB values are integers
+                    existing_records_lookup = set()
+                    for existing_record in raw_existing:
+                        # Convert each value in the tuple to match the type from object extraction
+                        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)
+                        converted_tuple = tuple(converted_record)
+                        existing_records_lookup.add(converted_tuple)
+
+                    logger.debug(f"DEBUG: Found {len(raw_existing)} existing records from DB")
+                    logger.debug(f"DEBUG: Existing records lookup set: {existing_records_lookup}")
 
                     # Separate records based on whether they already exist
                     for obj in objs:
@@ -589,10 +610,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.272.dist-info → django_bulk_hooks-0.1.274.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.272
+Version: 0.1.274
 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.272.dist-info → django_bulk_hooks-0.1.274.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=7J1kU8trTSXKatdu2cE7sN39KTL5179YYPjjjzbBOe4,90788
+django_bulk_hooks/queryset.py,sha256=N5Xff3rxGKLvVzQO6NXypSDaAFFVZhRjTcdGSCeSy3c,92606
 django_bulk_hooks/registry.py,sha256=GRUTGVQEO2sdkC9OaZ9Q3U7mM-3Ix83uTyvrlTtpatw,1317
-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,,
+django_bulk_hooks-0.1.274.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.274.dist-info/METADATA,sha256=WOMGNPawACo8Tsnwffd0NVzsdjWRpbsa5LEMt_7DqE8,9115
+django_bulk_hooks-0.1.274.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.274.dist-info/RECORD,,