django-bulk-hooks 0.1.166__tar.gz → 0.1.168__tar.gz

{django_bulk_hooks-0.1.166 → django_bulk_hooks-0.1.168}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.166
+Version: 0.1.168
 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.166 → django_bulk_hooks-0.1.168}/django_bulk_hooks/decorators.py

@@ -66,7 +66,8 @@ def select_related(*related_fields):
 
             fetched = {}
             if ids_to_fetch:
-                fetched = model_cls.objects.select_related(*related_fields).in_bulk(ids_to_fetch)
+                # Use the base manager to avoid recursion
+                fetched = model_cls._base_manager.select_related(*related_fields).in_bulk(ids_to_fetch)
 
             for obj in new_records:
                 preloaded = fetched.get(obj.pk)

{django_bulk_hooks-0.1.166 → django_bulk_hooks-0.1.168}/django_bulk_hooks/queryset.py

@@ -24,7 +24,8 @@ class HookQuerySet(models.QuerySet):
         objs = list(self)
         if not objs:
             return 0
-        return self.model.objects.bulk_delete(objs)
+        # Call the base QuerySet implementation to avoid recursion
+        return super().bulk_delete(objs)
 
     @transaction.atomic
     def update(self, **kwargs):
@@ -36,7 +37,8 @@ class HookQuerySet(models.QuerySet):
         pks = [obj.pk for obj in instances]
 
         # Load originals for hook comparison and ensure they match the order of instances
-        original_map = {obj.pk: obj for obj in model_cls.objects.filter(pk__in=pks)}
+        # Use the base manager to avoid recursion
+        original_map = {obj.pk: obj for obj in model_cls._base_manager.filter(pk__in=pks)}
         originals = [original_map.get(obj.pk) for obj in instances]
 
         # Apply field updates to instances
@@ -49,8 +51,8 @@ class HookQuerySet(models.QuerySet):
         engine.run(model_cls, BEFORE_UPDATE, instances, originals, ctx=ctx)
 
         # Use Django's built-in update logic directly
-        queryset = self.model.objects.filter(pk__in=pks)
-        update_count = queryset.update(**kwargs)
+        # Call the base QuerySet implementation to avoid recursion
+        update_count = super().update(**kwargs)
 
         # Run AFTER_UPDATE hooks
         engine.run(model_cls, AFTER_UPDATE, instances, originals, ctx=ctx)
@@ -202,9 +204,10 @@ class HookQuerySet(models.QuerySet):
 
         if not bypass_hooks:
             # Load originals for hook comparison and ensure they match the order of new instances
+            # Use the base manager to avoid recursion
             original_map = {
                 obj.pk: obj
-                for obj in model_cls.objects.filter(pk__in=[obj.pk for obj in objs])
+                for obj in model_cls._base_manager.filter(pk__in=[obj.pk for obj in objs])
             }
             originals = [original_map.get(obj.pk) for obj in objs]
 
@@ -274,9 +277,9 @@ class HookQuerySet(models.QuerySet):
 
         pks = [obj.pk for obj in objs if obj.pk is not None]
 
-        # Use base manager for the actual deletion to prevent recursion
+        # Call the base QuerySet implementation to avoid recursion
         # The hooks have already been fired above, so we don't need them again
-        model_cls._base_manager.filter(pk__in=pks).delete()
+        super().bulk_delete(objs, batch_size=batch_size)
 
         if not bypass_hooks:
             engine.run(model_cls, AFTER_DELETE, objs, ctx=ctx)

{django_bulk_hooks-0.1.166 → django_bulk_hooks-0.1.168}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.166"
+version = "0.1.168"
 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"