django-bulk-hooks 0.2.7__py3-none-any.whl → 0.2.8__py3-none-any.whl

django_bulk_hooks/conditions.py

@@ -3,38 +3,38 @@ import logging
 logger = logging.getLogger(__name__)
 
 
-def resolve_dotted_attr(instance, dotted_path):
+def resolve_field_path(instance, field_path):
     """
-    Recursively resolve a dotted attribute path, e.g., "type.category".
+    Recursively resolve a field path using Django's __ notation, e.g., "author__profile__name".
 
     CRITICAL: For foreign key fields, uses attname to access the ID directly
     to avoid hooking Django's descriptor protocol which causes N+1 queries.
     """
-    # For simple field access (no dots), use optimized field access
-    if "." not in dotted_path:
+    # For simple field access (no __), use optimized field access
+    if "__" not in field_path:
         try:
             # Get the field from the model's meta to check if it's a foreign key
-            field = instance._meta.get_field(dotted_path)
+            field = instance._meta.get_field(field_path)
             if field.is_relation and not field.many_to_many:
                 # For foreign key fields, use attname to get the ID directly
                 # This avoids hooking Django's descriptor protocol
                 return getattr(instance, field.attname, None)
             else:
                 # For regular fields, use normal getattr
-                return getattr(instance, dotted_path, None)
+                return getattr(instance, field_path, None)
         except Exception:
             # If field lookup fails, fall back to normal getattr
-            return getattr(instance, dotted_path, None)
+            return getattr(instance, field_path, None)
 
-    # For dotted paths, traverse the relationship chain with FK optimization
+    # For paths with __, traverse the relationship chain with FK optimization
     current_instance = instance
-    for i, attr in enumerate(dotted_path.split(".")):
+    for i, attr in enumerate(field_path.split("__")):
         if current_instance is None:
             return None
 
         try:
             # Check if this is the last attribute and if it's a FK field
-            is_last_attr = i == len(dotted_path.split(".")) - 1
+            is_last_attr = i == len(field_path.split("__")) - 1
             if is_last_attr and hasattr(current_instance, "_meta"):
                 try:
                     field = current_instance._meta.get_field(attr)
@@ -79,11 +79,11 @@ class IsNotEqual(HookCondition):
         self.only_on_change = only_on_change
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
         if self.only_on_change:
             if original_instance is None:
                 return False
-            previous = resolve_dotted_attr(original_instance, self.field)
+            previous = resolve_field_path(original_instance, self.field)
             return previous == self.value and current != self.value
         else:
             return current != self.value
@@ -96,12 +96,12 @@ class IsEqual(HookCondition):
         self.only_on_change = only_on_change
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
 
         if self.only_on_change:
             if original_instance is None:
                 return False
-            previous = resolve_dotted_attr(original_instance, self.field)
+            previous = resolve_field_path(original_instance, self.field)
             return previous != self.value and current == self.value
         else:
             return current == self.value
@@ -116,8 +116,8 @@ class HasChanged(HookCondition):
         if not original_instance:
             return False
 
-        current = resolve_dotted_attr(instance, self.field)
-        previous = resolve_dotted_attr(original_instance, self.field)
+        current = resolve_field_path(instance, self.field)
+        previous = resolve_field_path(original_instance, self.field)
 
         return (current != previous) == self.has_changed
 
@@ -135,9 +135,9 @@ class WasEqual(HookCondition):
     def check(self, instance, original_instance=None):
         if original_instance is None:
             return False
-        previous = resolve_dotted_attr(original_instance, self.field)
+        previous = resolve_field_path(original_instance, self.field)
         if self.only_on_change:
-            current = resolve_dotted_attr(instance, self.field)
+            current = resolve_field_path(instance, self.field)
             return previous == self.value and current != self.value
         else:
             return previous == self.value
@@ -155,8 +155,8 @@ class ChangesTo(HookCondition):
     def check(self, instance, original_instance=None):
         if original_instance is None:
             return False
-        previous = resolve_dotted_attr(original_instance, self.field)
-        current = resolve_dotted_attr(instance, self.field)
+        previous = resolve_field_path(original_instance, self.field)
+        current = resolve_field_path(instance, self.field)
         return previous != self.value and current == self.value
 
 
@@ -166,7 +166,7 @@ class IsGreaterThan(HookCondition):
         self.value = value
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
         return current is not None and current > self.value
 
 
@@ -176,7 +176,7 @@ class IsGreaterThanOrEqual(HookCondition):
         self.value = value
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
         return current is not None and current >= self.value
 
 
@@ -186,7 +186,7 @@ class IsLessThan(HookCondition):
         self.value = value
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
         return current is not None and current < self.value
 
 
@@ -196,7 +196,7 @@ class IsLessThanOrEqual(HookCondition):
         self.value = value
 
     def check(self, instance, original_instance=None):
-        current = resolve_dotted_attr(instance, self.field)
+        current = resolve_field_path(instance, self.field)
         return current is not None and current <= self.value
 
 

{django_bulk_hooks-0.2.7.dist-info → django_bulk_hooks-0.2.8.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.2.7
+Version: 0.2.8
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 License: MIT
 Keywords: django,bulk,hooks

{django_bulk_hooks-0.2.7.dist-info → django_bulk_hooks-0.2.8.dist-info}/RECORD

@@ -1,6 +1,6 @@
 django_bulk_hooks/__init__.py,sha256=4QlWY5rqR9o2ddrNB-ypM4s1GtNJ1ZvL2ABhybaPpio,1823
 django_bulk_hooks/changeset.py,sha256=WALeiWDcjOBNdCKeidVKOPKAySKj9ZOvUJ-kWaVZYhM,7444
-django_bulk_hooks/conditions.py,sha256=tNnQZvcR-jOB8ZzpoVd03PVIy8rjiFdzmufD5mP7fTg,8107
+django_bulk_hooks/conditions.py,sha256=qtGjToKXC8FPUPK31Mib-GMzc9GSdrH90M2pT3CIsh8,8111
 django_bulk_hooks/constants.py,sha256=PxpEETaO6gdENcTPoXS586lerGKVP3nmjpDvOkmhYxI,509
 django_bulk_hooks/context.py,sha256=mqaC5-yESDTA5ruI7fuXlt8qSgKuOFp0mjq7h1-4HdQ,1926
 django_bulk_hooks/debug_utils.py,sha256=6T32E_Pms6gbCl94A55fJAe_ynFsK_CJBTaPcsG8tik,4578
@@ -20,7 +20,7 @@ django_bulk_hooks/operations/mti_handler.py,sha256=eIH-tImMqcWR5lLQr6Ca-HeVYta-U
 django_bulk_hooks/operations/mti_plans.py,sha256=fHUYbrUAHq8UXqxgAD43oHdTxOnEkmpxoOD4Qrzfqk8,2878
 django_bulk_hooks/queryset.py,sha256=ody4MXrRREL27Ts2ey1UpS0tb5Dxnw-6kN3unxPQ3zY,5860
 django_bulk_hooks/registry.py,sha256=UPerNhtVz_9tKZqrYSZD2LhjAcs4F6hVUuk8L5oOeHc,8821
-django_bulk_hooks-0.2.7.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.2.7.dist-info/METADATA,sha256=4sY-WsqTUdlCIoNQBgnNpdScPMZ7BEbi980D4T4MsYE,9264
-django_bulk_hooks-0.2.7.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.2.7.dist-info/RECORD,,
+django_bulk_hooks-0.2.8.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.2.8.dist-info/METADATA,sha256=bkF722Ehasysowep-24uwfmkGPCvCcYuVz7v2-kawcM,9264
+django_bulk_hooks-0.2.8.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.2.8.dist-info/RECORD,,