[Erp5-report] r27513 - /erp5/trunk/products/ERP5/Document/TradeModelLine.py

[nobody at svn.erp5.org]

Author: jp
Date: Thu Jun 11 10:36:59 2009
New Revision: 27513

URL: http://svn.erp5.org?rev=27513&view=rev
Log:
Added comments with TODO and request for better comments.

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

Modified: erp5/trunk/products/ERP5/Document/TradeModelLine.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/TradeModelLine.py?rev=27513&r1=27512&r2=27513&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/TradeModelLine.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/TradeModelLine.py [utf8] Thu Jun 11 10:36:59 2009
@@ -39,6 +39,11 @@
 
 class TradeModelLine(Predicate, XMLMatrix, Amount):
     """Trade Model Line is a way to represent trade transformation for movements
+
+    TODO: 
+      - make this code readable 
+      - use comments 
+      - use docstrings 
     """
     meta_type = 'ERP5 Trade Model Line'
     portal_type = 'Trade Model Line'
@@ -94,6 +99,7 @@
       if movement_list is None:
         movement_list = []
 
+      # XXX - really unreadable - more comments please
       if len(movement_list) == 0:
         if context.getPortalType() == 'Applied Rule':
           movement_list = [context.getParentValue()]