Line Amt wrongly recalculated for price list includes tax

Description

We found that the code from this ticket:
https://sourceforge.net/p/adempiere/bugs/600/

Included a couple of mistakes:
1 - is calculating taxes just for line level taxes - it must include also doclevel taxes
2 - is calculating wrongly the new tax - it just work for exempt case, but not when changing to a new tax

Environment

None

Activity

Show:
Carlos Ruiz
April 28, 2018, 11:11 AM

After lot of internal discussions we arrived to the conclusion that the code added in the mentioned old sourceforge ticket was wrong.

It was corrupting the data in the order line net amt - and the calculation was severely wrong when the two taxes had rates.

Consequently, the offending code was removed with commit https://bitbucket.org/idempiere/idempiere/commits/2da1128

Regards,

Carlos Ruiz

Fixed

Assignee

Carlos Ruiz

Reporter

Carlos Ruiz

Labels

None

Tested By

None

Priority

Major