[6.0][account_voucher] customer payment seems broken

[Raphaël Valyi - http://www.akretion.com]

Hello,

using addons #3894, with demo data + account_voucher module + default chart of account.
Try to encode a new "customer payment" with the following data:
- partner: Axelor
- journal: bank (test)

Also encode one statement line specifying only the "amount" field with 100.

Now try to validate the voucher. Well it will valid!!
But there are critical bugs:
1) the voucher "Paid Amount" field has been left to 0. I think some check was expected against the sum of the lines, just like for bank statements instead. Look in the voucher list view for instance, that voucher has an amount of 0 while it's wrong!
2) look at the account moves in the "journal items" tab:
both debit an credit moves are done using the same partner account!!!
Instead, the debit move line should be done in the account specified at the voucher level, just like a bank statement would do.

This makes "customer payment" via voucher unusable currently I think.
Hope this helps

[qdp (OpenERP) (qdp)]

Hello,

thanks for the bug report. Unless i agree with you that the current behavior is not good, let me clarify some points:
*we are currently reviewing the account_voucher module for it to accept write-off. This feature won't allow us to make some check rules like "the total of the voucher that should be equal to the total of the mount lines".
*the problem here seems more that you created a _new_ line (like if it was an existing invoice for that partner) wich is not what that screen was intented to: i think that's there we should have put some check.
*i guess that the above point is guilty of the weird creation of the accounting entries that you described on point 2.

I'm confirming this bug because indeed i think that we still have some work to do on that module (and should be done for rc2), but also i'd like to emphasize that our current working process doesn't allow anymore the drivers to set the milestone themselves but i don't think that we already communicate about that... so in the meanwhile, please do not put milestone yourself otherwise we can't keep track of wich bugs are assigned to our development teams.

Thanks for your contribution,
Quentin

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

qdp,

Yous basically said "customer payment" vouchers are not intended for advance payments, is that what you mean? Then what do you recommend to use instead?
Note: I never liked the current "advance payment" from the order view as it relates to only one product and at least untill recently it was making an invoice too, which is not something we want. We often need that after registering a sale order, we have the ability to register a customer payment. Invoice might only come later, possibly upon the deliveries.
This is also related to that remark:
http://openerp-expert-accounting.71625.n3.nabble.com/invoice-payment-in-OpenERP-v6-tt1849762.html#a1849762

In v5, we were making bank statements for this. Are you sure we should continue using bank statements instead of vouchers for this? I think those are not real bank statements, so what are we supposed to use then?

[Mustufa Rangwala (Open ERP) (mra-tinyerp)]

Hello Raphaël,

As qdp said we are working on the write-off with Account voucher, And for the Journal items on Voucher it is working fine as it uses account specified at the voucher level.

Thank you,
Mustufa

[qdp (OpenERP) (qdp)]

the account_voucher moduel has been reviewed and improved in a way describe in comment #1... so i'm setting this bug report as invalid because i think we should create a new bug if needed.

This feature will soon be merged in trunk.
Thanks for the contribution