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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.120
+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.120 → 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
             
@@ -370,8 +383,9 @@ class HookQuerySet(models.QuerySet):
         
         # Handle auto_now_add and auto_now fields like Django does
         for field in parent_model._meta.local_fields:
-            if field.__class__.pre_save is not field.__class__.__bases__[0].pre_save:
-                # This field has a custom pre_save method (like auto_now_add/auto_now)
+            if hasattr(field, 'auto_now_add') and field.auto_now_add:
+                field.pre_save(parent_obj, add=True)
+            elif hasattr(field, 'auto_now') and field.auto_now:
                 field.pre_save(parent_obj, add=True)
         
         return parent_obj
@@ -392,8 +406,9 @@ class HookQuerySet(models.QuerySet):
         
         # Handle auto_now_add and auto_now fields like Django does
         for field in child_model._meta.local_fields:
-            if field.__class__.pre_save is not field.__class__.__bases__[0].pre_save:
-                # This field has a custom pre_save method (like auto_now_add/auto_now)
+            if hasattr(field, 'auto_now_add') and field.auto_now_add:
+                field.pre_save(child_obj, add=True)
+            elif hasattr(field, 'auto_now') and field.auto_now:
                 field.pre_save(child_obj, add=True)
         
         return child_obj

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

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