django-bulk-hooks 0.1.207__py3-none-any.whl → 0.1.209__py3-none-any.whl

django_bulk_hooks/engine.py

@@ -20,6 +20,10 @@ def run(model_cls, event, new_records, old_records=None, ctx=None):
     if not hooks:
         return
 
+    print(
+        f"DEBUG: engine.run called for {model_cls.__name__}.{event} with {len(new_records)} records"
+    )
+
     # For BEFORE_* events, run model.clean() first for validation
     if event.startswith("before_"):
         for instance in new_records:
@@ -47,6 +51,9 @@ def run(model_cls, event, new_records, old_records=None, ctx=None):
                 to_process_old.append(original)
 
         if to_process_new:
+            print(
+                f"DEBUG: Executing hook {handler_cls.__name__}.{method_name} for {len(to_process_new)} records"
+            )
             try:
                 func(
                     new_records=to_process_new,

django_bulk_hooks/models.py

@@ -56,11 +56,15 @@ class HookModelMixin(models.Model):
     def save(self, *args, bypass_hooks=False, **kwargs):
         # If bypass_hooks is True, use base manager to avoid triggering hooks
         if bypass_hooks:
+            print(
+                f"DEBUG: save() called with bypass_hooks=True for {self.__class__.__name__} pk={self.pk}"
+            )
             return self._base_manager.save(self, *args, **kwargs)
 
         is_create = self.pk is None
 
         if is_create:
+            print(f"DEBUG: save() creating new {self.__class__.__name__} instance")
             # For create operations, we don't have old records
             ctx = HookContext(self.__class__)
             run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
@@ -69,6 +73,9 @@ class HookModelMixin(models.Model):
 
             run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
         else:
+            print(
+                f"DEBUG: save() updating existing {self.__class__.__name__} instance pk={self.pk}"
+            )
             # For update operations, we need to get the old record
             try:
                 # Use _base_manager to avoid triggering hooks recursively

django_bulk_hooks/queryset.py

@@ -82,23 +82,24 @@ class HookQuerySetMixin:
 
         # If we used Subquery objects, refresh the instances to get computed values
         if has_subquery and instances:
-            # Single bulk query to get all updated values
-            # Use values() to get only the fields we need for maximum performance
-            field_names = [f.name for f in model_cls._meta.fields if f.name != "id"]
-            refreshed_data = {
-                obj["pk"]: obj
-                for obj in model_cls._base_manager.filter(pk__in=pks).values(
-                    "pk", *field_names
-                )
+            # Simple refresh of model fields without fetching related objects
+            # Subquery updates only affect the model's own fields, not relationships
+            refreshed_instances = {
+                obj.pk: obj for obj in model_cls._base_manager.filter(pk__in=pks)
             }
 
             # Bulk update all instances in memory
             for instance in instances:
-                if instance.pk in refreshed_data:
-                    data = refreshed_data[instance.pk]
-                    # Bulk copy all field values except primary key
-                    for field_name in field_names:
-                        setattr(instance, field_name, data[field_name])
+                if instance.pk in refreshed_instances:
+                    refreshed_instance = refreshed_instances[instance.pk]
+                    # Update all fields except primary key
+                    for field in model_cls._meta.fields:
+                        if field.name != "id":
+                            setattr(
+                                instance,
+                                field.name,
+                                getattr(refreshed_instance, field.name),
+                            )
 
         # Run AFTER_UPDATE hooks
         engine.run(model_cls, AFTER_UPDATE, instances, originals, ctx=ctx)
@@ -226,6 +227,7 @@ class HookQuerySetMixin:
                 break
 
         if not bypass_hooks:
+            print(f"DEBUG: bypass_hooks=False, running hooks for {len(objs)} objects")
             # Load originals for hook comparison
             original_map = {
                 obj.pk: obj
@@ -243,6 +245,8 @@ class HookQuerySetMixin:
 
             # Then run business logic hooks
             engine.run(model_cls, BEFORE_UPDATE, objs, originals, ctx=ctx)
+        else:
+            print(f"DEBUG: bypass_hooks=True, skipping hooks for {len(objs)} objects")
 
             # Detect modified fields during hooks
             modified_fields = self._detect_modified_fields(objs, originals)
@@ -277,7 +281,14 @@ class HookQuerySetMixin:
             result = super().bulk_update(objs, fields, **django_kwargs)
 
         if not bypass_hooks:
+            print(
+                f"DEBUG: bypass_hooks=False, running AFTER_UPDATE hooks for {len(objs)} objects"
+            )
             engine.run(model_cls, AFTER_UPDATE, objs, originals, ctx=ctx)
+        else:
+            print(
+                f"DEBUG: bypass_hooks=True, skipping AFTER_UPDATE hooks for {len(objs)} objects"
+            )
 
         return result
 

{django_bulk_hooks-0.1.207.dist-info → django_bulk_hooks-0.1.209.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-bulk-hooks
-Version: 0.1.207
+Version: 0.1.209
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 Home-page: https://github.com/AugendLimited/django-bulk-hooks
 License: MIT

{django_bulk_hooks-0.1.207.dist-info → django_bulk_hooks-0.1.209.dist-info}/RECORD

@@ -3,15 +3,15 @@ django_bulk_hooks/conditions.py,sha256=mTvlLcttixbXRkTSNZU5VewkPUavbXRuD2BkJbVWM
 django_bulk_hooks/constants.py,sha256=3x1H1fSUUNo0DZONN7GUVDuySZctTR-jtByBHmAIX5w,303
 django_bulk_hooks/context.py,sha256=4IPuOX8TBAYBEMzN0RNHWgE6Giy2ZnR5uRXfd1cpIwk,1051
 django_bulk_hooks/decorators.py,sha256=WD7Jn7QAvY8F4wOsYlIpjoM9-FdHXSKB7hH9ot-lkYQ,4896
-django_bulk_hooks/engine.py,sha256=nA5PU9msk_Ju5Gf_sTd7GqPscuTxEW5itCDAoSScYGI,1645
+django_bulk_hooks/engine.py,sha256=cIgykVNU7IeNjCNmlSqgqFRUvym-rQDJ4mBte6vWyRI,1916
 django_bulk_hooks/enums.py,sha256=Zo8_tJzuzZ2IKfVc7gZ-0tWPT8q1QhqZbAyoh9ZVJbs,381
 django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,4854
 django_bulk_hooks/manager.py,sha256=nfWiwU5-yAoxdnQsUMohxtyCpkV0MBv6X3wmipr9eQY,3697
-django_bulk_hooks/models.py,sha256=7fnx5xd4HWXfLVlFhhiRzR92JRWFEuxgk6aSWLEsyJg,3996
+django_bulk_hooks/models.py,sha256=TA2dBIA1nJBiYt6joefWkpFIQZWysF9kZlkBYvEe59c,4358
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=YMw8Za_1aqLkCmTqhYKnznq95IVlie3Wx9NBDND7KcM,31387
+django_bulk_hooks/queryset.py,sha256=MnyNHRorpuq5JL8vHBakzCNBgqyWLYsaOrv9DtrP5nk,31941
 django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
-django_bulk_hooks-0.1.207.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.207.dist-info/METADATA,sha256=6XzEu4cdHtO1kVPzw-vuUmoFjbEUaVrTdEEKAFqEP60,9049
-django_bulk_hooks-0.1.207.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
-django_bulk_hooks-0.1.207.dist-info/RECORD,,
+django_bulk_hooks-0.1.209.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.209.dist-info/METADATA,sha256=QVsx3CnCzMXTg_dt071vPf5LOxse3j9fCJ3_WzXZxm8,9049
+django_bulk_hooks-0.1.209.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
+django_bulk_hooks-0.1.209.dist-info/RECORD,,