django-bulk-hooks 0.1.225__tar.gz → 0.1.226__tar.gz

{django_bulk_hooks-0.1.225 → django_bulk_hooks-0.1.226}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-bulk-hooks
-Version: 0.1.225
+Version: 0.1.226
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 Home-page: https://github.com/AugendLimited/django-bulk-hooks
 License: MIT

{django_bulk_hooks-0.1.225 → django_bulk_hooks-0.1.226}/django_bulk_hooks/models.py

@@ -68,6 +68,7 @@ class HookModelMixin(models.Model):
             logger.debug(f"save() creating new {self.__class__.__name__} instance")
             # For create operations, we don't have old records
             ctx = HookContext(self.__class__)
+            run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
             run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
 
             super().save(*args, **kwargs)
@@ -80,6 +81,7 @@ class HookModelMixin(models.Model):
                 # Use _base_manager to avoid triggering hooks recursively
                 old_instance = self.__class__._base_manager.get(pk=self.pk)
                 ctx = HookContext(self.__class__)
+                run(self.__class__, VALIDATE_UPDATE, [self], [old_instance], ctx=ctx)
                 run(self.__class__, BEFORE_UPDATE, [self], [old_instance], ctx=ctx)
 
                 super().save(*args, **kwargs)
@@ -88,6 +90,7 @@ class HookModelMixin(models.Model):
             except self.__class__.DoesNotExist:
                 # If the old instance doesn't exist, treat as create
                 ctx = HookContext(self.__class__)
+                run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
                 run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
 
                 super().save(*args, **kwargs)

{django_bulk_hooks-0.1.225 → django_bulk_hooks-0.1.226}/django_bulk_hooks/queryset.py

@@ -65,11 +65,27 @@ class HookQuerySetMixin:
         }
         originals = [original_map.get(obj.pk) for obj in instances]
 
-        # Check if any of the update values are Subquery objects
+        # Check if any of the update values are complex database expressions (Subquery, Case, etc.)
         has_subquery = any(
-            hasattr(value, "query") and hasattr(value, "resolve_expression")
+            (hasattr(value, "query") and hasattr(value, "resolve_expression"))
+            or hasattr(
+                value, "resolve_expression"
+            )  # This catches Case, F expressions, etc.
             for value in kwargs.values()
         )
+        
+        # Also check if any of the instances have complex expressions in their attributes
+        # This can happen when bulk_update creates Case expressions and applies them to instances
+        if not has_subquery and instances:
+            for instance in instances:
+                for field_name in kwargs.keys():
+                    if hasattr(instance, field_name):
+                        field_value = getattr(instance, field_name)
+                        if hasattr(field_value, "resolve_expression"):
+                            has_subquery = True
+                            break
+                if has_subquery:
+                    break
 
         # Check if we're in a bulk operation context to prevent double hook execution
         from django_bulk_hooks.context import get_bypass_hooks
@@ -98,11 +114,11 @@ class HookQuerySetMixin:
                     engine.run(
                         model_cls, VALIDATE_UPDATE, instances, originals, ctx=ctx
                     )
-                except (TypeError, ValueError) as e:
-                    # If validation fails due to Subquery comparison, skip validation for subquery updates
-                    # This is a limitation - validation hooks cannot easily work with unresolved subqueries
+                except (TypeError, ValueError, AttributeError) as e:
+                    # If validation fails due to Subquery/Case comparison, skip validation for complex updates
+                    # This is a limitation - validation hooks cannot easily work with unresolved database expressions
                     logger.warning(
-                        f"Skipping validation hooks for subquery update due to: {e}"
+                        f"Skipping validation hooks for complex update due to: {e}"
                     )
 
                 # Execute the database update first to compute subquery values

{django_bulk_hooks-0.1.225 → django_bulk_hooks-0.1.226}/pyproject.toml

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