[Erp5-report] r30929 - /erp5/trunk/products/ERP5/Document/BusinessPath.py

[nobody at svn.erp5.org]

Author: luke
Date: Mon Nov 30 15:39:26 2009
New Revision: 30929

URL: http://svn.erp5.org?rev=30929&view=rev
Log:
 - note issue with oversimplified implementation

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

Modified: erp5/trunk/products/ERP5/Document/BusinessPath.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/BusinessPath.py?rev=30929&r1=30928&r2=30929&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/BusinessPath.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/BusinessPath.py [utf8] Mon Nov 30 15:39:26 2009
@@ -220,6 +220,9 @@
   def _getExplanationUidList(self, explanation):
     """Helper method to fetch really explanation related movements"""
     explanation_uid_list = [explanation.getUid()]
+    # XXX: getCausalityRelatedValueList is oversimplification, assumes
+    #      that documents are sequenced like simulation movements, which
+    #      is wrong
     for found_explanation in explanation.getCausalityRelatedValueList(
         portal_type=self.getPortalDeliveryTypeList()):
       explanation_uid_list.extend(self._getExplanationUidList(