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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

waltf’s picture

waltf CreditAttribution: waltf commented
Issue summary: View changes

edit spelling

szt’s picture

szt CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: gaele commented
Issue summary: View changes

edit

jphelan’s picture

jphelan CreditAttribution: 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 CreditAttribution: szt commented

Same issue on 7.x-2.0-beta1.

sourabh.singhal’s picture

sourabh.singhal CreditAttribution: sourabh.singhal as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commented
Assigned: Unassigned » sourabh.singhal
sourabh.singhal’s picture

sourabh.singhal CreditAttribution: sourabh.singhal as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commented
Status: Active » Needs review
FileSize
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 CreditAttribution: lomasr at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
FileSize
before.png47.1 KB
patch_apply.png39.16 KB
after.png46.51 KB

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

sourabh.singhal’s picture

sourabh.singhal CreditAttribution: sourabh.singhal as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd 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 CreditAttribution: nikhilsukul as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd 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 CreditAttribution: c_archer as a volunteer and at Upbeat Productions commented

Hi,

Thanks for the patch this worked as expected for me.

Cheers,
Chris

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T 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
Location Campinas
CreditAttribution: renatog at CI&T 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.