Review: Needs Fixing

I really don't like the idea of passing stuff in the context to work around a 
problem.  This is way too bug-prone.

Moreover, I believe your fix is incorrect.  From what I can see, when the taxes 
are included in the voucher, you use the voucher's total amount (inv.amount) 
instead of each voucher line amount.  If the voucher has N lines, you will 
create N move lines with the voucher's total amount!

Please test this on a voucher with at least two lines, and check all the move 
lines created.

Thanks,
Raphael



-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-17387-rha/+merge/76338
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-17387-rha.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to