[18.0][MIG] product_operating_unit: Migration to 18.0#830
Merged
OCA-git-bot merged 39 commits intoOCA:18.0from Mar 27, 2026
Merged
[18.0][MIG] product_operating_unit: Migration to 18.0#830OCA-git-bot merged 39 commits intoOCA:18.0from
OCA-git-bot merged 39 commits intoOCA:18.0from
Conversation
…the product template.
Co-Authored-By: Patrick Wilson <36892066+patrickrwilson@users.noreply.github.com>
…the product template.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: operating-unit-16.0/operating-unit-16.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-16-0/operating-unit-16-0-product_operating_unit/
Currently translated at 100.0% (7 of 7 strings) Translation: operating-unit-16.0/operating-unit-16.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-16-0/operating-unit-16-0-product_operating_unit/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: operating-unit-16.0/operating-unit-16.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-16-0/operating-unit-16-0-product_operating_unit/
Currently translated at 14.2% (1 of 7 strings) Translation: operating-unit-16.0/operating-unit-16.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-16-0/operating-unit-16-0-product_operating_unit/ja/
This PR fixes an issue with the method operating_unit_default_get where it would return an operating unit even for a company that was not active, which would cause all sorts of issues Steps to reproduce on runboat, taking as example purchase_operating_unit: - User has a default ou belonging to Company 1 - Switch to Company 2 (make sure Company 1 is not active at all) - Try to create a Purchase Order The issue arises because the operating_unit field is pre-compiled on the PO with an Operating Unit whose company is inactive, and the onchanges cannot find or access related data.
Currently translated at 100.0% (7 of 7 strings) Translation: operating-unit-16.0/operating-unit-16.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-16-0/operating-unit-16-0-product_operating_unit/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: operating-unit-17.0/operating-unit-17.0-product_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-17-0/operating-unit-17-0-product_operating_unit/
Quan-nhm
approved these changes
Feb 27, 2026
Quan-nhm
left a comment
Quan-nhm
left a comment
There was a problem hiding this comment.
Code review and functional test LGTM
Contributor
|
This PR has the |
Contributor
|
Can you check commit history? the last commit is a merge commit, it should not be. |
hieulucky111
force-pushed
the
18.0-mig-product_operating_unit
branch
from
703482a to
b254bcc
Compare
Contributor
Author
Hello, i have fixed the commit history. |
Contributor
|
@hieulucky111 ok thanks. /ocabot merge nobump |
Contributor
|
On my way to merge this fine PR! |
Contributor
|
Congratulations, your PR was merged at 8a6ec74. Thanks a lot for contributing to OCA. ❤️ |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
17 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.
No description provided.