Reverting a feature would not add AutoSKU settings to a product display.

The reason for this is that commerce_autosku_commerce_product_type_update() gets this array:

Array
(
    [type] => the_type
    [name] => The_Type
    [description] => Desc.
    [help] => 
    [revision] => 1
    [module] => commerce_product_ui
    [autosku] => Array
        (
            [product_type] => the_type
            [pattern] => [commerce-product:title]-[commerce-product:field_the_type_size]-[commerce-product:field_color]
            [advanced] => Array
                (
                    [update_existing] => 0
                    [hide_sku] => 1
                    [case] => uppercase
                )
        )
)

But it expects to see autosku_status which isn't there.

Fix posted in comment.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kotnik’s picture

Status: Active » Needs review
FileSize
1.16 KB

Patch attached.

kotnik’s picture

Status: Needs review » Needs work
kotnik’s picture

Status: Needs work » Needs review
FileSize
1.12 KB

Removed redundancy.

kotnik’s picture

Missed check during insert. Fixed. Now really ready for a review.

rbayliss’s picture

Looks good except that changes made through the UI then ignore the status checkbox. Rerolled with an extra check for when autosku_status is set to FALSE. Can you give this a try and let me know if it works for you?

kotnik’s picture

Status: Needs review » Reviewed & tested by the community

All good, thanks rbayliss.

rbayliss’s picture

Status: Reviewed & tested by the community » Fixed

Great! Committed to dev. Do you need a stable release to be able to get rid of the patch in Commerce Kickstart?

kotnik’s picture

Thanks rbayliss.

Stable release would be much appreciated, if you feel it can be released.

Status: Fixed » Closed (fixed)

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

jcandan’s picture

+1 to get to stable release please!