django-bulk-hooks 0.1.201__py3-none-any.whl → 0.1.202__py3-none-any.whl

django_bulk_hooks/queryset.py

@@ -321,35 +321,34 @@ class HookQuerySetMixin:
         """
         Detect fields that were modified during BEFORE_UPDATE hooks by comparing
         new instances with their original values.
+        Optimized to avoid dereferencing related objects which can trigger queries.
         """
         if not original_instances:
             return set()
 
         modified_fields = set()
 
-        # Since original_instances is now ordered to match new_instances, we can zip them directly
+        # Since original_instances is ordered to match new_instances, we can zip them directly
         for new_instance, original in zip(new_instances, original_instances):
-            if new_instance.pk is None or original is None:
+            if new_instance is None or original is None:
                 continue
 
-            # Compare all fields to detect changes
-            for field in new_instance._meta.fields:
-                if field.name == "id":
+            # Only check local concrete fields; skip PK and many-to-many
+            for field in new_instance._meta.local_concrete_fields:
+                if field.primary_key:
                     continue
 
-                new_value = getattr(new_instance, field.name)
-                original_value = getattr(original, field.name)
-
-                # Handle different field types appropriately
-                if field.is_relation:
-                    # For foreign keys, compare the pk values
-                    new_pk = new_value.pk if new_value else None
-                    original_pk = original_value.pk if original_value else None
-                    if new_pk != original_pk:
+                if getattr(field, "remote_field", None):
+                    # ForeignKey/OneToOne: compare the raw id values via attname to avoid fetching related objects
+                    new_id = getattr(new_instance, field.attname, None)
+                    old_id = getattr(original, field.attname, None)
+                    if new_id != old_id:
                         modified_fields.add(field.name)
                 else:
-                    # For regular fields, use direct comparison
-                    if new_value != original_value:
+                    # Regular value fields
+                    new_value = getattr(new_instance, field.attname if hasattr(field, "attname") else field.name)
+                    old_value = getattr(original, field.attname if hasattr(field, "attname") else field.name)
+                    if new_value != old_value:
                         modified_fields.add(field.name)
 
         return modified_fields

{django_bulk_hooks-0.1.201.dist-info → django_bulk_hooks-0.1.202.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.201
+Version: 0.1.202
 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.201.dist-info → django_bulk_hooks-0.1.202.dist-info}/RECORD

@@ -9,9 +9,9 @@ django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,
 django_bulk_hooks/manager.py,sha256=nfWiwU5-yAoxdnQsUMohxtyCpkV0MBv6X3wmipr9eQY,3697
 django_bulk_hooks/models.py,sha256=7fnx5xd4HWXfLVlFhhiRzR92JRWFEuxgk6aSWLEsyJg,3996
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
-django_bulk_hooks/queryset.py,sha256=J3qIJPo7-Ugj3vEwwMEr9HxMEjzlE6gVLT3HYy6abeQ,36442
+django_bulk_hooks/queryset.py,sha256=NRbkJL9D_V97W76UxodMye1zaB0nJZGjMmv0SR0QLzs,36628
 django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
-django_bulk_hooks-0.1.201.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.201.dist-info/METADATA,sha256=fHweGvMl0jrbXhvpD0KqubpJtop2agnqU6jBSxE4K8o,7418
-django_bulk_hooks-0.1.201.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.201.dist-info/RECORD,,
+django_bulk_hooks-0.1.202.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.202.dist-info/METADATA,sha256=sJY6iID8owgHJErIN8ZUhXNjJ7Znl1F8HedZiroE8_0,7418
+django_bulk_hooks-0.1.202.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.202.dist-info/RECORD,,