django-bulk-hooks 0.1.265__tar.gz → 0.1.266__tar.gz

{django_bulk_hooks-0.1.265 → django_bulk_hooks-0.1.266}/PKG-INFO

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

@@ -1122,6 +1122,20 @@ class HookQuerySetMixin:
             # Filter out parameters that are not supported by Django's bulk_update
             unsupported_params = ["unique_fields", "update_conflicts", "update_fields", "ignore_conflicts"]
             django_kwargs = {}
+
+            # Check if all objects have primary keys set before proceeding
+            if not all(obj._is_pk_set() for obj in objs):
+                missing_pk_count = sum(1 for obj in objs if not obj._is_pk_set())
+                logger.error(
+                    f"bulk_update failed: {missing_pk_count} out of {len(objs)} objects don't have primary keys set. "
+                    "All objects must be saved to the database before bulk_update can be used."
+                )
+                print(f"ERROR: {missing_pk_count} objects don't have primary keys set")
+                raise ValueError(
+                    f"All bulk_update() objects must have a primary key set. "
+                    f"{missing_pk_count} out of {len(objs)} objects are missing primary keys."
+                )
+
             for k, v in kwargs.items():
                 if k in unsupported_params:
                     logger.warning(
@@ -1490,6 +1504,19 @@ class HookQuerySetMixin:
         if inheritance_chain is None:
             inheritance_chain = self._get_inheritance_chain()
 
+        # Check if all objects have primary keys set before proceeding
+        if not all(obj._is_pk_set() for obj in objs):
+            missing_pk_count = sum(1 for obj in objs if not obj._is_pk_set())
+            logger.error(
+                f"MTI bulk_update failed: {missing_pk_count} out of {len(objs)} objects don't have primary keys set. "
+                "All objects must be saved to the database before bulk_update can be used."
+            )
+            print(f"ERROR: {missing_pk_count} objects don't have primary keys set")
+            raise ValueError(
+                f"All bulk_update() objects must have a primary key set. "
+                f"{missing_pk_count} out of {len(objs)} objects are missing primary keys."
+            )
+
         # Remove custom hook kwargs and unsupported parameters before passing to Django internals
         unsupported_params = ["unique_fields", "update_conflicts", "update_fields", "ignore_conflicts"]
         django_kwargs = {}

{django_bulk_hooks-0.1.265 → django_bulk_hooks-0.1.266}/pyproject.toml

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