Describe your bug or feature request.

The ProductDefaultVariationEvent allow changing the default variation of a product but doesn't allow to unset it (if no custom conditions are met for example). We should allow setting a NULL default variation.

CommentFileSizeAuthor
#2 commerce-product_default_variation_allow_null-3277852-2.patch872 bytesAerzas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aerzas created an issue. See original summary.

Aerzas’s picture

Aerzas CreditAttribution: Aerzas at Watch Dreamer SA commented
FileSize
commerce-product_default_variation_allow_null-3277852-2.patch872 bytes

Here is a patch proposal

Aerzas’s picture

Aerzas CreditAttribution: Aerzas at Watch Dreamer SA commented
Status: Active » Needs review
Issue tags: +Needs review

The test error seems unrelated.

  • jsacksick committed 460df29 on 8.x-2.x authored by Aerzas 
    Issue #3277852 by Aerzas: ProductDefaultVariationEvent::...

  • jsacksick committed 713ce55 on 3.0.x authored by Aerzas 
    Issue #3277852 by Aerzas: ProductDefaultVariationEvent::...
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.