Closed (fixed)
Project:
Commerce Kickstart
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
15 Jul 2012 at 21:36 UTC
Updated:
13 Sep 2012 at 19:17 UTC
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 commentedComment #2
guguss commentedComment #3
theo_ commentedAutoSKU configuration for 'product' product type added.
https://code.drupalcommerce.org/206
Comment #4
bojanz commentedMerged.
Comment #6
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 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 commentedSuper! thank you very much for the quick reply. And sorry again about the unsolicited PM on IRC.
Comment #9
bojanz commentedRemoved the offending code in https://code.drupalcommerce.org/#/c/426/.