django-bulk-hooks 0.1.120__py3-none-any.whl → 0.1.122__py3-none-any.whl

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.dist-info → django_bulk_hooks-0.1.122.dist-info}/METADATA

@@ -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.dist-info → django_bulk_hooks-0.1.122.dist-info}/RECORD

@@ -9,9 +9,9 @@ django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,
 django_bulk_hooks/manager.py,sha256=DjEW-nZjhlBW6cp8GRPl6xOSsAmmquP0Y-QyCZMoSHo,6946
 django_bulk_hooks/models.py,sha256=7RG7GrOdHXFjGVPV4FPRZVNMIHHW-hMCi6hn9LH_hVI,3331
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=zt6XToualdlA1MK_VT2ILovblZaKtul_elZSXnpk8hE,15885
+django_bulk_hooks/queryset.py,sha256=19uG70BxNAJwLqu3j3ZmT4DStX8sPRwbV9079m0yn84,16717
 django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
-django_bulk_hooks-0.1.120.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.120.dist-info/METADATA,sha256=VE12uNcfnuW92FhLRv9KealQLko0FPpy0OQKsDwJ6II,6951
-django_bulk_hooks-0.1.120.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.120.dist-info/RECORD,,
+django_bulk_hooks-0.1.122.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.122.dist-info/METADATA,sha256=yR16rQOflmIN7AKOrewUgbdHZUCA7-beKcHxJaa_tQ8,6951
+django_bulk_hooks-0.1.122.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.122.dist-info/RECORD,,