Dear community,It seems we have all made a quite common mistake in our python code - using empty lists or dictionaries as default values for functions. As it turns out, we should not have done that because that makes the defaults mutable, which may provoke strange bugs, very hard to reproduce. So please allow me to ask you all to change the code you contributed to the "community" and/or "extra-addons" branches. For example we should now change this
*def foo(...., context={},....):
....*
to this :
*def foo(...., context=None,....):
# you should add this test only if the context is actually used
if context is None:
context={}
....
*
I humbly suggest you fix the trunk branch in priority, and fix the 5.0
branches only if you feel totally comfortable with the changes.
Please read https://bugs.launchpad.net/openobject-server/+bug/525808 for more detail.
Lionel, for Numérigraphe SARL
<<attachment: informatique.vcf>>
_______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp
