django-bulk-hooks 0.1.163__py3-none-any.whl → 0.1.165__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,8 +46,8 @@ 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,
         ):
             if not condition or condition.check(new, original):
@@ -39,5 +55,12 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
                 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)
+            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

@@ -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/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.dist-info → django_bulk_hooks-0.1.165.dist-info}/METADATA

@@ -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.163.dist-info → django_bulk_hooks-0.1.165.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=bkMlx8zCf7n36UM_aB9-KAS8beMgOGchYfvJsdZi58M,2547
 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=7wQv2V1SgfjshFSqwDdIuWU3NrRKQ4y-3mpV47JTZXc,5035
+django_bulk_hooks/manager.py,sha256=ipOr2PMGycW0LpWsZ0asNRTPKVNpp3D081ibuY__cj0,5551
 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.163.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.163.dist-info/METADATA,sha256=dDALy1e__e2zGAPq-I-a7yhVVFAchL4-ebPpKP8lL8g,6951
-django_bulk_hooks-0.1.163.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.163.dist-info/RECORD,,
+django_bulk_hooks-0.1.165.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.165.dist-info/METADATA,sha256=6Wv5rwnHJosRAbyuT1JVGnxQ5pI9XX6V6PGYAjuToQ0,6951
+django_bulk_hooks-0.1.165.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.165.dist-info/RECORD,,