c2c-oerpscenario team mailing list archive

["Olivier Dony \(OpenERP\)" <525808@xxxxxxxxxxxxxxxxxx>]

FYI, this is still being worked on in R&D, to gradually clean it up. For example see:
 - https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-addons-context/+merge/41682
 - http://bazaar.launchpad.net/~openerp/openobject-server/trunk/revision/3084

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/525808

Title:
  Default values should be immutable

Status in OpenObject Addons Modules:
  Opinion
Status in OpenObject Addons extra-trunk series:
  Opinion
Status in OpenObject Addons trunk series:
  Opinion
Status in OpenObject GTK Client:
  Opinion
Status in OpenObject Server:
  Fix Released
Status in OpenObject Server trunk series:
  Fix Released

Bug description:
  Often in the code base we have:
def foo(...., context={},....):
    ....
As explained in comment #1, this should be changed to
def foo(...., context=None,....):
    # you should add this test only if the context is actually used
    if context is None:
        context={}
    ....

(This bug originally proposed the other way round hence the "nope" in comment #1.)

(xmo) Complements of information:

* The test should be `context is None` rather than `not context`: the caller could provide an empty dict argument {} and expect to have data in it after the call, if a test is done using `not context` an empty dict will match and the function will be creating a brand new context.
* The test should *only ever* performed when the current function fetches data from the context. If you don't do anything with the context but forward it to other functions and methods (e.g. read() or create()), no need to test for anything.