openerp-dev-web team mailing list archive

Thread
Date

Re: [Merge] lp:~openerp-dev/openobject-addons/trunk-boolean-asset-pso into lp:~openerp-dev/openobject-addons/trunk-account-asset

To: mp+60743@xxxxxxxxxxxxxxxxxx
From: "Mustufa Rangwala \(Open ERP\)" <mra@xxxxxxxxxxx>
Date: Thu, 12 May 2011 11:29:22 -0000
In-reply-to: <20110512091759.1360.69420.launchpad@loganberry.canonical.com>
Reply-to: mp+60743@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing
1. def get_move_line => rename it with def _get_move_check
2. 'move_line_present' => rename field with move_check and correct string too
3. context={} => context=None should be use
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-boolean-asset-pso/+merge/60743
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-account-asset.

References

[Merge] lp:~openerp-dev/openobject-addons/trunk-boolean-asset-pso into lp:~openerp-dev/openobject-addons/trunk-account-asset
From: Priyesh (Open ERP), 2011-05-12