openerp-dev-web team mailing list archive

Thread
Date

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

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

Review: Needs Fixing
Fix for Account Asset:

1. If i have 1000 Euro in depreciation amount and i create move  dep. account will be credited with 1000 and dep expence account will be debited with 1000.
In your case its reverse :P

2. #FIXME: required=True + add in the form view => remove it  (its done by you itself in this patch)

3. reference missing in create move from asset
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-button-move-pso/+merge/60756
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-button-move-pso into lp:~openerp-dev/openobject-addons/trunk-account-asset
From: Priyesh (Open ERP), 2011-05-12