Code review comment for lp:~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Hi,

l.11 The context must be propagated in the read.
l.24 No mutable in keyword argument: replace "c={}" by "c=None".

l.23-24 Indentation is dubious, see http://www.python.org/dev/peps/pep-0008/#indentation

review: Needs Fixing

« Back to merge proposal