From 05a1db46631b31a8052894afb98ed69ec8eb1565 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9rome=20Perrin?= <jerome@nexedi.com>
Date: Tue, 17 Oct 2006 08:31:36 +0000
Subject: [PATCH] fix or remove some copy 'n paste docstrings.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@10763 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/MovementGroup.py | 24 ++++++++----------------
 1 file changed, 8 insertions(+), 16 deletions(-)

diff --git a/product/ERP5/MovementGroup.py b/product/ERP5/MovementGroup.py
index 08f647efda..bd857ec9be 100644
--- a/product/ERP5/MovementGroup.py
+++ b/product/ERP5/MovementGroup.py
@@ -259,9 +259,7 @@ class OrderMovementGroup(RootMovementGroup):
 allow_class(OrderMovementGroup)
 
 class CausalityMovementGroup(RootMovementGroup):
-  """ Groups movement that comes from simulation movement that shares the
-  same explanation relation. For example, it groups in an Invoice
-  movements from the same Packing List. """
+  """ TODO: docstring """
   
   def __init__(self, movement, **kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
@@ -298,10 +296,7 @@ class CausalityMovementGroup(RootMovementGroup):
 allow_class(CausalityMovementGroup)
 
 class RootAppliedRuleCausalityMovementGroup(RootMovementGroup):
-  """ Groups movement that comes from simulation movement that shares the
-  same explanation relation. For example, it groups in an Invoice
-  movements from the same Packing List. """
-  
+  """ Group movement whose root apply rules have the same causality."""
   def __init__(self, movement, **kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     explanation_relative_url = self._getExplanationRelativeUrl(movement)
@@ -313,7 +308,7 @@ class RootAppliedRuleCausalityMovementGroup(RootMovementGroup):
     )
 
   def _getExplanationRelativeUrl(self, movement):
-    """ Get the order value for a movement """
+    """ TODO: docstring; method name is bad; variables names are bad """
     root_applied_rule = movement.getRootAppliedRule()
     explanation_value = root_applied_rule.getCausalityValue()
     explanation_relative_url = None
@@ -330,10 +325,7 @@ allow_class(RootAppliedRuleCausalityMovementGroup)
 # Sale invoices transactions lines. We need to put accounting lines
 # in the same invoices than invoice lines.
 class ParentExplanationMovementGroup(RootMovementGroup):
-  """ Groups movement that comes from simulation movement that shares the
-  same explanation relation. For example, it groups in an Invoice
-  movements from the same Packing List. """
-  
+  """ TODO: docstring """
   def __init__(self, movement, **kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     explanation_value = self._getParentExplanationValue(movement)
@@ -370,7 +362,7 @@ class PathMovementGroup(RootMovementGroup):
 allow_class(PathMovementGroup)
 
 class ColourMovementGroup(RootMovementGroup):
-  """ Group movements that have the same source and the same destination."""
+  """ Group movements that have the same color category."""
   def __init__(self, movement, **kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     self.colour = movement.getColour()
@@ -445,7 +437,7 @@ class QuantitySignMovementGroup(RootMovementGroup):
 allow_class(QuantitySignMovementGroup)
 
 class DateMovementGroup(RootMovementGroup):
-
+  """ Group movements that have exactly the same dates. """
   def __init__(self,movement,**kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     self.start_date = movement.getStartDate()
@@ -465,7 +457,7 @@ class DateMovementGroup(RootMovementGroup):
 allow_class(DateMovementGroup)
 
 class CriterionMovementGroup(RootMovementGroup):
-
+  
   def __init__(self,movement,**kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     if hasattr(movement, 'getGroupCriterion'):
@@ -484,7 +476,7 @@ class CriterionMovementGroup(RootMovementGroup):
 allow_class(CriterionMovementGroup)
 
 class ResourceMovementGroup(RootMovementGroup):
-
+  """ Group movements that have the same resource. """
   def __init__(self, movement, **kw):
     RootMovementGroup.__init__(self, movement=movement, **kw)
     self.resource = movement.getResource()
-- 
2.30.9