[Erp5-report] r28520 - /erp5/trunk/products/ERP5/Document/Movement.py

[nobody at svn.erp5.org]

Author: jerome
Date: Thu Aug 20 16:32:41 2009
New Revision: 28520

URL: http://svn.erp5.org?rev=28520&view=rev
Log:
- add a context argument to stop warning messages
- warn readers about this docstring which does not reflect the actual implementation

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

Modified: erp5/trunk/products/ERP5/Document/Movement.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/Movement.py?rev=28520&r1=28519&r2=28520&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/Movement.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/Movement.py [utf8] Thu Aug 20 16:32:41 2009
@@ -257,11 +257,14 @@
     return {'price': context.Movement_lookupPrice()}
 
   security.declareProtected(Permissions.AccessContentsInformation, 'getPrice')
-  def getPrice(self, default=None, evaluate=1, **kw):
+  def getPrice(self, default=None, context=None, evaluate=1, **kw):
     """
       Get the Price in the context.
 
       If price is not stored locally, lookup a price and store it.
+
+      FIXME: Don't trust this docstring, this method is not at all using the
+      passed context, but uses this movement as context.
     """
     # XXX As all accessors can recieve the default value as first positional
     # argument, so we changed the first positional argument from context to