problem with pricing rule since Aug 2023 , not yet solved at version v14.47.1

[thida-bcn]

          > When a user changes the exchange rate for a currency in a SI, the rate of the items are automatically updated. But the items which have a discount applied get an incorrect rate because the discount isn't applied again after the rate update. This is due to the `apply_price_list` function (which is called from `conversion_rate` when the exchange rate changes) not applying discount once the rate changes, so fixed that.

there was problem with pricing rule, before update version discount apply well
after updating patch screen refresh frequently and discount amount change

Originally posted by @thida-bcn in #36125 (comment)

[anandbaburajan]

@thida-bcn can you please share the steps to reproduce the issue or a screen recording?

[thida-bcn]

My discount amount is 4000
system changed frequently
https://github.com/frappe/erpnext/assets/62122640/1d90aac7-3b82-4373-8d38-6bf6c2dce163

[anandbaburajan]

@thida-bcn can you share your pricing rule too?

[thida-bcn]

Item Price

Price List

Pricing Rule

Item with Multi UOM

[thida-bcn]

@thida-bcn can you share your pricing rule too?

Item with Mulit UOM
Pricing Rule Discount Amount

[thida-bcn]

tested with v15 frappe cloud
https://github.com/frappe/erpnext/assets/62122640/68eba01c-a780-4cd0-9f94-4186f6b14c64

[thida-bcn]

1 PK = 35 Pack
Discount Amount 4000 for PK
Price of Pack 8643
Pack / PK 35
Price of PK 302505
Discount of PK 4000
Price of PK after Discount 298505
Now, system calculation changed to Discount of PK 4000.01 or 3999.99
due to decimal issues, we can't proceed our operation

[thida-bcn]

@thida-bcn can you share your pricing rule too?

hello, any progress?

[anandbaburajan]

@thida-bcn someone will look into this asap. Notes for the dev: according to @thida-bcn, this issue started after #36125 was merged.

Update: doesn't seem like it's related to #36125, but due to some "tax breakup decimal issue". More details in the comments below.

[thida-bcn]

@thida-bcn someone will look into this asap. Notes for the dev: according to @thida-bcn, this issue started after #36125 was merged.

hello,
we know that you all busy with event
the issue is not solved out in latest patch
we need it urgent
thanks

[thida-bcn]

@thida-bcn can you please share the steps to reproduce the issue or a screen recording?

hello, may i know progress?
It had been problems since 2 weeks ago

[thida-bcn]

latest version (problem) 14.39.0

version 13 (correct) 13.41.1

problem caused by tax breakup decimal issue

[thida-bcn]

@thida-bcn someone will look into this asap. Notes for the dev: according to @thida-bcn, this issue started after #36125 was merged.

hello sir,
may i know progress?

[anandbaburajan]

@thida-bcn what do you mean by "latest version" in this comment? Is it the latest version of develop or v14 branch? Also you mention that the issue is not there on v13, but #36125 is there on v13 as well, so it doesn't seem like the issue is due to #36125. It could be due to #36235 since you've mentioned tax breakup issues.

Thanks for your patience. Like I said earlier, someone will look into this asap.

[thida-bcn]

@thida-bcn what do you mean by "latest version" in this comment? Is it the latest version of develop or v14 branch? Also you mention that the issue is not there on v13, but #36125 is there on v13 as well, so it doesn't seem like the issue is due to #36125. It could be due to #36235 since you've mentioned tax breakup issues.

Thanks for your patience. Like I said earlier, someone will look into this asap.

#36876 (comment)

[thida-bcn]

[thida-bcn]

@thida-bcn what do you mean by "latest version" in this comment? Is it the latest version of develop or v14 branch? Also you mention that the issue is not there on v13, but #36125 is there on v13 as well, so it doesn't seem like the issue is due to #36125. It could be due to #36235 since you've mentioned tax breakup issues.

Thanks for your patience. Like I said earlier, someone will look into this asap.

it was still happening version 14.44.0

[thida-bcn]

@thida-bcn can you share your pricing rule too?

hi, issue still happening v14.47.1

[thida-bcn]

@thida-bcn what do you mean by "latest version" in this comment? Is it the latest version of develop or v14 branch? Also you mention that the issue is not there on v13, but #36125 is there on v13 as well, so it doesn't seem like the issue is due to #36125. It could be due to #36235 since you've mentioned tax breakup issues.

Thanks for your patience. Like I said earlier, someone will look into this asap.

hello, may i know progress