[Erp5-report] r35367 kazuhiko - /erp5/trunk/products/ERP5/Document/

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Sat May 15 11:06:19 2010
New Revision: 35367

URL: http://svn.erp5.org?rev=35367&view=rev
Log:
same as 35180, the expected behaviour of this movement group in delivery level is unclear.

Modified:
    erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py

Modified: erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py?rev=35367&r1=35366&r2=35367&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py [utf8] Sat May 15 11:06:19 2010
@@ -55,13 +55,16 @@
     else:
       target_property_list = self.getTestedPropertyList()
     if document == document.getDeliveryValue():
-      movement = document.getMovementList()[0].getDeliveryRelatedValue()
+      # XXX what is the expected behaviour of this movement group in
+      # delivery level?
+      pass
     else:
       movement = document.getDeliveryRelatedValue()
-    parent_delivery = self._getParentDelivery(movement)
+      if movement is None:
+        return False, {}
+      document = self._getParentDelivery(movement)
     for prop in target_property_list:
-      if property_dict['_%s' % prop] != self._getProperty(parent_delivery,
-                                                          prop):
+      if property_dict['_%s' % prop] != self._getProperty(document, prop):
         return False, {}
     return True, {}