Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
add ability to have several breadcrumbs on the page.
e.g. One breadcrumb is set using Drupal: Home -> book-parent -> book-child
And the second is set using taxonomy_menu module: Home -> book-page-category -> book-page-subcategory.
Admins can allow/disallow several breadcrumbs, and the last decision does to template's theme_breadcrumb().
Comment | File | Size | Author |
---|---|---|---|
#5 | bc-weight-73834-5.patch | 1.5 KB | pwolanin |
Comments
Comment #1
LAsan CreditAttribution: LAsan commentedFeature request, moving to cvs.
Comment #2
beginner CreditAttribution: beginner commentedI just posted about this on the dev list.
Comment #3
beginner CreditAttribution: beginner commentedsee :
http://lists.drupal.org/pipermail/development/2008-June/030337.html
and
#150854: fix breadcrumbs arbitration bug
Comment #4
pwolanin CreditAttribution: pwolanin commentedWell, the timing problem certainly is a problematic bug - seems like a simple solution might be (even for D6) to take several alternate BC arrays with each one getting a weight.
something like this:
Comment #5
pwolanin CreditAttribution: pwolanin commentedhere's a patch that might also be suitable for 6.x as a bug fix.
This is a minimal change that just allows a developer to be sure that their BC will have priority over one being set (for example) by a core module.
Comment #6
beginner CreditAttribution: beginner commentedI agree it's a bug. See my follow up on the other issue: #150854: fix breadcrumbs arbitration bug
I prefer the patch there, because it paves the way to a much for powerful hook_breadcrumbs() with $op, $path, $args, etc.
With that new hook_breadcrumbs(), themes would be able to get an array of breadcrumbs and display more than one set of breadcrumbs, which is the current feature request.
Comment #7
pwolanin CreditAttribution: pwolanin commentedComment #8
moshe weitzman CreditAttribution: moshe weitzman commentedThis can't be a bug IMO. The system is working as designed. Adding a second breadcrumb is a new feature. Seems clear cut to me.
Comment #9
pwolanin CreditAttribution: pwolanin commented@moshe - changing the title to reflect what the patch does.
Basically - if the breadcrumb is set in the page callback, I think there is pretty much no way to override it unless you hack the theme layer. That's what I'm identifying as the bug. The change proposed should be a backward-compatible and opens the window by adding the weight parameter (we could call it $priority or something else if you like).
Comment #10
beginner CreditAttribution: beginner commentedpwolanin: you are hijacking this issue. See the other issue for the bug you are trying to fix.
I mentioned your patch in the other issue.
Having several breadcrumbs is a bona fide feature request.
Comment #11
pwolanin CreditAttribution: pwolanin commented@beginner - sorry - I missed that link - didn't mean to hijack.
Comment #12
mdupontComment #13
kika CreditAttribution: kika commentedWas proposing it back at 2002. +1
Comment #14
sunBetter title.
FWIW, I once started the http://drupal.org/project/breadcrumb module with a seemingly identical goal; multiple breadcrumbs, and breadcrumbs as blocks. Given D8, I think this ties deeply into Blocks and Layouts.
Comment #15
Crell CreditAttribution: Crell commentedBreadcrumbs are now blocks. This is done. :-)