Tryton - Issues

 

Issue6248

Title Tax of loop not used in determination of taxes
Priority bug Status chatting
Superseder Nosy List ced, pokoli, ralf, reviewbot
Type behavior Components sale_shipment_cost
Assigned To ralf Keywords review
Reviews 33861002
View: 33861002

Created on 2017-02-07.11:47:27 by ralf, last changed by reviewbot.

Messages
review33861002 updated at https://codereview.tryton.org/33861002/#ps20001
msg32073 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-02-20.17:41:28
Indeed there should be two calls to apply one for each tax and one with None like it is done in other modules.
review33861002 updated at https://codereview.tryton.org/33861002/#ps1
msg31753 (view) Author: [hidden] (ralf) Date: 2017-02-07.11:47:26
In http://hg.tryton.org/modules/sale_shipment_cost/file/tip/stock.py#l114 there is a loop about the product.customer_taxes_used. But instead of using this tax in calling the apply-method there is a 'None' used.
I think this should be the tax, otherwise there is no correct match depending on tax possible.
History
Date User Action Args
2017-02-20 18:10:17reviewbotsetmessages: + msg32074
2017-02-20 17:41:28cedsetnosy: + ced
messages: + msg32073
2017-02-20 17:35:45reviewbotsetstatus: unread -> chatting
nosy: + reviewbot
messages: + msg32071
2017-02-20 17:21:27ralfsetreviews: 33861002
keyword: + review
2017-02-08 11:06:48ralfsetassignedto: ralf
2017-02-07 14:56:15pokolisetnosy: + pokoli
2017-02-07 11:47:27ralfcreate

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