In Kickstart distribution of Drupal Commerce, the breadcrumbs is hidden by default on this pages. How display the breacrumbs on all account pages (user/*)?

I have try to set position for the breacrumbs block (disabled by default) into "breacrumbs" region, it's correctly displayed on all pages but it's displayed duplicate as you can see.

What's the good way for this?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dxx’s picture

Issue summary: View changes
FileSize
15.22 KB
dxx’s picture

Confirmed on a fresh install. Need more information about this issue?

mglaman’s picture

Component: Code » User interface
Category: Bug report » Feature request

Breadcrumbs are displaying across the site (demo or non-demo) without the block for content. Moving this to feature request to ensure breadcrumbs show up under user/* pages.

mglaman’s picture

Oddly enough, code is in commerce_kickstart_product_ui, it seems.

/**
 * Implements hook_page_alter().
 */
function commerce_kickstart_product_ui_page_alter(&$page) {
  // Disable breadcumb on user, checkout and cart pages.
  $current_path = current_path();
  $exclude_path = "user*\ncheckout/*\ncart";
  $match = drupal_match_path(drupal_strtolower($current_path), drupal_strtolower($exclude_path));
  if ($match) {
    drupal_set_breadcrumb(array());
  }
}
mglaman’s picture

Issue tags: +sprint
mglaman’s picture

Issue tags: +Novice
mglaman’s picture

Status: Active » Needs review
FileSize
2.54 KB

Patch which moves user crumb logic to commerce_kickstart_user, also adds variable for allowing breadcrumbs (set commerce_kickstart_user_breadcrumbs to TRUE).

Build: https://travis-ci.org/commerceguys/commerce_kickstart/builds/71790651

howdytom’s picture

#7 patch is working for me. Excellent!

mglaman’s picture

Wrote some tests for this. Waiting for build to pass: https://travis-ci.org/commerceguys/commerce_kickstart/builds/72015955

  • mglaman committed 962c15f on 7.x-2.x
    Issue #2393519: Show breadcrumbs on account/profile pages
    
  • mglaman committed d90e347 on 7.x-2.x
    Issue #2393519: Show breadcrumbs on account/profile pages
    
mglaman’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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