Review: Needs Fixing
there is no need of 2 functions, 1 is enough. 

i don't understand this line:
 + if voucher.pay_now == 'pay_later':


You should probably use the amount_residual field of account.move.line to test 
if it's paid, it will save some code.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-742283-ara/+merge/55722
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-742283-ara.

_______________________________________________
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