Commit fe5a6620 authored by Jérome Perrin's avatar Jérome Perrin

- add a context argument to stop warning messages

- warn readers about this docstring which does not reflect the actual implementation


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@28520 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent e8f18544
...@@ -257,11 +257,14 @@ class Movement(XMLObject, Amount): ...@@ -257,11 +257,14 @@ class Movement(XMLObject, Amount):
return {'price': context.Movement_lookupPrice()} return {'price': context.Movement_lookupPrice()}
security.declareProtected(Permissions.AccessContentsInformation, 'getPrice') 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. Get the Price in the context.
If price is not stored locally, lookup a price and store it. 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 # XXX As all accessors can recieve the default value as first positional
# argument, so we changed the first positional argument from context to # argument, so we changed the first positional argument from context to
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment