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
[13.0][MIG] stock_available_mrp #803
Conversation
Hey @ps-tubtim, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
hello @ps-tubtim I can't enter the products screen after installing the module log: after add store=True to bom_id field to fix the previous issue, when I Create new product potential quantities values not calculated, but if I uninstall stock_available and install stock_available_mrp potential quantities values calculated and appear |
941fbe9
to
2bf56e6
Compare
@ps-tubtim You can rebase as #802 is merged |
53f70af
to
b011bb0
Compare
b011bb0
to
fca5f82
Compare
@ps-tubtim can you pls rebase and check travis build for errors |
fca5f82
to
b78a942
Compare
Hello @ps-tubtim a fix is included into #1070 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing returb
or (not b.product_id and b.product_tmpl_id == product.product_tmpl_id) | ||
) | ||
if product_boms: | ||
product.bom_id = first(product_boms) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
product.bom_id = first(product_boms) | |
product.bom_id = first(product_boms) | |
else: | |
product.bom_id = False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your review. Edited.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing return in compute function. This will lead to the impossibility to see products
We just merged this changes due to Odoo upstream changes that intruduced some incompatibilities. https://github.com/OCA/stock-logistics-warehouse/pull/1099/files Please include it in your migration |
Compute potential quantities for both product templates and variants. To keep the code simple, only the biggest potential of any single variant is accounted for in the template's potential. Take all levels of phantom BoM into account, respects validity dates etc. thanks to the use of the standard method _bom_explode, as suggested by @gdgellatly in OCA#5 (comment) Improve tests, rewritten in python. Adhere to new file/manifest/README conventions. Simplify copyright headers
sudo is not required since mrp.bom are readable to groups with access to the qty_x fields on a product. Moreover using sudo to retrive the bom will ignore the company_id defined on the bom
Record rules used to not be checked on stock quants, but now they are since Odoo's commit 2fd14db (odoo/odoo@2fd14db). In our test we changed the company of the products and BoMs but we neglected that the stock was not attached to the right company, and that made the test fail. To fix that, make the test inventory for the right company. Since there is a little inconsistency in the demo data with a negative quantity of an unrelated product, use the `partial` filter for the inventories instead of the `none` filter, so that no wrong inventory lines are added automatically.
…ch field use to compute potential
…ew + small improvements
* mrp_bom.name has been deleted. * mrp_bom_line.type moved to mrp_bom.type. * Fix missing group_mrp_user issue. * Change versions
…recordset + Condition never statisfied
…of immediately usable qty
…ilable_immediately
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_available_mrp Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_available_mrp/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_available_mrp Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_available_mrp/
Currently translated at 100.0% (7 of 7 strings) Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_available_mrp Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_available_mrp/zh_CN/
0159bce
to
11f612e
Compare
After this change odoo/odoo@3d34d58 it isn't possible to have an storable kit product anymore. The main issue is that we have to drop some uses cases that this module considered for this kind of situations.
11f612e
to
e38446a
Compare
f27011d
to
fc49cf9
Compare
This can be closed. The module has been migrated into another PR |
This PR is waiting for #802, you can always use this for readability.