django-bulk-hooks 0.1.219__py3-none-any.whl → 0.1.221__py3-none-any.whl

django_bulk_hooks/conditions.py

@@ -1,3 +1,8 @@
+import logging
+
+logger = logging.getLogger(__name__)
+
+
 def resolve_dotted_attr(instance, dotted_path):
     """
     Recursively resolve a dotted attribute path, e.g., "type.category".
@@ -66,19 +71,16 @@ class HasChanged(HookCondition):
         self.has_changed = has_changed
 
     def check(self, instance, original_instance=None):
-        print(f"DEBUG: HasChanged.check called for field '{self.field}' on instance {getattr(instance, 'pk', 'No PK')}")
-        print(f"DEBUG: Original instance: {getattr(original_instance, 'pk', 'No PK') if original_instance else 'None'}")
-        
         if not original_instance:
-            print(f"DEBUG: No original instance, returning False")
             return False
         
         current = resolve_dotted_attr(instance, self.field)
         previous = resolve_dotted_attr(original_instance, self.field)
         
-        # Add more detailed debugging
         result = (current != previous) == self.has_changed
-        print(f"DEBUG: HasChanged {self.field} result={result}")
+        # Only log when there's an actual change to reduce noise
+        if result:
+            logger.debug(f"HasChanged {self.field} detected change on instance {getattr(instance, 'pk', 'No PK')}")
         return result
 
 

django_bulk_hooks/models.py

@@ -1,3 +1,4 @@
+import logging
 from django.db import models
 
 from django_bulk_hooks.constants import (
@@ -15,6 +16,8 @@ from django_bulk_hooks.context import HookContext
 from django_bulk_hooks.engine import run
 from django_bulk_hooks.manager import BulkHookManager
 
+logger = logging.getLogger(__name__)
+
 
 class HookModelMixin(models.Model):
     objects = BulkHookManager()
@@ -56,15 +59,13 @@ class HookModelMixin(models.Model):
     def save(self, *args, bypass_hooks=False, **kwargs):
         # If bypass_hooks is True, use base manager to avoid triggering hooks
         if bypass_hooks:
-            print(
-                f"DEBUG: save() called with bypass_hooks=True for {self.__class__.__name__} pk={self.pk}"
-            )
+            logger.debug(f"save() called with bypass_hooks=True for {self.__class__.__name__} pk={self.pk}")
             return self._base_manager.save(self, *args, **kwargs)
 
         is_create = self.pk is None
 
         if is_create:
-            print(f"DEBUG: save() creating new {self.__class__.__name__} instance")
+            logger.debug(f"save() creating new {self.__class__.__name__} instance")
             # For create operations, we don't have old records
             ctx = HookContext(self.__class__)
             run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
@@ -73,9 +74,7 @@ class HookModelMixin(models.Model):
 
             run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
         else:
-            print(
-                f"DEBUG: save() updating existing {self.__class__.__name__} instance pk={self.pk}"
-            )
+            logger.debug(f"save() updating existing {self.__class__.__name__} instance pk={self.pk}")
             # For update operations, we need to get the old record
             try:
                 # Use _base_manager to avoid triggering hooks recursively

django_bulk_hooks/queryset.py

@@ -81,10 +81,10 @@ class HookQuerySetMixin:
         
         # If we're in a bulk operation context, skip hooks to prevent double execution
         if current_bypass_hooks:
-            logger.debug("update skipping hooks (bulk context)")
+            logger.debug("update: skipping hooks (bulk context)")
             ctx = HookContext(model_cls, bypass_hooks=True)
         else:
-            logger.debug("update running hooks (standalone)")
+            logger.debug("update: running hooks (standalone)")
             ctx = HookContext(model_cls, bypass_hooks=False)
             # Run validation hooks first
             engine.run(model_cls, VALIDATE_UPDATE, instances, originals, ctx=ctx)
@@ -118,10 +118,10 @@ class HookQuerySetMixin:
 
         # Run AFTER_UPDATE hooks only for standalone updates
         if not current_bypass_hooks:
-            logger.debug("update running AFTER_UPDATE")
+            logger.debug("update: running AFTER_UPDATE")
             engine.run(model_cls, AFTER_UPDATE, instances, originals, ctx=ctx)
         else:
-            logger.debug("update skipping AFTER_UPDATE (bulk context)")
+            logger.debug("update: skipping AFTER_UPDATE (bulk context)")
 
         return update_count
 
@@ -251,11 +251,11 @@ class HookQuerySetMixin:
                 break
 
         if not bypass_hooks:
-            logger.debug("bulk_update setting bypass_hooks=False (hooks will run in update())")
+            logger.debug("bulk_update: hooks will run in update()")
             ctx = HookContext(model_cls, bypass_hooks=False)
             originals = [None] * len(objs)  # Placeholder for after_update call
         else:
-            logger.debug("bulk_update setting bypass_hooks=True (no hooks)")
+            logger.debug("bulk_update: hooks bypassed")
             ctx = HookContext(model_cls, bypass_hooks=True)
             originals = [None] * len(objs)  # Ensure originals is defined for after_update call
 
@@ -289,9 +289,9 @@ class HookQuerySetMixin:
         # Note: We don't run AFTER_UPDATE hooks here to prevent double execution
         # The update() method will handle all hook execution based on thread-local state
         if not bypass_hooks:
-            logger.debug("bulk_update skipping AFTER_UPDATE (update() will handle)")
+            logger.debug("bulk_update: skipping AFTER_UPDATE (update() will handle)")
         else:
-            logger.debug("bulk_update bypassed hooks")
+            logger.debug("bulk_update: hooks bypassed")
 
         return result
 

django_bulk_hooks/registry.py

@@ -1,8 +1,11 @@
+import logging
 from collections.abc import Callable
 from typing import Union
 
 from django_bulk_hooks.priority import Priority
 
+logger = logging.getLogger(__name__)
+
 _hooks: dict[tuple[type, str], list[tuple[type, str, Callable, int]]] = {}
 
 
@@ -20,7 +23,9 @@ def register_hook(
 def get_hooks(model, event):
     key = (model, event)
     hooks = _hooks.get(key, [])
-    logger.debug(f"get_hooks {model.__name__}.{event} found {len(hooks)} hooks")
+    # Only log when hooks are found or for specific events to reduce noise
+    if hooks or event in ['after_update', 'before_update', 'after_create', 'before_create']:
+        logger.debug(f"get_hooks {model.__name__}.{event} found {len(hooks)} hooks")
     return hooks
 
 

{django_bulk_hooks-0.1.219.dist-info → django_bulk_hooks-0.1.221.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.219
+Version: 0.1.221
 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.219.dist-info → django_bulk_hooks-0.1.221.dist-info}/RECORD

@@ -1,5 +1,5 @@
 django_bulk_hooks/__init__.py,sha256=uUgpnb9AWjIAcWNpCMqBcOewSnpJjJYH6cjPbQkzoNU,140
-django_bulk_hooks/conditions.py,sha256=xHazd2hmSddQ3QcButFHjD4rG01tzD4hQMu8uoe7MRY,6513
+django_bulk_hooks/conditions.py,sha256=V_f3Di2uCVUjoyfiU4BQCHmI4uUIRSRroApDcXlvnso,6349
 django_bulk_hooks/constants.py,sha256=3x1H1fSUUNo0DZONN7GUVDuySZctTR-jtByBHmAIX5w,303
 django_bulk_hooks/context.py,sha256=_NbGWTq9s66g0vbFIaqN4GlIHWQmFg3EQ44qY8YvvEg,1537
 django_bulk_hooks/decorators.py,sha256=WD7Jn7QAvY8F4wOsYlIpjoM9-FdHXSKB7hH9ot-lkYQ,4896
@@ -7,11 +7,11 @@ django_bulk_hooks/engine.py,sha256=t_kvgex6_iZEFc5LK-srBTZPe-1bdlYdip5LfWOc6lc,2
 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=nfWiwU5-yAoxdnQsUMohxtyCpkV0MBv6X3wmipr9eQY,3697
-django_bulk_hooks/models.py,sha256=TA2dBIA1nJBiYt6joefWkpFIQZWysF9kZlkBYvEe59c,4358
+django_bulk_hooks/models.py,sha256=exnXYVKEVbYAXhChCP8VdWTnKCnm9DiTcokEIBee1I0,4350
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=H9toczyQ4KZUGSi-M594vF3BRaSVQX-tqKb6GHe7ua0,32648
-django_bulk_hooks/registry.py,sha256=N7ZG67KdM1Vz1jpEkSnOWzzine8fUkaVquBSPkZorrU,901
-django_bulk_hooks-0.1.219.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.219.dist-info/METADATA,sha256=zZC8QDWSEeetz38P_N83RqaTEufd99Qx2-U5Q9XpOJw,9061
-django_bulk_hooks-0.1.219.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.219.dist-info/RECORD,,
+django_bulk_hooks/queryset.py,sha256=YSDCMAf24YLeLJrRKCDYwbZInP2mK_Tcuw3EHLDkv_w,32605
+django_bulk_hooks/registry.py,sha256=8UuhniiH5ChSeOKV1UUbqTEiIu25bZXvcHmkaRbxmME,1131
+django_bulk_hooks-0.1.221.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.221.dist-info/METADATA,sha256=QVFXKdg3v7WcK1IsfDD3h3Nijg3tXdHtlqS6JoGemFo,9061
+django_bulk_hooks-0.1.221.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.221.dist-info/RECORD,,