django-bulk-hooks 0.1.89__py3-none-any.whl → 0.1.91__py3-none-any.whl

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/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/models.py

@@ -94,19 +94,29 @@ class HookModelMixin(models.Model):
         # Use a single context manager for all hooks
         with patch_foreign_key_behavior():
             if is_create:
-                # For create operations
+                # For create operations, run validation hooks first
+                run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
+                # Then run BEFORE hooks
                 run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
+                # Now save
                 super().save(*args, **kwargs)
+                # Finally run AFTER hooks
                 run(self.__class__, AFTER_CREATE, [self], ctx=ctx)
             else:
                 # For update operations
                 try:
                     old_instance = self.__class__.objects.get(pk=self.pk)
+                    # Run validation hooks first
+                    run(self.__class__, VALIDATE_UPDATE, [self], [old_instance], ctx=ctx)
+                    # Then run BEFORE hooks
                     run(self.__class__, BEFORE_UPDATE, [self], [old_instance], ctx=ctx)
+                    # Now save
                     super().save(*args, **kwargs)
+                    # Finally run AFTER hooks
                     run(self.__class__, AFTER_UPDATE, [self], [old_instance], ctx=ctx)
                 except self.__class__.DoesNotExist:
                     # If the old instance doesn't exist, treat as create
+                    run(self.__class__, VALIDATE_CREATE, [self], ctx=ctx)
                     run(self.__class__, BEFORE_CREATE, [self], ctx=ctx)
                     super().save(*args, **kwargs)
                     run(self.__class__, AFTER_CREATE, [self], ctx=ctx)

{django_bulk_hooks-0.1.89.dist-info → django_bulk_hooks-0.1.91.dist-info}/METADATA

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

@@ -1,16 +1,16 @@
 django_bulk_hooks/__init__.py,sha256=2PcJ6xz7t7Du0nmLO_5732G6u_oZTygogG0fKESRHHk,1082
-django_bulk_hooks/conditions.py,sha256=zeq1q4YgBV29kSkCpcAySA1CIMbVbL4TzNPVaESTsXw,13359
+django_bulk_hooks/conditions.py,sha256=w3uxXlsJcxSsoH7_7dfmYkO6oIr8OqgoaUGp6oDIGZo,13573
 django_bulk_hooks/constants.py,sha256=3x1H1fSUUNo0DZONN7GUVDuySZctTR-jtByBHmAIX5w,303
 django_bulk_hooks/context.py,sha256=HVDT73uSzvgrOR6mdXTvsBm3hLOgBU8ant_mB7VlFuM,380
 django_bulk_hooks/decorators.py,sha256=zstmb27dKcOHu3Atg7cauewCTzPvUmq03mzVKJRi56o,7230
-django_bulk_hooks/engine.py,sha256=N7cz0Tp0cMm9JQnrWwnGbYZwfVMRGaPSFVHpwNG5-h0,2828
+django_bulk_hooks/engine.py,sha256=1hqjmLRmxwI-OSC6GJizD_CFGPw7pndAa0GMIV7P-XQ,3544
 django_bulk_hooks/enums.py,sha256=Zo8_tJzuzZ2IKfVc7gZ-0tWPT8q1QhqZbAyoh9ZVJbs,381
 django_bulk_hooks/handler.py,sha256=Qpg_zT6SsQiTlhduvzXxPdG6uynjyR2fBjj-R6HZiXI,4861
 django_bulk_hooks/manager.py,sha256=DcVosEA4RS79KSYgw3Z14_a9Sd8CfxNNc5F3eSb8xc0,11459
-django_bulk_hooks/models.py,sha256=9KvWkmrR0wbTHN6r7-FrSSO9ViS83NvG7iXLBw_iDZs,4793
+django_bulk_hooks/models.py,sha256=gKOcOLhOHZ9i-25S-quxxvEBRgFwsrf7ir-6Jc5B8Co,5344
 django_bulk_hooks/queryset.py,sha256=7lLqhZ-XOYsZ1I3Loxi4Nhz79M8HlTYE413AW8nyeDI,1330
 django_bulk_hooks/registry.py,sha256=Vh78exKYcdZhM27120kQm-iXGOjd_kf9ZUYBZ8eQ2V0,683
-django_bulk_hooks-0.1.89.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.89.dist-info/METADATA,sha256=KSsqX4dcOdx1arfRBzwqojW0TuZyifAjpenVZAreRZc,9051
-django_bulk_hooks-0.1.89.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.89.dist-info/RECORD,,
+django_bulk_hooks-0.1.91.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.91.dist-info/METADATA,sha256=pNCdiwyWlepBlLyY6L7p9tLKxLAxuavYLsIBrInJVpY,9051
+django_bulk_hooks-0.1.91.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.91.dist-info/RECORD,,