Review: Needs Fixing

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
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-381612-ado/+merge/88682
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-381612-ado.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to