[19.0][MIG] stock_picking_report_valued: Migration to 19.0 #453
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For avoiding problems if you don't have access to sales orders (stricter warehouse users, inter-company records...).
- This commit odoo/odoo@41bea41 breaked the inheritance.
In certain conditions of the DB (i.e. integration tests), previous code doesn't assure the price is going to be 100.
[FIX] stock_picking_report_valued: Wrong taxes amount if round globally (OCA#56) [FIX] stock_picking_report_valued: Remove round_method condition because don't apply here [FIX] stock_picking_report_valued: price_reduce is stored with product decimal precision [FIX] stock_picking_report_valued: division_by_zero when unrelated sale order [FIX] stock_picking_report_valued: Improve code [FIX] stock_picking_report_valued: Multiple taxes in sale line (WIP)
…ferences TT17492 TT26034 TT33957 TT34954
The fix in fac3997 didn't consider move lines without a linked sale line. That would end up in inconsistent calculations and even the original line rewriting. TT38863
- In standard report, Odoo always show qty_done
…ots in delivery slip
Currently translated at 100.0% (23 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/es/
Currently translated at 100.0% (23 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/pt/
Currently translated at 73.9% (17 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/fr_FR/
Currently translated at 100.0% (23 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/es/
Currently translated at 100.0% (23 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/es/
Currently translated at 100.0% (23 of 23 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_valued/it/
Currently translated at 100.0% (22 of 22 strings) Translation: stock-logistics-reporting-17.0/stock-logistics-reporting-17.0-stock_picking_report_valued Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-17-0/stock-logistics-reporting-17-0-stock_picking_report_valued/it/
We make sure that the currency_id field always has a value to avoid errors. For example, when installing this module with stock_picking_report_delivery_cost and running the tests, there is a case where the picking has no currency defined, which blocks the CI.
Member
|
/ocabot migration stock_picking_report_valued Please take into account #450 |
Author
|
@pedrobaeza this is added |
arantxa-s73
force-pushed
the
19.0-mig-stock_picking_report_valued
branch
from
cf054f1 to
198f1a2
Compare
pablo-cort-s73
approved these changes
Dec 1, 2025
Contributor
|
This PR has the |
etobella
reviewed
Dec 31, 2025
arantxa-s73
force-pushed
the
19.0-mig-stock_picking_report_valued
branch
from
198f1a2 to
2aead4a
Compare
The way the fields were calculated has been modified, since after performing the new() function, it recalculates the fields and loses them, only showing the unit price at the end. Therefore, it has been modified so that if it has a quantity different from the sales order or a different UOM, it first calculates the subtotal price and then performs the new() function. This way, the prices are calculated correctly in the report and the lines.
arantxa-s73
force-pushed
the
19.0-mig-stock_picking_report_valued
branch
from
2aead4a to
3d34d01
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Module migrated to version 19.0