-
-
Notifications
You must be signed in to change notification settings - Fork 660
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
[16.0][ADD] stock_move_value #1777
base: 16.0
Are you sure you want to change the base?
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.
@AungKoKoLin1997 Can you please add a CONTEXT.md as the intention is probably not clear to most people.
"""
Odoo calculates the cost of outgoing stock moves based on the product's costing method, no matter what the cause of the operation is. This can be problematic especially when a user just reverts the purchase receipt operation which was done by mistake (e.g., processing the wrong receipt picking) for FIFO and AVCO products, as the returned value is not the same as the value of the original receipt. Some auditors might see such inconsistencies as the result of intended manipulation on the financial performance.
For FIFO products, you can use the OCA module stock_account_fifo_return_origin (yet to be added with #1649) to return the same value as the receipt. However, there is no complete solution for AVCO products.
This module aims to provide users with a means to identify discrepancies resulting from their purchase return operations, allowing them to take necessary actions (e.g., inventory revaluation) as appropriate.
"""
296ca3b
to
c813214
Compare
c813214
to
d601bc0
Compare
@yostashiro I added 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.
Code review and functional test.
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 and functional review
This PR has the |
This module adds value fields to the stock move model, to add visibility of how the move has affected the stock valuation.
@qrtl QT4685