Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Under the conditions below, the 'Site' portion of the breadcrumb is repeated.
- Enable apachesolr_search and facetapi
- Enable content type and post date facets
- Disable node and user searches and set apachesolr to the default
- Add the facet blocks so they are rendered on the search page
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | remove-current-page-breadcrumb-item-1693202-3.patch | 1008 bytes | klaasvw |
| Selection_047.png | 45.54 KB | jhedstrom |
Comments
Comment #1
jhedstromThe reason for this can be seen here.
due to the active class, they aren't equal so the
$last != $this_pageevaluates to TRUE.Comment #2
cpliakas commentedjhedstrom,
Thanks for the detailed explanation. This bug is confirmed.
Chris
Comment #3
klaasvw commentedHere's a patch that removes the
$this_pageitem entirely.I'm not sure why this was added in the first place. Isn't it so that by default drupal's breadcrumb never links to the current page you're viewing? That's exactly what
$this_pageis, a link to the current page. So to stay consistent with how breadcrumbs work in drupal it might be better to remove this.Comment #4
cpliakas commentedI have been looking at this for a while. I have no idea why the code is necessary, I just know that at one time it was. Removing it seems to fix the problem, so I am marking as RTBC. Let's see if people yell at me once this code is circulated.
Comment #5
cpliakas commentedCommitted to the 7.x versions of Facet API.
Congrats on becoming the 32nd code contributor to Facet API!
Comment #6
klaasvw commentedThanks for committing this!
Comment #7
cpliakas commentedNP. Look forward to your next patch!