django-bulk-hooks 0.1.82__tar.gz → 0.1.83__tar.gz

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Potentially problematic release.


This version of django-bulk-hooks might be problematic. Click here for more details.

@@ -1,6 +1,6 @@
1
1
  Metadata-Version: 2.3
2
2
  Name: django-bulk-hooks
3
- Version: 0.1.82
3
+ Version: 0.1.83
4
4
  Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
5
5
  License: MIT
6
6
  Keywords: django,bulk,hooks
@@ -57,32 +57,31 @@ class HookModelMixin(models.Model):
57
57
 
58
58
  def save(self, *args, **kwargs):
59
59
  is_create = self.pk is None
60
+ ctx = HookContext(self.__class__)
60
61
 
61
62
  if is_create:
62
- # For create operations, we don't have old records
63
- ctx = HookContext(self.__class__)
64
- run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
65
-
63
+ # For create operations, let Django save first so relationships are set up
66
64
  super().save(*args, **kwargs)
67
-
65
+
66
+ # Now run hooks after Django has set up the object
67
+ run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
68
68
  run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
69
69
  else:
70
70
  # For update operations, we need to get the old record
71
71
  try:
72
72
  old_instance = self.__class__.objects.get(pk=self.pk)
73
- ctx = HookContext(self.__class__)
74
- run(self.__class__, BEFORE_UPDATE, [self], [old_instance], ctx=ctx)
75
-
73
+
74
+ # Let Django save first
76
75
  super().save(*args, **kwargs)
77
-
76
+
77
+ # Now run hooks after Django has set up the object
78
+ run(self.__class__, BEFORE_UPDATE, [self], [old_instance], ctx=ctx)
78
79
  run(self.__class__, AFTER_UPDATE, [self], [old_instance], ctx=ctx)
79
80
  except self.__class__.DoesNotExist:
80
81
  # If the old instance doesn't exist, treat as create
81
- ctx = HookContext(self.__class__)
82
- run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
83
-
84
82
  super().save(*args, **kwargs)
85
-
83
+
84
+ run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
86
85
  run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
87
86
 
88
87
  return self
@@ -1,6 +1,6 @@
1
1
  [tool.poetry]
2
2
  name = "django-bulk-hooks"
3
- version = "0.1.82"
3
+ version = "0.1.83"
4
4
  description = "Hook-style hooks for Django bulk operations like bulk_create and bulk_update."
5
5
  authors = ["Konrad Beck <konrad.beck@merchantcapital.co.za>"]
6
6
  readme = "README.md"