django-bulk-hooks 0.2.27__tar.gz → 0.2.29__tar.gz

{django_bulk_hooks-0.2.27 → django_bulk_hooks-0.2.29}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.2.27
+Version: 0.2.29
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 License: MIT
 Keywords: django,bulk,hooks

{django_bulk_hooks-0.2.27 → django_bulk_hooks-0.2.29}/django_bulk_hooks/operations/bulk_executor.py

@@ -7,7 +7,7 @@ This service coordinates bulk database operations with validation and MTI handli
 import logging
 
 from django.db import transaction
-from django.db.models import AutoField
+from django.db.models import AutoField, ForeignKey
 
 logger = logging.getLogger(__name__)
 
@@ -455,58 +455,25 @@ class BulkExecutor:
                 case_statements = {}
                 for field_name in field_group.fields:
                     field = field_group.model_class._meta.get_field(field_name)
-
-                    # Use column name for FK fields
-                    if getattr(field, "is_relation", False) and hasattr(field, "attname"):
-                        db_field_name = field.attname
-                        # For FK fields, use the actual target field from the relationship
-                        # This ensures proper type casting for the database column
-                        if hasattr(field, "target_field"):
-                            target_field = field.target_field
-                        else:
-                            # Fallback: create a field that matches the FK column type
-                            from django.db.models import BigIntegerField
-                            target_field = BigIntegerField()
-                        
-                        logger.debug(f"FK field {field_name} -> column {db_field_name}, target_field: {target_field}")
-                    else:
-                        db_field_name = field_name
-                        target_field = field
-
                     when_statements = []
+
                     for pk, obj in zip(root_pks, plan.objects):
                         obj_pk = getattr(obj, "pk", None) or getattr(obj, "id", None)
                         if obj_pk is None:
                             continue
 
-                        value = getattr(obj, db_field_name)
-
-                        # For FK fields, ensure we get the actual ID value, not the related object
-                        if getattr(field, "is_relation", False) and hasattr(field, "attname"):
-                            # If value is a model instance, get its pk
-                            if value is not None and hasattr(value, "pk"):
-                                value = value.pk
-                            # If value is a string representation of an ID, convert to int
-                            elif value is not None and isinstance(value, str):
-                                if value.isdigit():
-                                    value = int(value)
-                                elif value == "":
-                                    value = None
-
-                        # Create the Value expression with proper type casting
-                        value_expr = Value(value, output_field=target_field)
+                        # Get the field value - handle ForeignKey fields specially
+                        value = getattr(obj, field.attname, None) if isinstance(field, ForeignKey) else getattr(obj, field_name)
                         
                         when_statements.append(
                             When(
                                 **{field_group.filter_field: pk},
-                                then=value_expr,
+                                then=Value(value, output_field=field),
                             ),
                         )
 
                     if when_statements:
-                        case_statements[db_field_name] = Case(
-                            *when_statements, output_field=target_field,
-                        )
+                        case_statements[field_name] = Case(*when_statements, output_field=field)
 
                 # Execute bulk update
                 if case_statements:

{django_bulk_hooks-0.2.27 → django_bulk_hooks-0.2.29}/pyproject.toml

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