Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-896462-mdi

Revision history for this message
DJ Patel (OpenERP) (mdi-openerp) wrote :

Hello Sir,

I use copy_data() to duplicate and modify an existing record properly instead of the strange stuff done in refund().

So refund() should be based on a call to copy_data() on the invoice, and not manual steps like that.

I totally removed _refund_cleanup_lines() function from 'account' and 'account_anglo_saxon'.

In order to not hide the functionality provided by 'account_anglo_saxon' module i override the 'refund' method.

Thanks,
Divyesh

review: Needs Resubmitting

« Back to merge proposal