django-bulk-hooks 0.1.239__py3-none-any.whl → 0.1.240__py3-none-any.whl

django_bulk_hooks/queryset.py

@@ -55,6 +55,7 @@ class HookQuerySetMixin:
 
     @transaction.atomic
     def update(self, **kwargs):
+        logger.debug(f"Entering update method with {len(kwargs)} kwargs")
         instances = list(self)
         if not instances:
             return 0
@@ -70,13 +71,29 @@ class HookQuerySetMixin:
         originals = [original_map.get(obj.pk) for obj in instances]
 
         # Check if any of the update values are Subquery objects
-        from django.db.models import Subquery
-        has_subquery = any(
-            isinstance(value, Subquery)
-            for value in kwargs.values()
-        )
+        try:
+            from django.db.models import Subquery
+            logger.debug(f"Successfully imported Subquery from django.db.models")
+        except ImportError as e:
+            logger.error(f"Failed to import Subquery: {e}")
+            raise
+        
+        logger.debug(f"Checking for Subquery objects in {len(kwargs)} kwargs")
+        
+        subquery_detected = []
+        for key, value in kwargs.items():
+            is_subquery = isinstance(value, Subquery)
+            logger.debug(f"Key '{key}': type={type(value).__name__}, is_subquery={is_subquery}")
+            if is_subquery:
+                subquery_detected.append(key)
+        
+        has_subquery = len(subquery_detected) > 0
+        logger.debug(f"Subquery detection result: {has_subquery}, detected keys: {subquery_detected}")
         
         # Debug logging for Subquery detection
+        logger.debug(f"Update kwargs: {list(kwargs.keys())}")
+        logger.debug(f"Update kwargs types: {[(k, type(v).__name__) for k, v in kwargs.items()]}")
+        
         if has_subquery:
             logger.debug(f"Detected Subquery in update: {[k for k, v in kwargs.items() if isinstance(v, Subquery)]}")
         else:
@@ -84,6 +101,8 @@ class HookQuerySetMixin:
             for k, v in kwargs.items():
                 if hasattr(v, 'query') and hasattr(v, 'resolve_expression'):
                     logger.warning(f"Potential Subquery-like object detected but not recognized: {k}={type(v).__name__}")
+                    logger.warning(f"Object attributes: query={hasattr(v, 'query')}, resolve_expression={hasattr(v, 'resolve_expression')}")
+                    logger.warning(f"Object dir: {[attr for attr in dir(v) if not attr.startswith('_')][:10]}")
 
         # Apply field updates to instances
         # If a per-object value map exists (from bulk_update), prefer it over kwargs
@@ -162,7 +181,49 @@ class HookQuerySetMixin:
 
         # Use Django's built-in update logic directly
         # Call the base QuerySet implementation to avoid recursion
-        update_count = super().update(**kwargs)
+        
+        # Additional safety check: ensure Subquery objects are properly handled
+        # This prevents the "cannot adapt type 'Subquery'" error
+        safe_kwargs = {}
+        logger.debug(f"Processing {len(kwargs)} kwargs for safety check")
+        
+        for key, value in kwargs.items():
+            logger.debug(f"Processing key '{key}' with value type {type(value).__name__}")
+            
+            if isinstance(value, Subquery):
+                logger.debug(f"Found Subquery for field {key}")
+                # Ensure Subquery has proper output_field
+                if not hasattr(value, 'output_field') or value.output_field is None:
+                    logger.warning(f"Subquery for field {key} missing output_field, attempting to infer")
+                    # Try to infer from the model field
+                    try:
+                        field = model_cls._meta.get_field(key)
+                        logger.debug(f"Inferred field type: {type(field).__name__}")
+                        value = value.resolve_expression(None, None)
+                        value.output_field = field
+                        logger.debug(f"Set output_field to {field}")
+                    except Exception as e:
+                        logger.error(f"Failed to infer output_field for Subquery on {key}: {e}")
+                        raise
+                else:
+                    logger.debug(f"Subquery for field {key} already has output_field: {value.output_field}")
+                safe_kwargs[key] = value
+            else:
+                logger.debug(f"Non-Subquery value for field {key}: {type(value).__name__}")
+                safe_kwargs[key] = value
+        
+        logger.debug(f"Safe kwargs keys: {list(safe_kwargs.keys())}")
+        logger.debug(f"Safe kwargs types: {[(k, type(v).__name__) for k, v in safe_kwargs.items()]}")
+        
+        logger.debug(f"Calling super().update() with {len(safe_kwargs)} kwargs")
+        try:
+            update_count = super().update(**safe_kwargs)
+            logger.debug(f"Super update successful, count: {update_count}")
+        except Exception as e:
+            logger.error(f"Super update failed: {e}")
+            logger.error(f"Exception type: {type(e).__name__}")
+            logger.error(f"Safe kwargs that caused failure: {safe_kwargs}")
+            raise
 
         # If we used Subquery objects, refresh the instances to get computed values
         if has_subquery and instances:

{django_bulk_hooks-0.1.239.dist-info → django_bulk_hooks-0.1.240.dist-info}/METADATA

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

@@ -9,9 +9,9 @@ django_bulk_hooks/handler.py,sha256=Bx-W6yyiciKMyy-BRxUt3CmRPCrX9_LhQgU-5LaJTjg,
 django_bulk_hooks/manager.py,sha256=nfWiwU5-yAoxdnQsUMohxtyCpkV0MBv6X3wmipr9eQY,3697
 django_bulk_hooks/models.py,sha256=exnXYVKEVbYAXhChCP8VdWTnKCnm9DiTcokEIBee1I0,4350
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=38BvCf_FgKMCAOxEoSYMSkNnmpO2bvUqaPcJkf_6gMc,38561
+django_bulk_hooks/queryset.py,sha256=bi8jE8yvl2ih0M3LbzW8TZvl7JFKbgysILRXIpci6KM,42055
 django_bulk_hooks/registry.py,sha256=GRUTGVQEO2sdkC9OaZ9Q3U7mM-3Ix83uTyvrlTtpatw,1317
-django_bulk_hooks-0.1.239.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.239.dist-info/METADATA,sha256=CDwXkK4tvZenRQrbIHkpYE75r2hZRwI8S8NDzmT5IgI,9061
-django_bulk_hooks-0.1.239.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.239.dist-info/RECORD,,
+django_bulk_hooks-0.1.240.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.240.dist-info/METADATA,sha256=hkqwdtVcH2Te0py2ryH-RXgAJpX93e0IDFteueUBYtQ,9061
+django_bulk_hooks-0.1.240.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.240.dist-info/RECORD,,