django-bulk-hooks 0.1.63__tar.gz → 0.1.65__tar.gz

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.63
+Version: 0.1.65
 Summary: Lifecycle-style hooks for Django bulk operations like bulk_create and bulk_update.
 License: MIT
 Keywords: django,bulk,hooks

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/django_bulk_hooks/decorators.py

@@ -3,7 +3,6 @@ from functools import wraps
 
 from django.core.exceptions import FieldDoesNotExist
 from django_bulk_hooks.enums import DEFAULT_PRIORITY
-from django_bulk_hooks.constants import BEFORE_UPDATE
 from django_bulk_hooks.registry import register_hook
 
 

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/django_bulk_hooks/engine.py

@@ -9,7 +9,7 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
     hooks = get_hooks(model_cls, event)
 
     logger.debug(
-        "bulk_hooks.run: model=%s, event=%s, #new=%d, #original=%d",
+        "Executing engine.run: model=%s, event=%s, #new_instances=%d, #original_instances=%d",
         model_cls.__name__,
         event,
         len(new_instances),

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/django_bulk_hooks/manager.py

@@ -37,7 +37,7 @@ class BulkLifecycleManager(models.Manager):
             originals = list(model_cls.objects.filter(pk__in=[obj.pk for obj in objs]))
             ctx = TriggerContext(model_cls)
             engine.run(model_cls, BEFORE_UPDATE, objs, originals, ctx=ctx)
-            
+
             # Automatically detect fields that were modified during BEFORE_UPDATE hooks
             modified_fields = self._detect_modified_fields(objs, originals)
             if modified_fields:
@@ -47,7 +47,7 @@ class BulkLifecycleManager(models.Manager):
                 fields = list(fields_set)
                 logger.info(
                     "Automatically including modified fields in bulk_update: %s",
-                    modified_fields
+                    modified_fields,
                 )
 
         for i in range(0, len(objs), self.CHUNK_SIZE):
@@ -67,28 +67,28 @@ class BulkLifecycleManager(models.Manager):
         """
         if not original_instances:
             return set()
-        
+
         # Create a mapping of pk to original instance for efficient lookup
         original_map = {obj.pk: obj for obj in original_instances if obj.pk is not None}
-        
+
         modified_fields = set()
-        
+
         for new_instance in new_instances:
             if new_instance.pk is None:
                 continue
-                
+
             original = original_map.get(new_instance.pk)
             if not original:
                 continue
-            
+
             # Compare all fields to detect changes
             for field in new_instance._meta.fields:
-                if field.name == 'id':
+                if field.name == "id":
                     continue
-                    
+
                 new_value = getattr(new_instance, field.name)
                 original_value = getattr(original, field.name)
-                
+
                 # Handle different field types appropriately
                 if field.is_relation:
                     # For foreign keys, compare the pk values
@@ -100,7 +100,7 @@ class BulkLifecycleManager(models.Manager):
                     # For regular fields, use direct comparison
                     if new_value != original_value:
                         modified_fields.add(field.name)
-        
+
         return modified_fields
 
     @transaction.atomic
@@ -120,9 +120,7 @@ class BulkLifecycleManager(models.Manager):
 
         for i in range(0, len(objs), self.CHUNK_SIZE):
             chunk = objs[i : i + self.CHUNK_SIZE]
-            result.extend(
-                super(models.Manager, self).bulk_create(chunk, **kwargs)
-            )
+            result.extend(super(models.Manager, self).bulk_create(chunk, **kwargs))
 
         if not bypass_hooks:
             engine.run(model_cls, AFTER_CREATE, result, ctx=ctx)
@@ -144,12 +142,14 @@ class BulkLifecycleManager(models.Manager):
         ctx = TriggerContext(model_cls)
 
         if not bypass_hooks:
+            logger.debug("Executing BEFORE_DELETE hooks for %s", model_cls.__name__)
             engine.run(model_cls, BEFORE_DELETE, objs, ctx=ctx)
 
         pks = [obj.pk for obj in objs if obj.pk is not None]
         model_cls.objects.filter(pk__in=pks).delete()
 
         if not bypass_hooks:
+            logger.debug("Executing AFTER_DELETE hooks for %s", model_cls.__name__)
             engine.run(model_cls, AFTER_DELETE, objs, ctx=ctx)
 
         return objs

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/django_bulk_hooks/registry.py

@@ -3,6 +3,10 @@ from typing import Union
 
 from django_bulk_hooks.priority import Priority
 
+import logging
+
+logger = logging.getLogger(__name__)
+
 _hooks: dict[tuple[type, str], list[tuple[type, str, Callable, int]]] = {}
 
 
@@ -14,7 +18,22 @@ def register_hook(
     hooks.append((handler_cls, method_name, condition, priority))
     # keep sorted by priority
     hooks.sort(key=lambda x: x[3])
+    logger.debug(
+        "Registering hook: model=%s, event=%s, handler_cls=%s, method_name=%s, condition=%s, priority=%s",
+        model.__name__,
+        event,
+        handler_cls.__name__,
+        method_name,
+        condition,
+        priority,
+    )
 
 
 def get_hooks(model, event):
+    logger.debug(
+        "Retrieving hooks: model=%s, event=%s, hooks_found=%d",
+        model.__name__,
+        event,
+        len(_hooks.get((model, event), [])),
+    )
     return _hooks.get((model, event), [])

{django_bulk_hooks-0.1.63 → django_bulk_hooks-0.1.65}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.63"
+version = "0.1.65"
 description = "Lifecycle-style hooks for Django bulk operations like bulk_create and bulk_update."
 authors = ["Konrad Beck <konrad.beck@merchantcapital.co.za>"]
 readme = "README.md"