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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz’s picture

Priority: Normal » Major

Confirmed. At worst, we should take the creation of these types out of features, and put it into hook_install().

Denial’s picture

Is 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.

xax’s picture

just 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?

Anonymous’s picture

nothing new here? i now have a store with lots of not desired taxonomy vocabularies and content types.

sandstrom’s picture

I 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

Jawi’s picture

Same issue here. Stil with latest Kcikstart version removing demostore content types and taxonomy lists is not possible.

Nice job!

bojanz’s picture

@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.

sandstrom’s picture

Status: Active » Fixed
sandstrom’s picture

Status: Fixed » Active
mglaman’s picture

Issue tags: +sprint

Tagging this for the kanban board: http://localhost:9000/#/board/commerce_kickstart

mglaman’s picture

Assigned: Unassigned » mglaman
mglaman’s picture

Assigned: mglaman » Unassigned
Issue summary: View changes
Status: Active » Closed (works as designed)

By default commerce_kickstart_lite_product has always created two sample products and taxonomy terms which can be deleted.

Title,Description,Category,SKU
Product 1,Nunc vulputate ligula a quam. Sed sit amet ipsum eu augue tempor sollicitudin.,"Sample Category 1","sample1"
Product 2,Nunc vulputate ligula a quam. Sed sit amet ipsum eu augue tempor sollicitudin.,"Sample Category 2","sample2"

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.

mglaman’s picture

Status: Closed (works as designed) » Active

Reopening, this is a bug. Per #12let's move the default items out of features.

mglaman’s picture

Status: Active » Needs review
FileSize
3.41 KB

Here 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.

mglaman’s picture

Travis passed, everything looks good from patch in #14. Any participants care to give it a run through?

mglaman’s picture

Status: Needs review » Fixed

Fix merged in.

  • mglaman committed 66c2ffd on 7.x-2.x
    Issue #2486325: Demo store not selected - still undeletable example...
  • mglaman committed 8f6616b on 7.x-2.x
    Merge pull request #118 from mglaman/2486325-demo-store-not-selected...

Status: Fixed » Closed (fixed)

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