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

[FIX] product: Issues with product configurator with single attribute values #32086

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@simongoffin
Copy link
Contributor

commented Mar 25, 2019

When creating several attribute lines for the same attribute, the function _update_product_template_attribute_values
in model 'product.template.attribute.line' only kept the product.template.attribute.value records of the last
created line. So it generated issues with the product configurator because no value was founded for certain attribute line
in function _get_possible_combinations defined in model 'product.template'.

Now it's forbidden to create several attribute lines for the same attribute.

opw:1950786

[FIX] product: Issues with product configurator with single attribute…
… values

When creating several attribute lines for the same attribute, the function _update_product_template_attribute_values
in model 'product.template.attribute.line' only kept the product.template.attribute.value records of the last
created line. So it generated issues with the product configurator because no value was founded for certain attribute line
in function _get_possible_combinations defined in model 'product.template'.

Now it's forbidden to create several attribute lines for the same attribute.

opw:1950786

@simongoffin simongoffin force-pushed the odoo-dev:12.0-opw-1950786-fix-sig branch from 81bad59 to ce00fcb Mar 26, 2019

@robodoo robodoo removed the CI 🤖 label Mar 26, 2019

@simongoffin

This comment has been minimized.

Copy link
Contributor Author

commented Mar 26, 2019

robodoo r+

@robodoo

This comment has been minimized.

Copy link
Contributor

commented Mar 26, 2019

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.