Refund a refund invoice is confusing

[Graeme Gellatly]

I am not sure what this functionality actually does, I tentatively tried to test it but got no result. Besides it makes no sense.

Open up a Paid/Open Credit Note under Customer Refunds in Accounting Menu. You will see a button called refund. It is confusing as anything. What am I refunding? If I was refunding the credit note, then the options would be to post Sales Journal, but they are for Sales Refund Journal. It just makes no sense to refund a refund.

On further inspection I discovered it creates an invoice of type out_invoice in the Sales Refund Journal which is nonsense anyway. Then following that it opens the Credit note screen with a blank list as obviously the type is different. In invoices there is a new draft invoice.

I'm not sure that the behaviour is necessarily wrong or undesirable. But maybe the options and wizard text need renaming in context of a credit note, because they do the opposite of what it says.

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

Hello Graeme,

If you are refunding "Customer refunds" it will create "Customer invoice" and If you are refunding "Customer Invoice" it will create "Customer refund" which is correct. Same for supplier invoices and supplier refund.

So what we have to fix when refunding from "Customer refunds" user has to select "Sales journals" in the wizard not "Sales refund journal"

Thanks for the reporting,
mra

[Ujjvala Collins (uco-openerp)]

Hello Graeme,

The fix has been committed on lp:~openerp-dev/openobject-addons/trunk-bug-789237-uco with revision 4780 <email address hidden>.

It will be merged into trunk-addons soon.

Thanks,
Ujjvala

[Fabien (Open ERP) (fp-tinyerp)]

The journal and view selection is correct now.
We did not renamed the button on the "refund invoice" form because as it shares the same view than the invoice form. Copy/pasting and changing on the refund invoice form would not be a good solution as it may be complex for others modules to inherit and extand all views. So, this is not perfect (the name of the button) but, now, it works fine.

[qdp (OpenERP) (qdp)]

released in revno 5068