Closed (fixed)
Project:
Menu Position
Version:
7.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Feb 2012 at 13:12 UTC
Updated:
25 Sep 2015 at 07:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mp165 commentedthanks @longwave - exactly what i needed!
Comment #2
ressa@longwave, great work! I am using Menu Position with Menu Breadcrumb, and the breadcrumb wouldn't show up, but the patch made it appear. Without the title, though, even though I have 'Append page title to breadcrumb' selected in Menu Breadcrumb settings. But I have a breadcrumb and that's what counts.
Comment #3
Wolfgang Reszel commentedThanks, this patch helps with Menu Breadcrumb. As I add the page title through the theme (Zentropy) it works perfect.
Comment #4
henrijs.seso commentedPatch works, but Menu Breadcrumb cannot add page title. Can something be done about it in Menu Breadcrumb module? I also tried to edit module weights, but that does not help.
Comment #5
erik seifert commentedWork also for panel paths like 'node/%node/article'
Comment #6
Rob_Feature commentedYup, works for me with menu breadcrumbs as well. Marking "reviewed and tested" since it seems like we've got a few folks reporting success.
Comment #7
rattusrattus commentedPatch updated to use paths relative to the module directory. Great work on this one, just what I needed.
Comment #8
ludo.rChanging modules weight has no effect because of this :
Perhaps menu_position should give a chance to other modules to alter the breadcrumb (like adding the current page title).
This could be done inside this function, by adding a drupal_alter() before setting active trail :
However, should menu_position prevail over all other breadcrumb related modules?
Using menu_position makes all other modules useless.
Comment #9
harrrrrrr commentedThe patch in #7 works fine for me. (I have no problem with the title, it's added to the bc trail)
Comment #10
ckrinaPatch in #7 works for me using Panels and solves my problem in this context.
Comment #11
rv0 commentedPatch in #7 works for me
Upping priority as the patch is 5 months old.
Comment #12
fabianx commented+1 for RTBC (using this in production)
Please lets get #7 in.
#8: An alter hook is a nice idea.
Overriding other modules is no problem, because you can select _when_ to apply menu position rules. Want another module? Just don't use mpr on this path.
Comment #13
vlad.dancer+1 for #7. Work for me too
Comment #14
Sborsody commentedThe patch in #7 solved an issue of the breadcrumb not appearing in a panel. Yay!
Comment #15
dabblela commentedPatch in #7 had whitespace issues for me. Re-rolled.
Comment #16
dabblela commentedSorry, path issues in that one.
Comment #17
heorhi lazarevich commentedPatch in #7 works for me using Adaptive Theme with options "Append the page title to the breadcrumb trail" enabled. Without this patch only page title was showing in the breadcrumbs without menu trail.
Comment #18
troybthompson commented#16 solved the problem for me.
Comment #19
peterlolty commentedComment #20
alan d. commentedYep, another rtbtc by me :)
Mine broke when using special menu items which was setting the trail.
Normally, I would not consider this a critical priority issue as there are not any wsod (white screens of death) or data loss. But this is really major as it is not actually working without it!
Comment #21
troybthompson commentedSolved my problem as well.
Comment #22
jhedstromAnother RTBC here. Fixes the conflict with items nested under a special menu item.
Comment #23
kbentham commentedRTBC here as well. This fixed my breadcrumb issues.
Comment #24
bryanhirsch commentedRTBC, resolves my issues.
Comment #25
agogo commented#16 fixed my problem with the breadcrumb as well. Someone please commit this.
Comment #26
peterlolty commentedRTBC, resolves my issues. thanks.
Comment #27
akalata commentedRTBC bump :)
Comment #28
schwarli4783 commented#16 solved the problem for me.
Comment #29
jenlampton#16 works for me to, +1 on RTBC
Comment #30
Rob_Feature commentedUm. "Me Too". This has been an issue forever...Would LOVE to see this committed!!!!
Comment #31
alexanansi commentedNot using special menu items, but whatever what was causing the problem was fixed by this patch
Comment #32
BarisW commentedRTBC-ed 3 years ago. Patch fixes the issue for a lot of people. Can this please be committed?
I'm happy to become co-maintainer if time is the issue here.
Comment #33
vood002 commentedThank you! #16 solved my problems
Comment #34
peterlolty commentedThis is already set to critical, so hope the maintainer look at here soon.
ps. RTBC is for #16
Comment #36
BarisW commentedCommitted and pushed to 7.x-1.x-dev. Thanks all!