django-bulk-hooks 0.1.68__tar.gz → 0.1.70__tar.gz

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-bulk-hooks
-Version: 0.1.68
+Version: 0.1.70
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 Home-page: https://github.com/AugendLimited/django-bulk-hooks
 License: MIT
@@ -54,11 +54,11 @@ class Account(HookModelMixin):
 ### Create a Hook Handler
 
 ```python
-from django_bulk_hooks import hook, AFTER_UPDATE, HookHandler
+from django_bulk_hooks import hook, AFTER_UPDATE, Hook
 from django_bulk_hooks.conditions import WhenFieldHasChanged
 from .models import Account
 
-class AccountHookHandler(HookHandler):
+class AccountHooks(Hook):
     @hook(AFTER_UPDATE, model=Account, condition=WhenFieldHasChanged("balance"))
     def log_balance_change(self, new_records, old_records):
         print("Accounts updated:", [a.pk for a in new_records])
@@ -166,7 +166,7 @@ Account.objects.bulk_delete(accounts)
 ### Advanced Hook Usage
 
 ```python
-class AdvancedAccountHandler(HookHandler):
+class AdvancedAccountHooks(Hook):
     @hook(BEFORE_UPDATE, model=Account, condition=WhenFieldHasChanged("balance"))
     def validate_balance_change(self, new_records, old_records):
         for new_account, old_account in zip(new_records, old_records):

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/README.md

@@ -35,11 +35,11 @@ class Account(HookModelMixin):
 ### Create a Hook Handler
 
 ```python
-from django_bulk_hooks import hook, AFTER_UPDATE, HookHandler
+from django_bulk_hooks import hook, AFTER_UPDATE, Hook
 from django_bulk_hooks.conditions import WhenFieldHasChanged
 from .models import Account
 
-class AccountHookHandler(HookHandler):
+class AccountHooks(Hook):
     @hook(AFTER_UPDATE, model=Account, condition=WhenFieldHasChanged("balance"))
     def log_balance_change(self, new_records, old_records):
         print("Accounts updated:", [a.pk for a in new_records])
@@ -147,7 +147,7 @@ Account.objects.bulk_delete(accounts)
 ### Advanced Hook Usage
 
 ```python
-class AdvancedAccountHandler(HookHandler):
+class AdvancedAccountHooks(Hook):
     @hook(BEFORE_UPDATE, model=Account, condition=WhenFieldHasChanged("balance"))
     def validate_balance_change(self, new_records, old_records):
         for new_account, old_account in zip(new_records, old_records):

django_bulk_hooks-0.1.70/django_bulk_hooks/__init__.py

@@ -0,0 +1,4 @@
+from django_bulk_hooks.handler import Hook
+from django_bulk_hooks.manager import BulkHookManager
+
+__all__ = ["BulkHookManager", "Hook"]

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/conditions.py

@@ -1,8 +1,3 @@
-import logging
-
-logger = logging.getLogger(__name__)
-
-
 def resolve_dotted_attr(instance, dotted_path):
     """
     Recursively resolve a dotted attribute path, e.g., "type.category".
@@ -39,12 +34,6 @@ class IsNotEqual(HookCondition):
 
     def check(self, instance, original_instance=None):
         current = resolve_dotted_attr(instance, self.field)
-        logger.debug(
-            "%s current=%r, original=%r",
-            self.field,
-            current,
-            resolve_dotted_attr(original_instance, self.field) if original_instance else None,
-        )
         if self.only_on_change:
             if original_instance is None:
                 return False
@@ -62,12 +51,6 @@ class IsEqual(HookCondition):
 
     def check(self, instance, original_instance=None):
         current = resolve_dotted_attr(instance, self.field)
-        logger.debug(
-            "%s current=%r, original=%r",
-            self.field,
-            current,
-            resolve_dotted_attr(original_instance, self.field) if original_instance else None,
-        )
         if self.only_on_change:
             if original_instance is None:
                 return False

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/constants.py

@@ -4,8 +4,6 @@ BEFORE_UPDATE = "before_update"
 AFTER_UPDATE = "after_update"
 BEFORE_DELETE = "before_delete"
 AFTER_DELETE = "after_delete"
-
-# Validation constants - run during clean() for admin/form validation
 VALIDATE_CREATE = "validate_create"
 VALIDATE_UPDATE = "validate_update"
 VALIDATE_DELETE = "validate_delete"

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/engine.py

@@ -1,5 +1,7 @@
 import logging
+
 from django.core.exceptions import ValidationError
+
 from django_bulk_hooks.registry import get_hooks
 
 logger = logging.getLogger(__name__)
@@ -8,22 +10,11 @@ logger = logging.getLogger(__name__)
 def run(model_cls, event, new_instances, original_instances=None, ctx=None):
     hooks = get_hooks(model_cls, event)
 
-    logger.info(
-        "Executing engine.run: model=%s, event=%s, #new_instances=%d, #original_instances=%d, #hooks=%d",
-        model_cls.__name__,
-        event,
-        len(new_instances),
-        len(original_instances or []),
-        len(hooks),
-    )
-
     if not hooks:
-        logger.info("No hooks found for model=%s, event=%s", model_cls.__name__, event)
         return
 
     # For BEFORE_* events, run model.clean() first for validation
     if event.startswith("before_"):
-        logger.debug("Running model.clean() for %d instances", len(new_instances))
         for instance in new_instances:
             try:
                 instance.clean()
@@ -35,14 +26,6 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
         handler_instance = handler_cls()
         func = getattr(handler_instance, method_name)
 
-        logger.info(
-            "Executing hook %s for %s.%s with priority=%s",
-            func.__name__,
-            model_cls.__name__,
-            event,
-            priority,
-        )
-
         to_process_new = []
         to_process_old = []
 
@@ -51,28 +34,10 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
             original_instances or [None] * len(new_instances),
             strict=True,
         ):
-            logger.debug(
-                "  considering instance: new=%r, original=%r",
-                new,
-                original,
-            )
-
             if not condition or condition.check(new, original):
                 to_process_new.append(new)
                 to_process_old.append(original)
-                logger.debug("    -> will process (passed condition)")
-            else:
-                logger.debug("    -> skipped (condition returned False)")
 
         if to_process_new:
-            logger.info(
-                "Calling %s on %d instance(s): %r",
-                func.__name__,
-                len(to_process_new),
-                to_process_new,
-            )
-
             # Call the function with direct arguments
             func(to_process_new, to_process_old if any(to_process_old) else None)
-        else:
-            logger.debug("No instances to process for hook %s", func.__name__)

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/handler.py

@@ -64,7 +64,7 @@ class HookContextState:
 Hook = HookContextState()
 
 
-class HookHandlerMeta(type):
+class HookMeta(type):
     _registered = set()
 
     def __new__(mcs, name, bases, namespace):
@@ -73,14 +73,7 @@ class HookHandlerMeta(type):
             if hasattr(method, "hooks_hooks"):
                 for model_cls, event, condition, priority in method.hooks_hooks:
                     key = (model_cls, event, cls, method_name)
-                    if key not in HookHandlerMeta._registered:
-                        logger.info(
-                            "Registering hook via HookHandlerMeta: model=%s, event=%s, handler_cls=%s, method_name=%s",
-                            model_cls.__name__,
-                            event,
-                            cls.__name__,
-                            method_name,
-                        )
+                    if key not in HookMeta._registered:
                         register_hook(
                             model=model_cls,
                             event=event,
@@ -89,11 +82,11 @@ class HookHandlerMeta(type):
                             condition=condition,
                             priority=priority,
                         )
-                        HookHandlerMeta._registered.add(key)
+                        HookMeta._registered.add(key)
         return cls
 
 
-class HookHandler(metaclass=HookHandlerMeta):
+class Hook(metaclass=HookMeta):
     @classmethod
     def handle(
         cls,
@@ -131,7 +124,6 @@ class HookHandler(metaclass=HookHandlerMeta):
         hook_vars.model = model
 
         hooks = sorted(get_hooks(model, event), key=lambda x: x[3])
-        logger.debug("Processing %d hooks for %s.%s", len(hooks), model.__name__, event)
 
         def _execute():
             new_local = new_records or []
@@ -150,12 +142,6 @@ class HookHandler(metaclass=HookHandlerMeta):
                 handler = handler_cls()
                 method = getattr(handler, method_name)
 
-                logger.info(
-                    "Running hook %s.%s on %d items",
-                    handler_cls.__name__,
-                    method_name,
-                    len(new_local),
-                )
                 try:
                     method(
                         new_records=new_local,

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/manager.py

@@ -1,7 +1,4 @@
-import logging
-
 from django.db import models, transaction
-
 from django_bulk_hooks import engine
 from django_bulk_hooks.constants import (
     AFTER_CREATE,
@@ -17,8 +14,6 @@ from django_bulk_hooks.constants import (
 from django_bulk_hooks.context import HookContext
 from django_bulk_hooks.queryset import HookQuerySet
 
-logger = logging.getLogger(__name__)
-
 
 class BulkHookManager(models.Manager):
     CHUNK_SIZE = 200
@@ -56,10 +51,6 @@ class BulkHookManager(models.Manager):
                 fields_set = set(fields)
                 fields_set.update(modified_fields)
                 fields = list(fields_set)
-                logger.info(
-                    "Automatically including modified fields in bulk_update: %s",
-                    modified_fields,
-                )
 
         for i in range(0, len(objs), self.CHUNK_SIZE):
             chunk = objs[i : i + self.CHUNK_SIZE]
@@ -159,9 +150,6 @@ class BulkHookManager(models.Manager):
         ctx = HookContext(model_cls)
 
         if not bypass_hooks:
-            logger.info("Executing hooks for %s", model_cls.__name__)
-            logger.info("Number of objects to delete: %d", len(objs))
-            
             # Run validation hooks first
             if not bypass_validation:
                 engine.run(model_cls, VALIDATE_DELETE, objs, ctx=ctx)
@@ -173,8 +161,6 @@ class BulkHookManager(models.Manager):
         model_cls.objects.filter(pk__in=pks).delete()
 
         if not bypass_hooks:
-            logger.info("Executing AFTER_DELETE hooks for %s", model_cls.__name__)
-            logger.info("Number of objects deleted: %d", len(objs))
             engine.run(model_cls, AFTER_DELETE, objs, ctx=ctx)
 
         return objs

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/django_bulk_hooks/registry.py

@@ -1,11 +1,8 @@
-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]]] = {}
 
 
@@ -17,29 +14,13 @@ def register_hook(
     hooks.append((handler_cls, method_name, condition, priority))
     # keep sorted by priority
     hooks.sort(key=lambda x: x[3])
-    logger.info(
-        "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):
     hooks = _hooks.get((model, event), [])
-    logger.info(
-        "Retrieving hooks: model=%s, event=%s, hooks_found=%d",
-        model.__name__,
-        event,
-        len(hooks),
-    )
     return hooks
 
 
 def list_all_hooks():
     """Debug function to list all registered hooks"""
-    logger.debug("All registered hooks: %s", _hooks)
     return _hooks

{django_bulk_hooks-0.1.68 → django_bulk_hooks-0.1.70}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.68"
+version = "0.1.70"
 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.68/django_bulk_hooks/__init__.py

@@ -1,4 +0,0 @@
-from django_bulk_hooks.handler import HookHandler
-from django_bulk_hooks.manager import BulkHookManager
-
-__all__ = ["BulkHookManager", "HookHandler"]