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.
After losing a week on the demo store installation and eventually deciding to start all over, I installed Kickstart WITHOUT the demo store.
I did select some modules afterwards..which did not say anything about a demo store.
Now I appear to have example content/product types and example categories.
The example content/product types cannot be deleted or renamed. Renaming the machine name is not possible.
Comment | File | Size | Author |
---|---|---|---|
#14 | demo_store_not_selected-2486325-14.patch | 3.41 KB | mglaman |
Comments
Comment #1
bojanz CreditAttribution: bojanz at Centarro commentedConfirmed. At worst, we should take the creation of these types out of features, and put it into hook_install().
Comment #2
Denial CreditAttribution: Denial commentedIs dropping these content types from node_type manually a good idea in the meantime? I'm about to deliver a working version to a client and this is quite inconvenient.removing these content types from node_type doesn't work, they get recreated the moment the content types are loaded, guess it's a features thing.
Comment #3
xaxjust for reproduction purposes, updated a fully functional non-demo 2.21 kickstart to 2.24, and after running update.php hey presto all demo articles and taxonomy appeared. Must indeed be a features thing, but wonder why features installs these on a non-demo install?
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentednothing new here? i now have a store with lots of not desired taxonomy vocabularies and content types.
Comment #5
sandstrom CreditAttribution: sandstrom commentedI have installed commerce_kickstart-7.x-2.25 in my local environment with "Demo Store" option unchecked. I am able to rename the machine name of taxonomy vocabulary of "Product category" and also it is possible to delete/edit it's terms (sample category 1, sample category 2). I don't see any issue with these.
Please share your thoughts.
Thanks
Comment #6
Jawi CreditAttribution: Jawi as a volunteer and commentedSame issue here. Stil with latest Kcikstart version removing demostore content types and taxonomy lists is not possible.
Nice job!
Comment #7
bojanz CreditAttribution: bojanz at Centarro commented@Jawi
You're free to go be passive aggressive in another distribution's issue queue. Or better yet, make and share your own, so we can criticise you while contributing nothing.
@others
As with the other recent issues, this comes back to Features 2.x, and will take careful analysis to figure out how we can work around it.
I have ideas, but also a lack of time, so it might need to wait on mglaman to look into.
Comment #8
sandstrom CreditAttribution: sandstrom commentedComment #9
sandstrom CreditAttribution: sandstrom commentedComment #10
mglamanTagging this for the kanban board: http://localhost:9000/#/board/commerce_kickstart
Comment #11
mglamanComment #12
mglamanBy default commerce_kickstart_lite_product has always created two sample products and taxonomy terms which can be deleted.
Also, if you review the git blame for lite product, it has always provided a default product type and node type. Link: https://github.com/commerceguys/commerce_kickstart/blame/7.x-2.x/modules...
I'm marking this as works as designed. If you'd like to see this changed, I'm more than willing to move Product Display node type and Product product type into a hook_install() command that removes them out of code.
Comment #13
mglamanReopening, this is a bug. Per #12let's move the default items out of features.
Comment #14
mglamanHere is a PR with the changes and Travis run: https://github.com/commerceguys/commerce_kickstart/pull/118
Attaching patch as well. This allows the node type and product type to be deleted. The taxonomy vocabulary can be deleted regardless.
Comment #15
mglamanTravis passed, everything looks good from patch in #14. Any participants care to give it a run through?
Comment #16
mglamanFix merged in.