django-bulk-hooks 0.1.243__tar.gz → 0.1.244__tar.gz

{django_bulk_hooks-0.1.243 → django_bulk_hooks-0.1.244}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-bulk-hooks
-Version: 0.1.243
+Version: 0.1.244
 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.243 → django_bulk_hooks-0.1.244}/django_bulk_hooks/queryset.py

@@ -158,18 +158,20 @@ class HookQuerySetMixin:
                         else:
                             setattr(obj, field, value)
 
-        # Check if we're in a bulk operation context to prevent double hook execution
+        # Salesforce-style trigger behavior: Always run hooks, rely on Django's stack overflow protection
         from django_bulk_hooks.context import get_bypass_hooks
 
         current_bypass_hooks = get_bypass_hooks()
 
-        # If we're in a bulk operation context, skip hooks to prevent double execution
+        # Only skip hooks if explicitly bypassed (not for recursion prevention)
         if current_bypass_hooks:
-            logger.debug("update: skipping hooks (bulk context)")
+            logger.debug("update: hooks explicitly bypassed")
             ctx = HookContext(model_cls, bypass_hooks=True)
         else:
-            logger.debug("update: running hooks (standalone)")
+            # Always run hooks - Django will handle stack overflow protection
+            logger.debug("update: running hooks with Salesforce-style behavior")
             ctx = HookContext(model_cls, bypass_hooks=False)
+
             # Run validation hooks first
             engine.run(model_cls, VALIDATE_UPDATE, instances, originals, ctx=ctx)
             # Then run BEFORE_UPDATE hooks
@@ -395,19 +397,12 @@ class HookQuerySetMixin:
                                 getattr(refreshed_instance, field.name),
                             )
 
-        # Run AFTER_UPDATE hooks for standalone updates or subquery operations
-        # For subquery operations, we need to run hooks even if we're in a bulk context
-        # because subqueries bypass the normal object-level update flow
-        should_run_hooks = (
-            not current_bypass_hooks
-            or has_subquery  # Always run hooks for subquery operations
-        )
-
-        if should_run_hooks:
+        # Salesforce-style: Always run AFTER_UPDATE hooks unless explicitly bypassed
+        if not current_bypass_hooks:
             logger.debug("update: running AFTER_UPDATE")
             engine.run(model_cls, AFTER_UPDATE, instances, originals, ctx=ctx)
         else:
-            logger.debug("update: skipping AFTER_UPDATE (bulk context)")
+            logger.debug("update: AFTER_UPDATE explicitly bypassed")
 
         return update_count
 

{django_bulk_hooks-0.1.243 → django_bulk_hooks-0.1.244}/pyproject.toml

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