django-bulk-hooks 0.1.89__tar.gz → 0.1.90__tar.gz

{django_bulk_hooks-0.1.89 → django_bulk_hooks-0.1.90}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.89
+Version: 0.1.90
 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.89 → django_bulk_hooks-0.1.90}/django_bulk_hooks/conditions.py

@@ -209,8 +209,13 @@ class IsEqual(HookCondition):
         self.value = value
 
     def check(self, instance, original_instance=None):
-        current_value = resolve_dotted_attr(instance, self.field)
-        return current_value == self.value
+        # Handle the case where the field might not exist yet
+        try:
+            current_value = getattr(instance, self.field, None)
+            return current_value == self.value
+        except Exception:
+            # If there's any error accessing the field, treat it as None
+            return self.value is None
 
     def get_required_fields(self):
         return {self.field.split('.')[0]}

{django_bulk_hooks-0.1.89 → django_bulk_hooks-0.1.90}/django_bulk_hooks/engine.py

@@ -57,14 +57,26 @@ def run(model_cls, event, new_instances, original_instances=None, ctx=None):
             to_process_new = []
             to_process_old = []
 
+            logger.debug(f"Checking condition {condition.__class__.__name__} for {len(new_instances)} instances")
             for new, original in zip(new_instances, original_instances, strict=True):
-                if condition.check(new, original):
-                    to_process_new.append(new)
-                    to_process_old.append(original)
+                logger.debug(f"Checking instance {new.__class__.__name__}(pk={new.pk})")
+                try:
+                    matches = condition.check(new, original)
+                    logger.debug(f"Condition check result: {matches}")
+                    if matches:
+                        to_process_new.append(new)
+                        to_process_old.append(original)
+                except Exception as e:
+                    logger.error(f"Error checking condition: {e}")
+                    raise
 
             # Only call if we have matching instances
             if to_process_new:
+                logger.debug(f"Running hook for {len(to_process_new)} matching instances")
                 func(new_records=to_process_new, old_records=to_process_old if any(to_process_old) else None)
+            else:
+                logger.debug("No instances matched condition")
         else:
             # No condition, process all instances
+            logger.debug("No condition, processing all instances")
             func(new_records=new_instances, old_records=original_instances if any(original_instances) else None)

{django_bulk_hooks-0.1.89 → django_bulk_hooks-0.1.90}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-bulk-hooks"
-version = "0.1.89"
+version = "0.1.90"
 description = "Hook-style hooks for Django bulk operations like bulk_create and bulk_update."
 authors = ["Konrad Beck <konrad.beck@merchantcapital.co.za>"]
 readme = "README.md"