The attached patch will improve Active Trail handling in Custom Breadcrumbs.

CommentFileSizeAuthor
custom-breadcrumbs.patch3.89 KBmarrch_caat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MGN’s picture

Is anyone interested in testing this?

smartango’s picture

I have not tested this patch, I solved the problem of menu active trail copying code from menutrails module:
I added from menu_get_item to menu_set_item:

function custom_breadcrumbs_set_breadcrumb($breadcrumb, $objs = array()) {
  if ($breadcrumb && !custom_breadcrumbs_exclude_path()) {
    $locations = array();
    $trail = _custom_breadcrumbs_get_breadcrumb($breadcrumb, $objs, $locations);
    if (variable_get('custom_breadcrumbs_force_active_trail', FALSE)) {
      menu_set_active_trail($locations);
      $item = menu_get_item();
      $a = array_pop($locations);
      $item['href'] = $a['href'];
      menu_set_item(NULL,$item);
    }
    drupal_set_breadcrumb($trail);
    // Optionally save the unique breadcrumb id of the last set breadcrumb.
    custom_breadcrumbs_unique_breadcrumb_id($breadcrumb->breadcrumb_type, $breadcrumb->bid);
  }
}

this work for me, but I suspect that your patch is more efficient.

lamp5’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)