django-bulk-hooks 0.1.171__py3-none-any.whl → 0.1.172__py3-none-any.whl

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.

@@ -139,21 +139,17 @@ class HookQuerySet(models.QuerySet):
139
139
  )
140
140
  else:
141
141
  # For single-table models, use Django's built-in bulk_create
142
- # but we need to call it on the base manager to avoid recursion
143
142
  # Filter out custom parameters that Django's bulk_create doesn't accept
144
-
145
- # Use Django's original QuerySet to avoid recursive calls
146
- from django.db.models import QuerySet
147
-
148
- original_qs = QuerySet(model_cls, using=self.db)
149
- result = original_qs.bulk_create(
150
- objs,
151
- batch_size=batch_size,
152
- ignore_conflicts=ignore_conflicts,
153
- update_conflicts=update_conflicts,
154
- update_fields=update_fields,
155
- unique_fields=unique_fields,
156
- )
143
+ django_kwargs = {
144
+ "batch_size": batch_size,
145
+ "ignore_conflicts": ignore_conflicts,
146
+ "update_conflicts": update_conflicts,
147
+ "update_fields": update_fields,
148
+ "unique_fields": unique_fields,
149
+ }
150
+ # Remove custom hook kwargs if present
151
+ django_kwargs = {k: v for k, v in django_kwargs.items() if v is not None}
152
+ result = super().bulk_create(objs, **django_kwargs)
157
153
 
158
154
  # Fire AFTER_CREATE hooks
159
155
  if not bypass_hooks:
@@ -250,23 +246,9 @@ class HookQuerySet(models.QuerySet):
250
246
 
251
247
  pks = [obj.pk for obj in objs if obj.pk is not None]
252
248
 
253
- # Use Django's base manager to perform the actual deletion
254
- # This avoids recursion and uses Django's built-in delete logic
255
- from django.db.models import QuerySet
256
-
257
- base_qs = QuerySet(model_cls, using=self.db)
258
-
259
- # Delete in batches if batch_size is specified
260
- if batch_size:
261
- for i in range(0, len(objs), batch_size):
262
- batch = objs[i : i + batch_size]
263
- batch_pks = [obj.pk for obj in batch if obj.pk is not None]
264
- if batch_pks:
265
- base_qs.filter(pk__in=batch_pks).delete()
266
- else:
267
- # Delete all at once
268
- if pks:
269
- base_qs.filter(pk__in=pks).delete()
249
+ # Call the base QuerySet implementation to avoid recursion
250
+ # The hooks have already been fired above, so we don't need them again
251
+ super().bulk_delete(objs, batch_size=batch_size)
270
252
 
271
253
  if not bypass_hooks:
272
254
  engine.run(model_cls, AFTER_DELETE, objs, ctx=ctx)
@@ -1,6 +1,6 @@
1
1
  Metadata-Version: 2.1
2
2
  Name: django-bulk-hooks
3
- Version: 0.1.171
3
+ Version: 0.1.172
4
4
  Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
5
5
  Home-page: https://github.com/AugendLimited/django-bulk-hooks
6
6
  License: MIT
@@ -9,9 +9,9 @@ django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,
9
9
  django_bulk_hooks/manager.py,sha256=OSzW8eVzknLV1WCvZcBkWMz9x_Vjq4bJM8raVXKiZvI,5085
10
10
  django_bulk_hooks/models.py,sha256=7fnx5xd4HWXfLVlFhhiRzR92JRWFEuxgk6aSWLEsyJg,3996
11
11
  django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
12
- django_bulk_hooks/queryset.py,sha256=dHfZDjOPi5G4kY8VrC-M3ThjXTrh4Ik_7DN8sev-4I8,23826
12
+ django_bulk_hooks/queryset.py,sha256=yGzVFWWbg7mjJxH9Gx1hXw9Mf82H0XmMlfKZn47WMJw,23222
13
13
  django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
14
- django_bulk_hooks-0.1.171.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
15
- django_bulk_hooks-0.1.171.dist-info/METADATA,sha256=it1WojkivQKXX8ubgT-SFxiKfrKaV9GXh8rC8azUfL4,6939
16
- django_bulk_hooks-0.1.171.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
17
- django_bulk_hooks-0.1.171.dist-info/RECORD,,
14
+ django_bulk_hooks-0.1.172.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
15
+ django_bulk_hooks-0.1.172.dist-info/METADATA,sha256=JQDexi4jxTyskLLwCDmRemDvHPeihDdkNZOJSohsLjk,6939
16
+ django_bulk_hooks-0.1.172.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
17
+ django_bulk_hooks-0.1.172.dist-info/RECORD,,