Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Title says it all. We're using the default product type provided by Commerce, and it has no autosku configuration of course.
So, commerce_kickstart_lite_product (or whatever we rename it to) should have some code in hook_install() that configures autosku correctly.
Comments
Comment #1
GuGuss CreditAttribution: GuGuss commentedComment #2
GuGuss CreditAttribution: GuGuss commentedComment #3
theo_ CreditAttribution: theo_ commentedAutoSKU configuration for 'product' product type added.
https://code.drupalcommerce.org/206
Comment #4
bojanz CreditAttribution: bojanz commentedMerged.
Comment #6
ericmulder1980 CreditAttribution: ericmulder1980 commentedThe result of this fix is that the autosku configuration for the default product type 'product' cannot be changed. Why would you hardcode something like a field configuration into hook_commerce_product_type_info_alter()?
I have been checking database tables and different parts of code just to find out that it is impossible for me to change this setting trough the config. If this hook is needed for the install profile than at least make it possible to override this in the product type edit form.
Change it to something like this?
Comment #7
bojanz CreditAttribution: bojanz commentedYes, that's wrong.
I will be removing AutoSKU completely (as well as size and color attributes) from the no demo store product module, so that code will go away in a few hours.
Comment #8
ericmulder1980 CreditAttribution: ericmulder1980 commentedSuper! thank you very much for the quick reply. And sorry again about the unsolicited PM on IRC.
Comment #9
bojanz CreditAttribution: bojanz commentedRemoved the offending code in https://code.drupalcommerce.org/#/c/426/.