django-bulk-hooks 0.1.91__tar.gz → 0.1.92__tar.gz

{django_bulk_hooks-0.1.91 → django_bulk_hooks-0.1.92}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.91
+Version: 0.1.92
 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.91 → django_bulk_hooks-0.1.92}/django_bulk_hooks/models.py

@@ -91,34 +91,27 @@ class HookModelMixin(models.Model):
         is_create = self.pk is None
         ctx = HookContext(self.__class__)
 
-        # Use a single context manager for all hooks
+        # Let Django save first to handle form validation
+        super().save(*args, **kwargs)
+
+        # Then run our hooks with the validated data
         with patch_foreign_key_behavior():
             if is_create:
-                # For create operations, run validation hooks first
+                # For create operations
                 run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
-                # Then run BEFORE hooks
                 run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
-                # Now save
-                super().save(*args, **kwargs)
-                # Finally run AFTER hooks
                 run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
             else:
                 # For update operations
                 try:
                     old_instance = self.__class__.objects.get(pk=self.pk)
-                    # Run validation hooks first
                     run(self.__class__, VALIDATE_UPDATE, [self], [old_instance], ctx=ctx)
-                    # Then run BEFORE hooks
                     run(self.__class__, BEFORE_UPDATE, [self], [old_instance], ctx=ctx)
-                    # Now save
-                    super().save(*args, **kwargs)
-                    # Finally run AFTER hooks
                     run(self.__class__, AFTER_UPDATE, [self], [old_instance], ctx=ctx)
                 except self.__class__.DoesNotExist:
                     # If the old instance doesn't exist, treat as create
                     run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
                     run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
-                    super().save(*args, **kwargs)
                     run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
 
         return self

{django_bulk_hooks-0.1.91 → django_bulk_hooks-0.1.92}/pyproject.toml

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