Skip to content
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

[FW][FIX] {purchase_,}mrp: display routes for subcontracted boms #159580

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 27, 2024

  1. Fixes an issue where the parent product wasn't correctly set when computing the routes of a component, leading on components from subcontracted products displaying the wrong route (as it was trying to resupply the selected warehouse instead of the subcontracted location).
    Note: This part of the fix is only necessary up to version saas-16.4, as it was fixed from 17.0 onwards through f9c58a6.

  2. If a route is found when searching for subcontracting routes but doesn't lead to a way to resupply the stock (either buy buying or manufacturing something), then ignore the found rules and revert to the default of trying to resupply the stock location.
    This avoids issue when using reordering rules to resupply the subcontracted location instead, where the 'Buy' route would be hidden even if it was selected.


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #159383
Forward-Port-Of: #158785

@robodoo
Copy link
Contributor

robodoo commented Mar 27, 2024

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Mar 27, 2024
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 27, 2024
If a route is found when searching for subcontracting routes but
doesn't lead to a way to resupply the stock (either buy buying or
manufacturing something), then ignore the found rules and revert to the
default of trying to resupply the stock location.
This avoids issue when using reordering rules to resupply the
subcontracted location instead, where the 'Buy' route would be hidden
even if it was selected.

X-original-commit: c7b84a4
@clesgow clesgow force-pushed the master-16.0-fix-bom-overview-subcontracting-quwo-MAjU-fw branch from 0debbce to cf6fc85 Compare March 28, 2024 10:08
@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 28, 2024

@clesgow @svs-odoo this PR was modified / updated and has become a normal PR. It should be merged the normal way (via @robodoo)

@svs-odoo
Copy link
Contributor

robodoo r+

robodoo pushed a commit that referenced this pull request Mar 30, 2024
If a route is found when searching for subcontracting routes but
doesn't lead to a way to resupply the stock (either buy buying or
manufacturing something), then ignore the found rules and revert to the
default of trying to resupply the stock location.
This avoids issue when using reordering rules to resupply the
subcontracted location instead, where the 'Buy' route would be hidden
even if it was selected.

closes #159580

X-original-commit: c7b84a4
Signed-off-by: Steve Van Essche <svs@odoo.com>
@robodoo robodoo closed this Mar 31, 2024
@robodoo robodoo added the 17.3 label Mar 31, 2024
@fw-bot fw-bot deleted the master-16.0-fix-bom-overview-subcontracting-quwo-MAjU-fw branch April 14, 2024 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
17.3 forwardport This PR was created by @fw-bot RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants