[Erp5-report] r27903 - /erp5/trunk/products/ERP5/Document/

[nobody at svn.erp5.org]

Author: luke
Date: Fri Jul  3 13:29:16 2009
New Revision: 27903

URL: http://svn.erp5.org?rev=27903&view=rev
Log:
 - revert 27902 - docstrings are needed by Zope, until proper solution is found and applied globally it shall be not implemented in specific cases

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

Modified: erp5/trunk/products/ERP5/Document/BusinessPath.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/BusinessPath.py?rev=27903&r1=27902&r2=27903&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/BusinessPath.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/BusinessPath.py [utf8] Fri Jul  3 13:29:16 2009
@@ -176,6 +176,8 @@
 
   # IBusinessBuildable implementation
   def isBuildable(self, explanation):
+    """
+    """
     if self.isCompleted(explanation):
       return False # No need to build what was already built
     if self.isFrozen(explanation):
@@ -193,6 +195,9 @@
     """
 
   def build(self, explanation):
+    """
+      Build
+    """
     builder_list = self.getBuilderList() # Missing method
     for builder in builder_list:
       builder.build(causality_uid=self.getUid()) # This is one way of doing
@@ -208,6 +213,10 @@
 
   # IBusinessCompletable implementation
   def isCompleted(self, explanation):
+    """
+      Looks at all simulation related movements
+      and checks the simulation_state of the delivery
+    """
     acceptable_state_list = self.getCompletedStateList()
     for movement in self._getRelatedSimulationMovementList(explanation):
       if movement.getSimulationState() not in acceptable_state_list:
@@ -215,6 +224,10 @@
     return True
 
   def isPartiallyCompleted(self, explanation):
+    """
+      Looks at all simulation related movements
+      and checks the simulation_state of the delivery
+    """
     acceptable_state_list = self.getCompletedStateList()
     for movement in self._getRelatedSimulationMovementList(explanation):
       if movement.getSimulationState() in acceptable_state_list:
@@ -222,6 +235,10 @@
     return False
 
   def isFrozen(self, explanation):
+    """
+      Looks at all simulation related movements
+      and checks if frozen
+    """
     movement_list = self._getRelatedSimulationMovementList(explanation)
     if len(movement_list) == 0:
       return False # Nothing to be considered as Frozen
@@ -270,6 +287,13 @@
         return expected_date - self.getLeadTime()
 
   def getExpectedStopDate(self, explanation, predecessor_date=None, *args, **kwargs):
+    """
+      Returns the expected stop date for this
+      path based on the explanation.
+
+      predecessor_date -- if provided, computes the date base on the
+                          date value provided
+    """
     return self._getExpectedDate(explanation,
                                  self._getRootExplanationExpectedStopDate,
                                  self._getPredecessorExpectedStopDate,

Modified: erp5/trunk/products/ERP5/Document/BusinessProcess.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/BusinessProcess.py?rev=27903&r1=27902&r2=27903&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/BusinessProcess.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/BusinessProcess.py [utf8] Fri Jul  3 13:29:16 2009
@@ -142,6 +142,9 @@
     return filter(lambda x:x.isPartiallyCompleted(explanation), self.getStateValueList())
 
   def getLatestCompletedStateValue(self, explanation):
+    """
+      Returns the most advanced completed state
+    """
     for state in self.getCompletedStateValueList(explanation):
       for path in state.getPredecessorRelatedValueList():
         if not path.isCompleted(explanation):
@@ -149,6 +152,9 @@
     return None
 
   def getLatestPartiallyCompletedStateValue(self, explanation):
+    """
+      Returns the most advanced completed state
+    """
     for state in self.getCompletedStateValueList(explanation):
       for path in state.getPredecessorRelatedValueList():
         if not path.isPartiallyCompleted(explanation):
@@ -156,6 +162,9 @@
     return None
 
   def getLatestCompletedStateValueList(self, explanation):
+    """
+      Returns the most advanced completed state
+    """
     result = []
     for state in self.getCompletedStateValueList(explanation):
       for path in state.getPredecessorRelatedValueList():
@@ -164,6 +173,9 @@
     return result
 
   def getLatestPartiallyCompletedStateValueList(self, explanation):
+    """
+      Returns the most advanced completed state
+    """
     result = []
     for state in self.getCompletedStateValueList(explanation):
       for path in state.getPredecessorRelatedValueList():
@@ -185,6 +197,9 @@
     return self.getReferentialDate() == 'stop_date'
 
   def getTradePhaseList(self):
+    """
+      Returns all trade_phase of this business process
+    """
     path_list = self.objectValues(portal_type=self.getPortalBusinessPathTypeList())
     return filter(None, [path.getTradePhase()
                          for path in path_list])