django-bulk-hooks 0.2.41__py3-none-any.whl → 0.2.42__py3-none-any.whl

django_bulk_hooks/operations/mti_handler.py

@@ -259,10 +259,11 @@ class MTIHandler:
                         uf for uf in (update_fields or []) if uf in model_fields_by_name
                     ]
 
-                    if filtered_updates:
-                        level_update_conflicts = True
-                        level_unique_fields = normalized_unique
-                        level_update_fields = filtered_updates
+                    # Enable upsert even if no fields to update at this level
+                    # This prevents unique constraint violations on parent tables
+                    level_update_conflicts = True
+                    level_unique_fields = normalized_unique
+                    level_update_fields = filtered_updates  # Can be empty list
 
             # Create parent level
             parent_level = ParentLevel(

django_bulk_hooks/queryset.py

@@ -1,191 +1,188 @@
-"""
-HookQuerySet - Django QuerySet with hook support.
-
-This is a thin coordinator that delegates all complex logic to services.
-It follows the Facade pattern, providing a simple interface over the
-complex coordination required for bulk operations with hooks.
-"""
-
-import logging
-
-from django.db import models
-from django.db import transaction
-
-logger = logging.getLogger(__name__)
-
-
-class HookQuerySet(models.QuerySet):
-    """
-    QuerySet with hook support.
-
-    This is a thin facade over BulkOperationCoordinator. It provides
-    backward-compatible API for Django's QuerySet while integrating
-    the full hook lifecycle.
-
-    Key design principles:
-    - Minimal logic (< 10 lines per method)
-    - No business logic (delegate to coordinator)
-    - No conditionals (let services handle it)
-    - Transaction boundaries only
-    """
-
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self._coordinator = None
-
-    @property
-    def coordinator(self):
-        """Lazy initialization of coordinator"""
-        if self._coordinator is None:
-            from django_bulk_hooks.operations import BulkOperationCoordinator
-
-            self._coordinator = BulkOperationCoordinator(self)
-        return self._coordinator
-
-    @transaction.atomic
-    def bulk_create(
-        self,
-        objs,
-        batch_size=None,
-        ignore_conflicts=False,
-        update_conflicts=False,
-        update_fields=None,
-        unique_fields=None,
-        bypass_hooks=False,
-        bypass_validation=False,
-    ):
-        """
-        Create multiple objects with hook support.
-
-        This is the public API - delegates to coordinator.
-        """
-        return self.coordinator.create(
-            objs=objs,
-            batch_size=batch_size,
-            ignore_conflicts=ignore_conflicts,
-            update_conflicts=update_conflicts,
-            update_fields=update_fields,
-            unique_fields=unique_fields,
-            bypass_hooks=bypass_hooks,
-            bypass_validation=bypass_validation,
-        )
-
-    @transaction.atomic
-    def bulk_update(
-        self,
-        objs,
-        fields=None,
-        batch_size=None,
-        bypass_hooks=False,
-        bypass_validation=False,
-        **kwargs,
-    ):
-        """
-        Update multiple objects with hook support.
-
-        This is the public API - delegates to coordinator.
-
-        Args:
-            objs: List of model instances to update
-            fields: List of field names to update (optional, will auto-detect if None)
-            batch_size: Number of objects per batch
-            bypass_hooks: Skip all hooks if True
-            bypass_validation: Skip validation hooks if True
-
-        Returns:
-            Number of objects updated
-        """
-        # If fields is None, auto-detect changed fields using analyzer
-        if fields is None:
-            fields = self.coordinator.analyzer.detect_changed_fields(objs)
-            if not fields:
-                logger.debug(
-                    f"bulk_update: No fields changed for {len(objs)} {self.model.__name__} objects",
-                )
-                return 0
-
-        return self.coordinator.update(
-            objs=objs,
-            fields=fields,
-            batch_size=batch_size,
-            bypass_hooks=bypass_hooks,
-            bypass_validation=bypass_validation,
-        )
-
-    @transaction.atomic
-    def update(self, bypass_hooks=False, bypass_validation=False, **kwargs):
-        """
-        Update QuerySet with hook support.
-
-        This is the public API - delegates to coordinator.
-
-        Args:
-            bypass_hooks: Skip all hooks if True
-            bypass_validation: Skip validation hooks if True
-            **kwargs: Fields to update
-
-        Returns:
-            Number of objects updated
-        """
-        return self.coordinator.update_queryset(
-            update_kwargs=kwargs,
-            bypass_hooks=bypass_hooks,
-            bypass_validation=bypass_validation,
-        )
-
-    @transaction.atomic
-    def bulk_delete(
-        self, objs, bypass_hooks=False, bypass_validation=False, **kwargs,
-    ):
-        """
-        Delete multiple objects with hook support.
-
-        This is the public API - delegates to coordinator.
-
-        Args:
-            objs: List of objects to delete
-            bypass_hooks: Skip all hooks if True
-            bypass_validation: Skip validation hooks if True
-
-        Returns:
-            Tuple of (count, details dict)
-        """
-        # Filter queryset to only these objects
-        pks = [obj.pk for obj in objs if obj.pk is not None]
-        if not pks:
-            return 0
-
-        # Create a filtered queryset
-        filtered_qs = self.filter(pk__in=pks)
-
-        # Use coordinator with the filtered queryset
-        from django_bulk_hooks.operations import BulkOperationCoordinator
-
-        coordinator = BulkOperationCoordinator(filtered_qs)
-
-        count, details = coordinator.delete(
-            bypass_hooks=bypass_hooks,
-            bypass_validation=bypass_validation,
-        )
-
-        # For bulk_delete, return just the count to match Django's behavior
-        return count
-
-    @transaction.atomic
-    def delete(self, bypass_hooks=False, bypass_validation=False):
-        """
-        Delete QuerySet with hook support.
-
-        This is the public API - delegates to coordinator.
-
-        Args:
-            bypass_hooks: Skip all hooks if True
-            bypass_validation: Skip validation hooks if True
-
-        Returns:
-            Tuple of (count, details dict)
-        """
-        return self.coordinator.delete(
-            bypass_hooks=bypass_hooks,
-            bypass_validation=bypass_validation,
-        )
+"""
+HookQuerySet - Django QuerySet with hook support.
+
+This is a thin coordinator that delegates all complex logic to services.
+It follows the Facade pattern, providing a simple interface over the
+complex coordination required for bulk operations with hooks.
+"""
+
+import logging
+
+from django.db import models
+from django.db import transaction
+
+logger = logging.getLogger(__name__)
+
+
+class HookQuerySet(models.QuerySet):
+    """
+    QuerySet with hook support.
+
+    This is a thin facade over BulkOperationCoordinator. It provides
+    backward-compatible API for Django's QuerySet while integrating
+    the full hook lifecycle.
+
+    Key design principles:
+    - Minimal logic (< 10 lines per method)
+    - No business logic (delegate to coordinator)
+    - No conditionals (let services handle it)
+    - Transaction boundaries only
+    """
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self._coordinator = None
+
+    @property
+    def coordinator(self):
+        """Lazy initialization of coordinator"""
+        if self._coordinator is None:
+            from django_bulk_hooks.operations import BulkOperationCoordinator
+
+            self._coordinator = BulkOperationCoordinator(self)
+        return self._coordinator
+
+    @transaction.atomic
+    def bulk_create(
+        self,
+        objs,
+        batch_size=None,
+        ignore_conflicts=False,
+        update_conflicts=False,
+        update_fields=None,
+        unique_fields=None,
+        bypass_hooks=False,
+        bypass_validation=False,
+    ):
+        """
+        Create multiple objects with hook support.
+
+        This is the public API - delegates to coordinator.
+        """
+        return self.coordinator.create(
+            objs=objs,
+            batch_size=batch_size,
+            ignore_conflicts=ignore_conflicts,
+            update_conflicts=update_conflicts,
+            update_fields=update_fields,
+            unique_fields=unique_fields,
+            bypass_hooks=bypass_hooks,
+            bypass_validation=bypass_validation,
+        )
+
+    @transaction.atomic
+    def bulk_update(
+        self,
+        objs,
+        fields=None,
+        batch_size=None,
+        bypass_hooks=False,
+        bypass_validation=False,
+        **kwargs,
+    ):
+        """
+        Update multiple objects with hook support.
+
+        This is the public API - delegates to coordinator.
+
+        Args:
+            objs: List of model instances to update
+            fields: List of field names to update (optional, will auto-detect if None)
+            batch_size: Number of objects per batch
+            bypass_hooks: Skip all hooks if True
+            bypass_validation: Skip validation hooks if True
+
+        Returns:
+            Number of objects updated
+        """
+        # If fields is None, auto-detect changed fields using analyzer
+        if fields is None:
+            fields = self.coordinator.analyzer.detect_changed_fields(objs)
+            if not fields:
+                return 0
+
+        return self.coordinator.update(
+            objs=objs,
+            fields=fields,
+            batch_size=batch_size,
+            bypass_hooks=bypass_hooks,
+            bypass_validation=bypass_validation,
+        )
+
+    @transaction.atomic
+    def update(self, bypass_hooks=False, bypass_validation=False, **kwargs):
+        """
+        Update QuerySet with hook support.
+
+        This is the public API - delegates to coordinator.
+
+        Args:
+            bypass_hooks: Skip all hooks if True
+            bypass_validation: Skip validation hooks if True
+            **kwargs: Fields to update
+
+        Returns:
+            Number of objects updated
+        """
+        return self.coordinator.update_queryset(
+            update_kwargs=kwargs,
+            bypass_hooks=bypass_hooks,
+            bypass_validation=bypass_validation,
+        )
+
+    @transaction.atomic
+    def bulk_delete(
+        self, objs, bypass_hooks=False, bypass_validation=False, **kwargs,
+    ):
+        """
+        Delete multiple objects with hook support.
+
+        This is the public API - delegates to coordinator.
+
+        Args:
+            objs: List of objects to delete
+            bypass_hooks: Skip all hooks if True
+            bypass_validation: Skip validation hooks if True
+
+        Returns:
+            Tuple of (count, details dict)
+        """
+        # Filter queryset to only these objects
+        pks = [obj.pk for obj in objs if obj.pk is not None]
+        if not pks:
+            return 0
+
+        # Create a filtered queryset
+        filtered_qs = self.filter(pk__in=pks)
+
+        # Use coordinator with the filtered queryset
+        from django_bulk_hooks.operations import BulkOperationCoordinator
+
+        coordinator = BulkOperationCoordinator(filtered_qs)
+
+        count, details = coordinator.delete(
+            bypass_hooks=bypass_hooks,
+            bypass_validation=bypass_validation,
+        )
+
+        # For bulk_delete, return just the count to match Django's behavior
+        return count
+
+    @transaction.atomic
+    def delete(self, bypass_hooks=False, bypass_validation=False):
+        """
+        Delete QuerySet with hook support.
+
+        This is the public API - delegates to coordinator.
+
+        Args:
+            bypass_hooks: Skip all hooks if True
+            bypass_validation: Skip validation hooks if True
+
+        Returns:
+            Tuple of (count, details dict)
+        """
+        return self.coordinator.delete(
+            bypass_hooks=bypass_hooks,
+            bypass_validation=bypass_validation,
+        )