openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~yann-papouin/ocb-addons/6.1-bug-947838-sale-recreate-packing-duplicates into lp:ocb-addons/6.1

To: Yann Papouin <yann.papouin@xxxxxxxxx>
From: "Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>
Date: Mon, 20 Jan 2014 10:55:51 -0000
In-reply-to: <20131205130125.32123.78129.launchpad@ackee.canonical.com>
Reply-to: mp+194523@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

No, on browse records there's no need to pass the context. It's even harmful, does the code you submitted now run? I'd expect and exception about passing a duplicate keyword argument names 'context'
-- 
https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-947838-sale-recreate-packing-duplicates/+merge/194523
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1.

References

lp:~yann-papouin/ocb-addons/6.1-bug-947838-sale-recreate-packing-duplicates into lp:ocb-addons/6.1
From: Yann Papouin, 2013-12-05