-
-
Notifications
You must be signed in to change notification settings - Fork 695
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
[15.0][IMP] stock_request: Change rounding method to HALF-UP in quantities computation #1989
[15.0][IMP] stock_request: Change rounding method to HALF-UP in quantities computation #1989
Conversation
Maybe the same in request.qty_done and request.qty_cancelled ? |
a403df6
to
a16e260
Compare
You are right @AaronHForgeFlow , I've just applied what you suggest |
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.
LGTM
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.
code review LGTM
This PR has the |
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.
There are more usage in other files (examples:
rec.requested_product_qty = rec.product_uom_id._compute_quantity( |
rec.product_qty = rec.product_uom_id._compute_quantity( |
One more request, I think it would be great to depict the issue with a test case if possible.
0ee24b4
to
3079e6c
Compare
@LoisRForgeFlow comments attended |
3079e6c
to
9770a58
Compare
The error in the test are caused by another module. I have done a fix: #2026 |
9770a58
to
0ba93ed
Compare
cc060be
to
a1b4f67
Compare
a1b4f67
to
de2a85e
Compare
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.
Let's move this one forward
/ocabot merge patch
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at eb34b0e. Thanks a lot for contributing to OCA. ❤️ |
Switched the rounding method from "UP" (if not indicated default is "UP") to "HALF-UP" in the
qty_in_progress
computation to ensure more accurate roundings.