Review: Needs Fixing

Hi Rifakat,

For the rename of lang to partner_lang, context to partner_context that ok.

But your MP modifies a lot of methods API (adding "context" param) that should 
be avoid, because this can break custom modules + most v6.0 client doesn't sent 
the user context at all (so this won't work).

Perhaps could you rework this by using, at start of the method:

  context = self.pool.get("res.users").context_get(cr, uid)

This will allow to get a correct context (containing user language).

Thanks,
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-279130-rha/+merge/88818
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-279130-rha.

_______________________________________________
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