- move isFrozen to business_completable

 - make isFrozen definition clear for simulation_movement and business_completable


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

product/ERP5/interfaces/business_buildable.py

@@ -30,12 +30,7 @@
 from zope.interface import Interface
 
 class IBusinessBuildable(Interface):
-  """Business Buildable interface specification
-
-  TODO:
-    - is isFrozen useful ? is it the same as isCompleted ?
-    - why isFrozen is here and not in Completable
-  """
+  """Business Buildable interface specification"""
   def isBuildable(explanation):
     """Returns True if any of the related Simulation Movement
     is buildable and if the predecessor state is completed.
@@ -52,17 +47,6 @@ class IBusinessBuildable(Interface):
     cause of a root applied rule in the simulation
     """
 
-  def isFrozen(explanation):
-    """Returns True if all related movements in the simulation
-    are frozen
-
-    'explanation' is the Order or Item or Document which is the
-    cause of a root applied rule in the simulation
-
-    XXX - could be redundant with isBuildable
-    (isFrozen = not isBuildable ?)
-    """
-
   def build(explanation):
     """Builds all related movements in the simulation
     using the builders of Business Path

product/ERP5/interfaces/business_completable.py

@@ -45,6 +45,11 @@ class IBusinessCompletable(Interface):
     as finished.
     """
 
+  def isFrozen(explanation):
+    """True if all related simulation movements for this explanation
+    are frozen.
+    """
+
   def getExpectedCompletionDate(task):
     """Returns the date at which the given state is expected to
     be completed, based on the start_date and stop_date of

product/ERP5/interfaces/simulation_movement.py

@@ -103,11 +103,10 @@ class ISimulationMovement(Interface):
   def isFrozen():
     """Tells whether the simulation movement is frozen.
     By default, looks up the related Business Process Path
-    and tells if the simulation state is part of completed
+    and tells if the simulation state is part of frozen
     states.
 
-    XXX - should this be renamed isCompleted ? are the
-    notions of isFrozen and isCompleted same or different ?
+    Frozen simulation movement cannot be modified by expanding.
     """
 
   def isSimulated():