django-bulk-hooks 0.1.163__tar.gz → 0.1.165__tar.gz

{django_bulk_hooks-0.1.163 → django_bulk_hooks-0.1.165}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.163
+Version: 0.1.165
 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.165/django_bulk_hooks/engine.py

@@ -0,0 +1,66 @@
+import logging
+
+from django.core.exceptions import ValidationError
+
+from django_bulk_hooks.registry import get_hooks
+
+logger = logging.getLogger(__name__)
+
+
+def run(model_cls, event, new_records, old_records=None, ctx=None):
+    """
+    Run hooks for a given model, event, and records.
+    """
+    print(f"DEBUG: engine.run called for {model_cls} with event {event}")
+    print(f"DEBUG: Number of new_records: {len(new_records) if new_records else 0}")
+    print(f"DEBUG: Number of old_records: {len(old_records) if old_records else 0}")
+    
+    if not new_records:
+        print(f"DEBUG: No new_records, skipping hooks")
+        return
+
+    # Get hooks for this model and event
+    hooks = get_hooks(model_cls, event)
+    print(f"DEBUG: Found {len(hooks)} hooks for {model_cls}.{event}")
+    
+    if not hooks:
+        print(f"DEBUG: No hooks found for {model_cls}.{event}")
+        return
+
+    # For BEFORE_* events, run model.clean() first for validation
+    if event.startswith("before_"):
+        for instance in new_records:
+            try:
+                instance.clean()
+            except ValidationError as e:
+                logger.error("Validation failed for %s: %s", instance, e)
+                raise
+
+    # Process hooks
+    for handler_cls, method_name, condition, priority in hooks:
+        print(f"DEBUG: Processing hook: {handler_cls.__name__}.{method_name}")
+        handler_instance = handler_cls()
+        func = getattr(handler_instance, method_name)
+
+        to_process_new = []
+        to_process_old = []
+
+        for new, original in zip(
+            new_records,
+            old_records or [None] * len(new_records),
+            strict=True,
+        ):
+            if not condition or condition.check(new, original):
+                to_process_new.append(new)
+                to_process_old.append(original)
+
+        if to_process_new:
+            print(f"DEBUG: Executing hook {handler_cls.__name__}.{method_name} with {len(to_process_new)} records")
+            try:
+                func(new_records=to_process_new, old_records=to_process_old if any(to_process_old) else None)
+                print(f"DEBUG: Hook {handler_cls.__name__}.{method_name} executed successfully")
+            except Exception as e:
+                print(f"DEBUG: Hook {handler_cls.__name__}.{method_name} failed with error: {e}")
+                raise
+        else:
+            print(f"DEBUG: Skipping hook {handler_cls.__name__}.{method_name} - no records to process")

{django_bulk_hooks-0.1.163 → django_bulk_hooks-0.1.165}/django_bulk_hooks/manager.py

@@ -24,9 +24,11 @@ class BulkHookManager(models.Manager):
         # Check if this is our HookQuerySet or a different QuerySet
         if hasattr(qs, 'bulk_update') and 'bypass_hooks' in inspect.signature(qs.bulk_update).parameters:
             # Our HookQuerySet - pass all parameters
+            print(f"DEBUG: Using our HookQuerySet for {self.model}")
             return qs.bulk_update(objs, fields, bypass_hooks=bypass_hooks, bypass_validation=bypass_validation, **kwargs)
         else:
             # Different QuerySet (like queryable_properties) - only pass standard parameters
+            print(f"DEBUG: Using different QuerySet ({type(qs)}) for {self.model}, bypassing hooks")
             django_kwargs = {k: v for k, v in kwargs.items() if k not in ['bypass_hooks', 'bypass_validation']}
             return qs.bulk_update(objs, fields, **django_kwargs)
 
@@ -52,6 +54,7 @@ class BulkHookManager(models.Manager):
         # Check if this is our HookQuerySet or a different QuerySet
         if hasattr(qs, 'bulk_create') and 'bypass_hooks' in inspect.signature(qs.bulk_create).parameters:
             # Our HookQuerySet - pass all parameters
+            print(f"DEBUG: Using our HookQuerySet for {self.model}")
             kwargs = {
                 'batch_size': batch_size,
                 'ignore_conflicts': ignore_conflicts,
@@ -62,6 +65,7 @@ class BulkHookManager(models.Manager):
             return qs.bulk_create(objs, bypass_hooks=bypass_hooks, bypass_validation=bypass_validation, **kwargs)
         else:
             # Different QuerySet - only pass standard parameters
+            print(f"DEBUG: Using different QuerySet ({type(qs)}) for {self.model}, bypassing hooks")
             kwargs = {
                 'batch_size': batch_size,
                 'ignore_conflicts': ignore_conflicts,
@@ -85,12 +89,14 @@ class BulkHookManager(models.Manager):
         # Check if this is our HookQuerySet or a different QuerySet
         if hasattr(qs, 'bulk_delete') and 'bypass_hooks' in inspect.signature(qs.bulk_delete).parameters:
             # Our HookQuerySet - pass all parameters
+            print(f"DEBUG: Using our HookQuerySet for {self.model}")
             kwargs = {
                 'batch_size': batch_size,
             }
             return qs.bulk_delete(objs, bypass_hooks=bypass_hooks, bypass_validation=bypass_validation, **kwargs)
         else:
             # Different QuerySet - only pass standard parameters
+            print(f"DEBUG: Using different QuerySet ({type(qs)}) for {self.model}, bypassing hooks")
             kwargs = {
                 'batch_size': batch_size,
             }

{django_bulk_hooks-0.1.163 → django_bulk_hooks-0.1.165}/django_bulk_hooks/queryset.py

@@ -113,9 +113,15 @@ class HookQuerySet(models.QuerySet):
         # Fire hooks before DB ops
         if not bypass_hooks:
             print(f"DEBUG: Firing BEFORE_CREATE hooks for {model_cls}")
+            print(f"DEBUG: Number of objects: {len(objs)}")
+            print(f"DEBUG: Object types: {[type(obj) for obj in objs]}")
+            print(f"DEBUG: QuerySet type: {type(self)}")
+            print(f"DEBUG: Is this HookQuerySet? {isinstance(self, HookQuerySet)}")
             ctx = HookContext(model_cls)
             if not bypass_validation:
+                print(f"DEBUG: Running VALIDATE_CREATE hooks")
                 engine.run(model_cls, VALIDATE_CREATE, objs, ctx=ctx)
+            print(f"DEBUG: Running BEFORE_CREATE hooks")
             engine.run(model_cls, BEFORE_CREATE, objs, ctx=ctx)
         else:
             print(f"DEBUG: Skipping hooks due to bypass_hooks=True for {model_cls}")
@@ -156,6 +162,9 @@ class HookQuerySet(models.QuerySet):
         # Fire AFTER_CREATE hooks
         if not bypass_hooks:
             print(f"DEBUG: Firing AFTER_CREATE hooks for {model_cls}")
+            print(f"DEBUG: Number of objects: {len(objs)}")
+            print(f"DEBUG: QuerySet type: {type(self)}")
+            print(f"DEBUG: Is this HookQuerySet? {isinstance(self, HookQuerySet)}")
             engine.run(model_cls, AFTER_CREATE, objs, ctx=ctx)
         else:
             print(f"DEBUG: Skipping AFTER_CREATE hooks due to bypass_hooks=True for {model_cls}")
@@ -180,6 +189,8 @@ class HookQuerySet(models.QuerySet):
         model_cls = self.model
         print(f"DEBUG: Model class: {model_cls}")
         print(f"DEBUG: bypass_hooks value: {bypass_hooks}")
+        print(f"DEBUG: QuerySet type: {type(self)}")
+        print(f"DEBUG: Is this HookQuerySet? {isinstance(self, HookQuerySet)}")
         
         if not objs:
             return []
@@ -204,7 +215,13 @@ class HookQuerySet(models.QuerySet):
                 engine.run(model_cls, VALIDATE_UPDATE, objs, originals, ctx=ctx)
 
             # Then run business logic hooks
-            engine.run(model_cls, BEFORE_UPDATE, objs, originals, ctx=ctx)
+            if not bypass_hooks:
+                print(f"DEBUG: Firing BEFORE_UPDATE hooks for {model_cls}")
+                print(f"DEBUG: Number of objects: {len(objs)}")
+                print(f"DEBUG: Object types: {[type(obj) for obj in objs]}")
+                engine.run(model_cls, BEFORE_UPDATE, objs, originals, ctx=ctx)
+            else:
+                print(f"DEBUG: Skipping hooks due to bypass_hooks=True for {model_cls}")
 
             # Automatically detect fields that were modified during BEFORE_UPDATE hooks
             modified_fields = self._detect_modified_fields(objs, originals)
@@ -224,6 +241,7 @@ class HookQuerySet(models.QuerySet):
 
         if not bypass_hooks:
             print(f"DEBUG: Firing AFTER_UPDATE hooks for {model_cls}")
+            print(f"DEBUG: Number of objects: {len(objs)}")
             engine.run(model_cls, AFTER_UPDATE, objs, originals, ctx=ctx)
         else:
             print(f"DEBUG: Skipping AFTER_UPDATE hooks due to bypass_hooks=True for {model_cls}")

{django_bulk_hooks-0.1.163 → django_bulk_hooks-0.1.165}/pyproject.toml

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

django_bulk_hooks-0.1.163/django_bulk_hooks/engine.py

@@ -1,43 +0,0 @@
-import logging
-
-from django.core.exceptions import ValidationError
-
-from django_bulk_hooks.registry import get_hooks
-
-logger = logging.getLogger(__name__)
-
-
-def run(model_cls, event, new_instances, original_instances=None, ctx=None):
-    hooks = get_hooks(model_cls, event)
-
-    if not hooks:
-        return
-
-    # For BEFORE_* events, run model.clean() first for validation
-    if event.startswith("before_"):
-        for instance in new_instances:
-            try:
-                instance.clean()
-            except ValidationError as e:
-                logger.error("Validation failed for %s: %s", instance, e)
-                raise
-
-    for handler_cls, method_name, condition, priority in hooks:
-        handler_instance = handler_cls()
-        func = getattr(handler_instance, method_name)
-
-        to_process_new = []
-        to_process_old = []
-
-        for new, original in zip(
-            new_instances,
-            original_instances or [None] * len(new_instances),
-            strict=True,
-        ):
-            if not condition or condition.check(new, original):
-                to_process_new.append(new)
-                to_process_old.append(original)
-
-        if to_process_new:
-            # Call the function with keyword arguments
-            func(new_records=to_process_new, old_records=to_process_old if any(to_process_old) else None)