django-bulk-hooks 0.1.121__tar.gz → 0.1.122__tar.gz

{django_bulk_hooks-0.1.121 → django_bulk_hooks-0.1.122}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.121
+Version: 0.1.122
 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.121 → django_bulk_hooks-0.1.122}/django_bulk_hooks/queryset.py

@@ -284,6 +284,10 @@ class HookQuerySet(models.QuerySet):
         if inheritance_chain is None:
             inheritance_chain = self._get_inheritance_chain()
             
+        # Safety check to prevent infinite recursion
+        if len(inheritance_chain) > 10:  # Arbitrary limit to prevent infinite loops
+            raise ValueError("Inheritance chain too deep - possible infinite recursion detected")
+            
         batch_size = kwargs.get("batch_size") or len(objs)
         created_objects = []
         with transaction.atomic(using=self.db, savepoint=False):
@@ -323,19 +327,28 @@ class HookQuerySet(models.QuerySet):
         # If the child model is still MTI, call our own logic recursively
         if len([p for p in child_model._meta.parents.keys() if not p._meta.proxy]) > 0:
             # Build inheritance chain for the child model
-            inheritance_chain = []
+            child_inheritance_chain = []
             current_model = child_model
             while current_model:
                 if not current_model._meta.proxy:
-                    inheritance_chain.append(current_model)
+                    child_inheritance_chain.append(current_model)
                 parents = [
                     parent
                     for parent in current_model._meta.parents.keys()
                     if not parent._meta.proxy
                 ]
                 current_model = parents[0] if parents else None
-            inheritance_chain.reverse()
-            created = self._mti_bulk_create(child_objects, inheritance_chain, **kwargs)
+            child_inheritance_chain.reverse()
+            
+            # For nested MTI, we can't use bulk operations recursively
+            # because it would create infinite recursion. Instead, we save each child individually.
+            # We use super().save() to avoid triggering hooks that would cause recursion.
+            created = []
+            for child_obj in child_objects:
+                # Use the base model's save method to avoid triggering hooks
+                # This prevents infinite recursion when hooks try to query the database
+                super(child_obj.__class__, child_obj).save()
+                created.append(child_obj)
         else:
             # Single-table, safe to use bulk_create
             

{django_bulk_hooks-0.1.121 → django_bulk_hooks-0.1.122}/pyproject.toml

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