Tryton - Issues

 

Issue6841

Title Wrong product invoices when changing product of stock move
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components purchase, sale
Assigned To ced Keywords review
Reviews 43601002,40751002
View: 43601002, 40751002

Created on 2017-10-11.18:06:07 by ced, last changed by roundup-bot.

Messages
New changeset 507b86438625 by Cédric Krier in branch 'default':
Invoice received product instead of purchased product
http://hg.tryton.org/modules/purchase/rev/507b86438625
New changeset 543bea11f640 by Cédric Krier in branch 'default':
Invoice shipped product instead of sold product
http://hg.tryton.org/modules/sale/rev/543bea11f640
review40751002 updated at https://codereview.tryton.org/40751002/#ps1
review43601002 updated at https://codereview.tryton.org/43601002/#ps1
msg36224 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-10-11.18:06:07
In a similar scenario as issue6839, when the user change the product of stock move linked to purchase or sale, the invoice generated still uses the product of the purchase or sale.
I do not think it is good for statistics nor coherence of the link between the invoice line and the stock moves.
Also the customer will receive a delivery note with different products as on the invoice.
So I think the sale and purchase should try to use, in case the invoice method is based on shipment, the product of the stock move. But I think it should keep the same taxes and unit price because those are part of the agreement. It is up to the user to change products with equivalent.
History
Date User Action Args
2017-11-11 16:09:43roundup-botsetmessages: + msg36835
2017-11-11 16:09:14roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg36834
2017-10-11 18:10:05reviewbotsetmessages: + msg36228
2017-10-11 18:09:59reviewbotsetnosy: + reviewbot
messages: + msg36227
2017-10-11 18:08:16cedsetstatus: in-progress -> testing
reviews: 43601002,40751002
keyword: + review
2017-10-11 18:06:07cedcreate

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