[19.0][MIG] account_move_line_purchase_info #2187
Open
+4,376
−0
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
Contributor
|
Dependency is merged. Can you rebase? Thanks. |
Currently translated at 100,0% (5 of 5 strings) Translation: account-financial-tools-11.0/account-financial-tools-11.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-11-0/account-financial-tools-11-0-account_move_line_purchase_info/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_move_line_purchase_info/
Currently translated at 100.0% (5 of 5 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_move_line_purchase_info/pt_BR/
To force wheel regeneration, because the currently published wheel wrongly depends on odoo12-addon-purchase_stock which is a standard Odoo module.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_move_line_purchase_info/
Currently translated at 100.0% (5 of 5 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_move_line_purchase_info/pt/
Currently translated at 100.0% (6 of 6 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_move_line_purchase_info/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-13.0/account-financial-tools-13.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-13-0/account-financial-tools-13-0-account_move_line_purchase_info/
…ensure invoice count consistency
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-13.0/account-financial-tools-13.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-13-0/account-financial-tools-13-0-account_move_line_purchase_info/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_line_purchase_info/
Currently translated at 100.0% (7 of 7 strings) Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_line_purchase_info/pt_BR/
Currently translated at 100.0% (7 of 7 strings) Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_line_purchase_info/it/
Currently translated at 100.0% (7 of 7 strings) Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_line_purchase_info/sv/
Use _compute_display_name instead.
…try lines on revaluations Up to now, stock entries were considered in the revaluation process as they were stored on field invoice_lines. To avoid that we introduced a dedicated field to manage stock entries and avoid taking them into account in the revaluation odoo core process.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_move_line_purchase_info/
Currently translated at 100.0% (9 of 9 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_move_line_purchase_info/it/
Currently translated at 100.0% (9 of 9 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_move_line_purchase_info Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_move_line_purchase_info/tr/
a0a52ca to
98d7870
Compare
|
|
||
| - When a supplier invoice is created referencing purchase orders, the | ||
| purchase order line will be copied to the corresponding journal item. | ||
| - When a stock move is validated and generates a journal entry, the |
Contributor
There was a problem hiding this comment.
Can you remove this statement? it is not True in this version.
Also please mention in another paragraph how the changes in standard Odoo valuation affect the this module. Thanks.
You can say that since stock interim accounts are not used in v19 there is no need to add the PO info to the journal items, because there are no journal items.
AaronHForgeFlow
requested changes
Dec 22, 2025
Contributor
AaronHForgeFlow
left a comment
There was a problem hiding this comment.
Looking into more detail I think this module will be very simple in v19, just keeping the views so users can see the PO info when looking to journal items. Also keeping the _compute_display_name logic.
Anything else is not necessary I think.
98d7870 to
e62eb59
Compare
Contributor
Author
|
@AaronHForgeFlow, I have attended all comments. |
e62eb59 to
9eaadf9
Compare
Contributor
|
Thank you @JasminSForgeFlow Can you please mention in the readme and in the description of the PR that there is a big refactor due to the changes in Odoo standard stock valuation? |
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.
Standard Migration
@ForgeFlow