New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IDEMPIERE-5671: Fixing Line net amount has error due to rounding. #1782
IDEMPIERE-5671: Fixing Line net amount has error due to rounding. #1782
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @dpansheriya - I tested this successfully.
However, I think it is wrong to round the price based on the UOM precision. The UOM precision is for quantities and is being used for rounding prices in several places.
Also, the prices can accept 12 decimal points when entered manually in the same product UOM, but the unit price is being rounded when there is a UOM conversion, I think both cases must accept the 12 decimals for consistency.
Can you please integrate the following patch:
pr1782PR.patch.txt
Carlos, I think 12 decimal hard coding is not good. I think we should round priceActual to costing precision as that is what used for costing too. If 12 digits precision printed on reports, it may not be good. also some how it must be controlled based on countries |
@dpansheriya - the 12 decimal hard coding is already there when you enter a price manually, so, I'm not introducing something new, but making the price rounding consistent when manual vs UOM-converted. If I remember correctly that was implemented to solve a big issue when doing multiple invoices for a single order. About your concern when printing, in invoices the printed price is the PriceEntered, and if for any reason you need to print the UOM-converted price, then I think is more accurate to print 12.8033 than 12.8 |
… rounding error Co-authored-by: carg67@gmail.com
|
Pull Request Checklist
Tests
Documentation