django-bulk-hooks 0.1.164__py3-none-any.whl → 0.1.166__py3-none-any.whl

django_bulk_hooks/engine.py

@@ -7,22 +7,38 @@ 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)
+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_instances:
+        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)
 
@@ -30,14 +46,28 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
         to_process_old = []
 
         for new, original in zip(
-            new_instances,
-            original_instances or [None] * len(new_instances),
+            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:
-            # Call the function with keyword arguments
-            func(new_records=to_process_new, old_records=to_process_old if any(to_process_old) else None)
+            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/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/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.dist-info → django_bulk_hooks-0.1.166.dist-info}/METADATA

@@ -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.164.dist-info → django_bulk_hooks-0.1.166.dist-info}/RECORD

@@ -3,15 +3,15 @@ django_bulk_hooks/conditions.py,sha256=mTvlLcttixbXRkTSNZU5VewkPUavbXRuD2BkJbVWM
 django_bulk_hooks/constants.py,sha256=3x1H1fSUUNo0DZONN7GUVDuySZctTR-jtByBHmAIX5w,303
 django_bulk_hooks/context.py,sha256=4IPuOX8TBAYBEMzN0RNHWgE6Giy2ZnR5uRXfd1cpIwk,1051
 django_bulk_hooks/decorators.py,sha256=tckDcxtOzKCbgvS9QydgeIAWTFDEl-ch3_Q--ruEGdQ,4831
-django_bulk_hooks/engine.py,sha256=3HbgV12JRYIy9IlygHPxZiHnFXj7EwzLyTuJNQeVIoI,1402
+django_bulk_hooks/engine.py,sha256=fqd32vwpNLoph2B6O7BW2eeja8yzUzenLx3Iwj2Mz_U,2964
 django_bulk_hooks/enums.py,sha256=Zo8_tJzuzZ2IKfVc7gZ-0tWPT8q1QhqZbAyoh9ZVJbs,381
 django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,4854
-django_bulk_hooks/manager.py,sha256=ipOr2PMGycW0LpWsZ0asNRTPKVNpp3D081ibuY__cj0,5551
+django_bulk_hooks/manager.py,sha256=nccIBEWLc9T18A97ZYT1KU4JzGSOvTjBdDOxS4D5bX0,5810
 django_bulk_hooks/models.py,sha256=5PBjBoGlHwAE5b4yaZ5kDjt5UtHfQp6pDrwB2XPs2tk,3850
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=qMMr_IgOrWqV5dYoEqOSKjJcxgepcovdjAn77QL5bs8,23574
+django_bulk_hooks/queryset.py,sha256=pdxnZKJU6eMHT-bxp3s4r8MUwaTXgnIUxgyKvN-1WB8,24723
 django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
-django_bulk_hooks-0.1.164.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.164.dist-info/METADATA,sha256=ctAhRwnzonbXKWblwj0eU6agetmVEhVcmvj2TqtQls0,6951
-django_bulk_hooks-0.1.164.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.164.dist-info/RECORD,,
+django_bulk_hooks-0.1.166.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.166.dist-info/METADATA,sha256=5-zw3ByLKxog0xPphuXBcwESKyJIXRfNFLApVQSG6n4,6951
+django_bulk_hooks-0.1.166.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.166.dist-info/RECORD,,