Views Pages Breadcrumb:news (below Environmental Sustainabilit)
Titles:
Environmental Sustainability
News
Path:
environmental-sustainability
<none>

Taxonomy Term Breadcrumb:News (below Environmental Sustainability)
Path: environmental-sustainability/news

Path Breadcrumb:
Titles:
Environmental Sustainability
Path:
environmental-sustainability

Breadcrumb Displayed at View path:
Facilities and Services > Environmental Sustainability > News
(^This view has no menu item set)
Breadcrumb Displayed at Panel path (Custom Breadcrumb Path):
Facilities and Services > Environmental Sustainability > News

The panel is displaying a block of the News View
I've just confirmed this block is causing the incorrect path on the Panel Custom Page, removing the news block from the panel causes the breadcrumb to output as expected:
Facilities and Services > Environmental Sustainability

Adding the block to the panel again causes the breadcrumb to display incorrectly again:
Facilities and Services > Environmental Sustainability > News

CommentFileSizeAuthor
#2 519608_custom_breadcrumbs_views.module.patch686 bytesrealityloop

Comments

realityloop’s picture

realityloop commented

This might help, I haven't figured out how to to get it to work with the Breadcrumb visibility field on the view custom breadcrumb though:

<?php 
if (panels_get_current_page_display()) {
  echo "this is a panel";
} else {
  echo "this isn't a panel";
}
?>

Could you have a look at including this behavior by default, or use it to create Custom Breadcrumbs for Panels please?

realityloop’s picture

realityloop commented
StatusFileSize
new 519608_custom_breadcrumbs_views.module.patch686 bytes

Attached patch fixes the issue here..

realityloop’s picture

realityloop commented
Status: Active » Patch (to be ported)

Updating status

edit: sorry if I chose the wrong one..

MGN’s picture

MGN commented
Status: Patch (to be ported) » Postponed

At this point, panels is not supported, so I am not really surprised to see problems on panels pages.

The problem with the approach used in the patch in #2 is that it will crash a site not using panels. You would have to first check to see if the function (or api) exists. Also, I think the same can be accomplished using:
Breadcrumb Visibility: return !panels_get_current_page_display();

One of the longstanding TODO items has been to get a custom breadcrumbs to work with panels. So this issue will be better addressed when that task gets underway. I would hesitate trying to patch the views code at this point, just because I am not sure how the cb_panels submodule is going to work and because I think the visibility approach should work well in the meantime. So I am going to mark this as postponed for now, so we can remember to consider the interaction between views and panels breadcrumbs when developing the cb_panels submodule.

BTW: The correct status would have been "needs review." Thanks for the effort!

MGN’s picture

MGN commented
Status: Postponed » Fixed

A fix, based on the patch in #2, has been committed http://drupal.org/cvs?commit=246144

Status: Fixed » Closed (fixed)

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