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

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Wed May 26 14:43:51 2010
New Revision: 35645

URL: http://svn.erp5.org?rev=35645&view=rev
Log:
fix a bug that can call getDeliveryValue() for applied rule not for simulation movement.

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

Modified: erp5/trunk/products/ERP5/Document/FirstCausalityMovementGroup.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/FirstCausalityMovementGroup.py?rev=35645&r1=35644&r2=35645&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/FirstCausalityMovementGroup.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/FirstCausalityMovementGroup.py [utf8] Wed May 26 14:43:51 2010
@@ -58,12 +58,16 @@
   def _getExplanationRelativeUrl(self, movement):
     """ Get the order value for a movement """
     applied_rule = movement.getParentValue()
-    parent = applied_rule.getParentValue()
+    if applied_rule.isRootAppliedRule():
+      return None
+    parent_movement = applied_rule.getParentValue()
     # Go upper into the simulation tree in order to find a delivery link
-    parent_delivery = parent.getDeliveryValue()
-    while parent_delivery is None and not parent.isRootAppliedRule():
-      parent = parent.getParentValue()
-      parent_delivery = parent.getDeliveryValue()
+    parent_delivery = parent_movement.getDeliveryValue()
+    applied_rule = parent_movement.getParentValue()
+    while parent_delivery is None and not applied_rule.isRootAppliedRule():
+      parent_movement = applied_rule.getParentValue()
+      parent_delivery = parent_movement.getDeliveryValue()
+      applied_rule = parent_movement.getParentValue()
     delivery_movement = parent_delivery
     delivery_url = None
     if delivery_movement is not None: