openerp-community team mailing list archive

Thread
Date

Re: lp:~camptocamp/openerp-product-attributes/7.0-adding-stock_production_lot_custom_attributes-lep into lp:openerp-product-attributes

To: "Leonardo Pistone @ camptocamp" <leonardo.pistone@xxxxxxxxxxxxxx>
From: "Leonardo Pistone @ camptocamp" <leonardo.pistone@xxxxxxxxxxxxxx>
Date: Mon, 25 Nov 2013 16:21:29 -0000
In-reply-to: <20131121161920.7187.54676.codereview@wampee.canonical.com>
Reply-to: mp+195950@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Thanks for the review Nicolas.

The bug seems to not exist in 7.0, removed the workaround.
Refactored the code to get read of the read()'s and improved the readability
Added doctrings and comments.

L
-- 
https://code.launchpad.net/~camptocamp/openerp-product-attributes/7.0-adding-stock_production_lot_custom_attributes-lep/+merge/195950
Your team OpenERP Community is subscribed to branch lp:openerp-product-attributes.

References

Re: lp:~camptocamp/openerp-product-attributes/7.0-adding-stock_production_lot_custom_attributes-lep into lp:openerp-product-attributes
From: Nicolas Bessi - Camptocamp, 2013-11-21