Code review comment for lp:~openerp-dev/openobject-server/6.0-opw-381612-ado

Revision history for this message
Xavier ALT (dex-phx) wrote :

Hello,

Better change default parameter for res to None not "[]", and add a check like for "context" on top of method. That way you could remove changes inside "button_install" and also gracefully handle other call to state_update() (ex: in bin/addons/base/res/res_config.py)

Cheers,
Xavier

review: Needs Fixing

« Back to merge proposal