Skip to content

[13.0] [OU-FIX] product: archive duplicated variants - #6070

Merged
pedrobaeza merged 1 commit into
OCA:13.0from
eLBati:13.0-ou_fix-product-duplicated-variants
Oct 29, 2026
Merged

pedrobaeza merged 1 commit into
OCA:13.0from
eLBati:13.0-ou_fix-product-duplicated-variants

Conversation

@eLBati

@eLBati eLBati commented Oct 28, 2026

Copy link
Copy Markdown
Member

Before odoo/odoo@088edee2ce71 ("[FIX] product: do not create duplicate variants", odoo/odoo#35013), Odoo 12 reactivated archived variants whose combination already had an active variant (sell the variants, remove an attribute from the template, add it again). That fix prevents new duplicates but doesn't clean the existing ones, so v12 databases can contain several active variants for the same combination, and the creation of the product_product_combination_unique index on v13 fails.

Variants without attribute values get an empty combination_indices, as Odoo computes it, so they are also covered. For each template and combination, the variant with the lowest ID is kept active and the others are archived: unlike #2513, no records are merged or deleted.

Before odoo/odoo@088edee2ce71 ("[FIX] product: do not create duplicate
variants", odoo/odoo#35013), Odoo 12 reactivated archived variants whose
combination already had an active variant (sell the variants, remove an
attribute from the template, add it again). That fix prevents new
duplicates but doesn't clean the existing ones, so v12 databases can
contain several active variants for the same combination, and the
creation of the product_product_combination_unique index on v13 fails.

Variants without attribute values get an empty combination_indices, as
Odoo computes it, so they are also covered. For each template and
combination, the variant with the lowest ID is kept active and the
others are archived: unlike OCA#2513, no records are merged or deleted.

Assisted-by: Claude Opus 5.5
@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor

That's an interesting approach, an alternative to my old PR.

@pedrobaeza
pedrobaeza merged commit 8829b13 into OCA:13.0 Oct 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants