With 7.x-2.x i get following problems:

Undefined index: menu_item in custom_breadcrumbs_theme_registry_alter() (line 841 ...sites/all/modules/custom_breadcrumbs/custom_breadcrumbs.module).`

Undefined index: menu_item in custom_breadcrumbs_theme_registry_alter() (line 842 ...sites/all/modules/custom_breadcrumbs/custom_breadcrumbs.module).

CommentFileSizeAuthor
#8 custom_breadcrumbs-fixed_undefined_index_bug-1366394-8-do-not-test.patch950 bytessourabh.singhal
#9 after.png46.51 KBlomasr
#9 patch_apply.png39.16 KBlomasr
#9 before.png47.1 KBlomasr

Comments

waltf’s picture

waltf commented
Issue summary: View changes

edit spelling

szt’s picture

szt commented
Status: Active » Needs review

The possible reason if the experimental "Force the active trail" setting is enabled at admin/config/user-interface/custom-breadcrumbs.

Marko B’s picture

Marko B commented
Status: Needs review » Fixed

I had same problem, and yes this "force active trail" made a mess. It also made my language switcher stop working and showing.

Status: Fixed » Closed (fixed)

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

gaele’s picture

gaele commented
Priority: Critical » Normal
Status: Closed (fixed) » Active

@szt and deepM:

The bug still exists and there's no patch or solution, only a work-around (disable the "Force the active trail" setting), so "active" would be the right status for this issue. This way the module maintainers may notice and can look into it themselves.

I ran into this myself and need this functionality.

gaele’s picture

gaele commented
Issue summary: View changes

edit

jphelan’s picture

jphelan commented
Priority: Normal » Major
Issue summary: View changes

I'm having this issue on both 7.x-2.x-dev and 7.x-2.0-alpha3. It breaks all the menus.

szt’s picture

szt commented

Same issue on 7.x-2.0-beta1.

sourabh.singhal’s picture

sourabh.singhal commented
Assigned: Unassigned » sourabh.singhal
sourabh.singhal’s picture

sourabh.singhal commented
Status: Active » Needs review
StatusFileSize
new custom_breadcrumbs-fixed_undefined_index_bug-1366394-8-do-not-test.patch950 bytes

Hi waltf,

I have fixed this issue and prepared a patch for the same. Please review and let me know if any thing else required.

Thanks
Sourdrup

lomasr’s picture

lomasr commented
StatusFileSize
new before.png47.1 KB
new patch_apply.png39.16 KB
new after.png46.51 KB

Patch removed the error (Enabled the "Force the active trail" setting) Please see screens.

sourabh.singhal’s picture

sourabh.singhal commented

Hi lomasr,

Thanks for reviewing the patch, it seems the bug is fixed.

@waltf, can you please let me know, in case if you need any thing else from my side.
Do let me know when you will be able to close this ticket.

Thanks
Sourdrup

nikhilsukul’s picture

nikhilsukul commented
Assigned: sourabh.singhal » Unassigned
Status: Needs review » Reviewed & tested by the community

Thanks @sourdrup,

It is working fine for me as well

c_archer’s picture

c_archer commented

Hi,

Thanks for the patch this worked as expected for me.

Cheers,
Chris

renatog’s picture

renatog commented
Assigned: Unassigned » renatog
Issue tags: +ciandt-contrib

Hi people.

Make sense. Thank you very much for contribution.

Good Work.

  • RenatoG committed edec5ac on 7.x-2.x authored by sourdrup 
    Issue #1366394 by sourdrup, lomasr, waltf, szt, gaele, Marko B, jphelan...
renatog’s picture

renatog commented
Status: Reviewed & tested by the community » Fixed

Fixed.

Commited in dev branch.

Thank you very much people.

Good Work.

Regards.

Status: Fixed » Closed (fixed)

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