Tryton - Issues

 

Issue7682

Title Set cancel state on refunded invoice
Priority feature Status resolved
Superseder Nosy List Timitos, ced, reviewbot, roundup-bot
Type feature request Components account_invoice
Assigned To ced Keywords review
Reviews 48581002
View: 48581002

Created on 2018-09-06.12:15:28 by ced, last changed by roundup-bot.

Messages
New changeset 8344345baba2 by Cédric Krier in branch 'default':
Update scenario as credit invoice has cancel state
https://hg.tryton.org/tryton-env/rev/8344345baba2
New changeset 90acbec912b7 by Cédric Krier in branch 'default':
Update scenario as credit invoice has cancel state
https://hg.tryton.org/modules/analytic_invoice/rev/90acbec912b7
New changeset 8562668620b7 by Cédric Krier in branch 'default':
Set cancel state when refund with credit note
https://hg.tryton.org/tryton-env/rev/8562668620b7
New changeset 22eabf722a91 by Cédric Krier in branch 'default':
Set cancel state when refund with credit note
https://hg.tryton.org/modules/account_invoice/rev/22eabf722a91
review48581002 updated at https://codereview.tryton.org/48581002/#ps40001
review48581002 updated at https://codereview.tryton.org/48581002/#ps20001
review48581002 updated at https://codereview.tryton.org/48581002/#ps1
msg43740 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-09-06.12:15:28
There are a scenario which is not very good. If a sale is set to ship on invoice paid and that the posted invoice is refund with a credit note. In this case, the invoice is marked as paid and so the sale will create the shipment instead of being in invoice exception.
So I think we could set the state of the invoice when refunded to cancel. For the accounting it does not change anything as the moves are still the same. Also the canceled invoice can not be deleted as it has a number.
History
Date User Action Args
2018-11-02 20:17:05roundup-botsetmessages: + msg44731
2018-11-02 20:07:39roundup-botsetmessages: + msg44725
2018-10-15 23:00:08roundup-botsetmessages: + msg44430
2018-10-15 23:00:05roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg44429
2018-10-08 12:31:50reviewbotsetmessages: + msg44361
2018-10-04 00:28:08reviewbotsetmessages: + msg44305
2018-09-06 15:33:46reviewbotsetnosy: + reviewbot
messages: + msg43745
2018-09-06 15:19:44cedsetstatus: in-progress -> testing
reviews: 48581002
keyword: + review
2018-09-06 12:29:15Timitossetnosy: + Timitos
2018-09-06 12:15:28cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)