Every once in a while I get requests on Crumbs that refer to the 7.x-1.x version, and are already fixed in 7.x-2.x.
E.g. #2408879: Commerce Kickstart Checkout Use Case
People then tell me that Kickstart does not want them to upgrade any of the included modules.

I have not studied Kickstart myself, and don't know if this true, or some kind of misunderstanding.

If there is a safe way for people to upgrade, maybe this should be done first (adding it to the docs, with an "at your own risk"), and then after a while, Kickstart can officially switch to the crumbs-7.x-2.x.
Of course I will be happy for any testing and feedback from the Kickstart side.
Especially the upgrade path needs a special look, I think.

Comments

suntower’s picture

I guess I'm the guinea pig.

I updated to 7.x-2 and I am -more- confused than ever. I basically need -5- minutes of your time to understand the UI. I (literally) cannot figure out how to enable various plug-ins or change their weights or... basically -anything-. I cannot be the only one who feels the same.

In return, I will HAPPILY write a proper user guide. I've written professional online help for 25 years.

I contacted you via IRC a few weeks ago on this. I will follow up w an e-mail.

TIA.

donquixote’s picture

Ok, catch me on IRC.

donquixote’s picture

@ Kickstart maintainers:
I don't know if you share the concerns raised by suntower. The admin UI has always been a controversial subject, and I have yet not found a "perfect" UI for the plugin management.
Either way, users that want to upgrade should not be stopped by Kickstart.

Maybe the explanation is that you are not supposed to modify the code within the kickstart folders, but you can download alternate module versions and place them in sites/all/modules/contrib, which (I assume) has priority over the modules in the Kickstart folders. So you would have crumbs-7.x-1.x in the kickstart folders, but crumbs-7.x-2.x in the sites/all/modules/contrib folder.

Is this correct?
@suntower: Is this the way you installed crumbs-7.x-2.x?

suntower’s picture

@donquixote

I replaced the existing crumbs folder which is inside the kickstart profile.

/profiles/commerce_kickstart/modules/contrib/crumbs

I did -not- install it in sites/all/modules

I think -one- issue is that the 7.1 v included with kickstart contains a class in crumbs.taxonomy.inc. for commerce products.

I'm sending you an e-mail.

donquixote’s picture

I think -one- issue is that the 7.1 v included with kickstart contains a class in crumbs.taxonomy.inc. for commerce products.

I had a look at the module included in Kickstart, and I found that it is all unmodified native code from Crumbs 7.x-1.x.
And the product-specific code in there only applies on sites where product/% is a path to a commerce product. This is the case if you use the commerce_product_page module, but not with native Commerce only. In native Commerce, product entities do not have a relevant user-facing url where you would care about the breadcrumb. (Users visit the "product display" node url, not the product url).

The 7.x-2.x branch still supports term references from commerce products. But it does so in a generic way, by supporting term reference fields on all entity paths that are known to Crumbs. E.g. if the bean module would register a bean/% route for bean entities, and tell Crumbs about this route, then the term reference, entityreference and other entity-related Crumbs plugins would all work for bean/% magically.

bojanz’s picture

Title: Use Crumbs 7.x-2.*, or tell people how to upgrade. » Use Crumbs 7.x-2.x

I would be happy to update the shipped Crumbs to 2.x, we just need to make sure that the few previous settings are migrated successfuly.

alexp999’s picture

After a long time digging in to why the crumbs never appeared on a lot of our product category pages, I found this.

Upgraded to Crumbs 7.x-2.x and it is all fixed and working well with CK 7.x-2.x!

bojanz’s picture

Status: Active » Needs work

We now have a PR up for this update: https://github.com/commerceguys/commerce_kickstart/pull/114

However, mglaman reports issues with blog categories and product breadcrumbs, so it needs more investigation.

donquixote’s picture

Just for the record: Crumbs 7.x-3.x is in progress!
https://github.com/donquixote/drupal-crumbs/issues/2

I can't really give a date. It regularly gets interrupted by client work. And it will need a testing and evaluation phase.

It all depends how much you consider 7.x-2.x an improvement to 7.x-1.x, to justify it being used in kickstart now.

mglaman’s picture

Category: Task » Feature request
rszrama’s picture

Status: Needs work » Closed (outdated)

Commerce Kickstart 2.x is in minimal maintenance mode. Closing out all outdated tickets now to maintain focus on Commerce Kickstart 3.x.