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

[15.0][FIX] stock_available_mrp: Use correct bom for computation #1887

Merged

Conversation

mt-software-de
Copy link
Contributor

@mt-software-de mt-software-de commented Nov 13, 2023

If a product has siblings and more then one has a bom first(product.bom_ids) could return a bom
for a diferent product, because bom_ids is a one2many field of product.template and this contains all boms of variants Therefor switch to the default methods of odoo
_bom_find

Forwardport of #1884
cc @rousseldenis @acsonefho

If a product has siblings and more then one has a bom
first(product.bom_ids) could return a bom
for a diferent product, because bom_ids is a one2many field
of product.template and this contains all boms of variants
Therefor switch to the default methods of odoo
_bom_find and _get_product2bom
Use _bom_find instead of deprecated _get_product2bom
@mt-software-de mt-software-de force-pushed the 15-fix-stock_available_mrp-explode branch from 8c1a276 to aca698d Compare November 13, 2023 14:50
@mt-software-de
Copy link
Contributor Author

ping @rousseldenis

@rousseldenis
Copy link
Sponsor Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-1887-by-rousseldenis-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot OCA-git-bot merged commit 214641f into OCA:15.0 Nov 21, 2023
5 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9d9cd8f. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants