Wrong breadcrumb with view [#225385]

If there is a view displayed in the content area of a node, the taxonomy breadcrumb module does not show the breadcrumb of the node which is displayed on the page but of the last node, which is in the view. It seems that the taxonomy_breadcrumb_nodeapi is executed not only for the node which is to show on the page but also for every node which is in the view.

To get this fixed I enhanced the first line of the taxonomy_breadcrumb_nodeapi to check, if the node being processed is the one in the url. So I changed

if ($op == 'view' && $a4 && !drupal_is_front_page()) {

if ($op == 'view' && $a4 && !drupal_is_front_page() && arg(1) == $node->nid) {

For me this works fine. But I would like to ask if this is the right place to fix this or if it wouldn't be better to do a change so that the taxonomy_breadcrumb_nodeapi is only called for the page but not for the nodes of the view and how this could be done.

Comments

Comment #1

_craig CreditAttribution: _craig commented 23 February 2008 at 02:46

Interesting solution. I like it.

Comment #2

_craig CreditAttribution: _craig commented 15 March 2008 at 18:59

Status:

Active

» Closed (fixed)

Comment #3

smitty CreditAttribution: smitty commented 11 September 2008 at 12:15

Status:

Closed (fixed)

» Postponed (maintainer needs more info)

Did you commit this to a new version (I could't find any)? Or ist there another solution to this problem?

Comment #4

Xano

he/they

English

Southampton

CreditAttribution: Xano commented 29 September 2008 at 07:50

Assigned:

Unassigned

» Xano

What about this?

if (arg(0) == 'node') {
// Do the move!
}

//edit: @ Smitty: This fix hasn't been committed yet to the latest dev version.

Comment #5

smitty CreditAttribution: smitty commented 9 December 2008 at 09:23

That's not working because in the view you display many nodes (teasers) on the page of the node, the breadcrumb is for. So for setting the breadcrumb you have to make sure, that you process the node of the page (and not the teasers of the view) ant that works only if the nid of the processed node is identical with arg(1).