no need to access the delivery movement, checking if a category is set is

likely enough


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@44184 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5/mixin/rule.py

@@ -219,7 +219,7 @@ class RuleMixin(Predicate):
     Only account movements which are not associated to a delivery;
     Whenever delivery is there, delivery has priority
     """
-    return movement.getDeliveryValue() is None
+    return not movement.getDelivery()
 
   # Implementation of IDivergenceController # XXX-JPS move to IDivergenceController only mixin for 
   security.declareProtected( Permissions.AccessContentsInformation,
@@ -229,10 +229,9 @@ class RuleMixin(Predicate):
     Returns true if the Simulation Movement is divergent comparing to
     the delivery value
     """
-    delivery = movement.getDeliveryValue()
-    if delivery is None:
+    if not movement.getDelivery():
       return False
-    return len(self.getDivergenceList(movement)) != 0
+    return bool(self.getDivergenceList(movement))
 
   security.declareProtected(Permissions.View, 'getDivergenceList')
   def getDivergenceList(self, movement):