I'm attaching a simple patch to resolve this issue.

CommentFileSizeAuthor
#4 custom_breadcrumbs-taxonomy_page_breadcrumbs-2229391-4.patch823 bytesOleksiy
Taxonomy_term_breadcrumbs_not_set.patch621 bytesarosboro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ron_s’s picture

ron_s CreditAttribution: ron_s commented

Yes, yes, and yes! Tried every possible option for taxonomy term pages, and could not get the breadcrumb to work correctly.

Bumping this up... need to get this added, assuming there are no other conflicts.

ron_s’s picture

ron_s CreditAttribution: ron_s commented

Well, this patch did help fix the noted issue, but there still seems to be other problems with the taxonomy submodule. I had a number of scenarios where certain parts of the taxonomy breadcrumb would not show up when expected.

While I really like the Custom Breadcrumbs module a lot, for the taxonomy functionality I'm going to use Path Breadcrumbs. I also think PB handles the taxonomy options in a more intuitive manner. As well I'm going to continue to monitor this discussion: https://www.drupal.org/node/1823080 :-)

jive01’s picture

jive01 CreditAttribution: jive01 commented

I attempted to apply this patch but the lines of code were already applied... (so this must have been added already)

Custom Breadcrumb paths simply does not show up on taxonomy term pages still however.

Oleksiy’s picture

Oleksiy
Primary language Ukrainian
Location Lutsk 🇺🇦
CreditAttribution: Oleksiy at Skilld commented
Priority: Normal » Major
FileSize
custom_breadcrumbs-taxonomy_page_breadcrumbs-2229391-4.patch823 bytes

Patch above allows to set breadcrumbs based on Taxonomy Term. Have updated it to allow set breadcrumbs based also on Taxonomy Vocabulary. Note that "Use the taxonomy hierarchy to set the breadcrumb trail for nodes and taxonomy pages" option need to be disabled to make possible use these types of custom breadcrumbs.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Yep, that works

adam-delaney’s picture

adam-delaney CreditAttribution: adam-delaney commented

This works for me and the patch makes sense.

chrlvclaudiu’s picture

chrlvclaudiu CreditAttribution: chrlvclaudiu as a volunteer commented

This #5 patch works for me, too.

Mzambrana’s picture

Mzambrana CreditAttribution: Mzambrana commented

This #5 patch works for me, too.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented

Really works good

  • RenatoG committed b7d2d78 on 7.x-2.x authored by Oleksiy 
    Issue #2229391 by arosboro, Oleksiy, ron_s, jive01, andypost, adam-...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Reviewed & tested by the community » Fixed

Committed to the dev branch. Thank you for contribution.

Status: Fixed » Closed (fixed)

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