django-bulk-hooks 0.1.164__tar.gz → 0.1.166__tar.gz

{django_bulk_hooks-0.1.164 → django_bulk_hooks-0.1.166}/PKG-INFO

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

@@ -0,0 +1,73 @@
+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,
+        ):
+            print(f"DEBUG: Checking condition for {handler_cls.__name__}.{method_name}")
+            print(f"DEBUG:  - new: {type(new)}")
+            print(f"DEBUG:  - original: {type(original)}")
+            print(f"DEBUG:  - condition: {condition}")
+            if not condition or condition.check(new, original):
+                print(f"DEBUG:  - condition passed, adding to process list")
+                to_process_new.append(new)
+                to_process_old.append(original)
+            else:
+                print(f"DEBUG:  - condition failed, skipping")
+
+        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.164 → django_bulk_hooks-0.1.166}/django_bulk_hooks/manager.py

@@ -5,7 +5,10 @@ from django_bulk_hooks.queryset import HookQuerySet
 
 class BulkHookManager(models.Manager):
     def get_queryset(self):
-        return HookQuerySet(self.model, using=self._db)
+        qs = HookQuerySet(self.model, using=self._db)
+        print(f"DEBUG: BulkHookManager.get_queryset() called for {self.model}")
+        print(f"DEBUG: Returning QuerySet type: {type(qs)}")
+        return qs
 
     def bulk_update(
         self, objs, fields, bypass_hooks=False, bypass_validation=False, **kwargs
@@ -20,6 +23,8 @@ class BulkHookManager(models.Manager):
         print(f"DEBUG: bulk_update method signature: {inspect.signature(method)}")
         print(f"DEBUG: Calling with args: objs={type(objs)}, fields={fields}, bypass_hooks={bypass_hooks}, bypass_validation={bypass_validation}, kwargs={kwargs}")
         print(f"DEBUG: QuerySet class: {type(qs)}")
+        print(f"DEBUG: Manager type: {type(self)}")
+        print(f"DEBUG: Model: {self.model}")
         
         # 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:

{django_bulk_hooks-0.1.164 → django_bulk_hooks-0.1.166}/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.164 → django_bulk_hooks-0.1.166}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.164"
+version = "0.1.166"
 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.164/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)