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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.91
+Version: 0.1.93
 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.93}/django_bulk_hooks/handler.py

@@ -142,10 +142,20 @@ class HookHandler(metaclass=HookMeta):
                 handler = handler_cls()
                 method = getattr(handler, method_name)
 
+                # Inspect the method signature to determine parameter order
+                import inspect
+                sig = inspect.signature(method)
+                params = list(sig.parameters.keys())
+                
+                # Remove 'self' from params if it exists
+                if params and params[0] == 'self':
+                    params = params[1:]
+
+                # Always call with keyword arguments to make order irrelevant
                 try:
                     method(
-                        new_records=new_local,
                         old_records=old_local,
+                        new_records=new_local,
                         **kwargs,
                     )
                 except Exception:

{django_bulk_hooks-0.1.91 → django_bulk_hooks-0.1.93}/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.93}/pyproject.toml

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