I have a website with a single quote in its breadcrumb title:
Something's fishy with China algae plant

This was being displayed as:
Something's fishy with China algae plant

In custom_breadcrumb.module's custom_breadcrumb_nodeapi(), I added the html decode in the following line:
$trail[] = l(html_entity_decode($title, ENT_QUOTES, 'UTF-8'), trim($paths[$i]));

This fixed the problem nicely.

CommentFileSizeAuthor
#4 custom-breadcrumbs.patch693 bytesherve

Comments

MGN’s picture

MGN commented

Thanks for the idea. An alternative solution would be to use decode_entities($title) (see http://api.drupal.org/api/function/decode_entities ). Older sites still running php4 will have problems with html_entity_decode producing a warning something like "cannot yet handle MBCS in html_entity_decode" but decode_entities from the drupal 6 api seems to work fine.

MGN’s picture

MGN commented
Version: 6.x-1.3 » 6.x-2.x-dev
Status: Active » Fixed

This has been implemented in custom breadcrumbs 6.x-2.x-dev.

Status: Fixed » Closed (fixed)

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

herve’s picture

herve commented
Version: 6.x-2.x-dev » 7.x-1.0-alpha1
StatusFileSize
new custom-breadcrumbs.patch693 bytes

A patch for the same bug in D7 version of the custom breadcrumbs module.

herve’s picture

herve commented
Status: Closed (fixed) » Patch (to be ported)

Update the status to patch the D7 version.

jvandooren’s picture

jvandooren commented

Patch works great for D7.

dkre’s picture

dkre commented

Thanks for the patch Herve.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Version: 7.x-1.0-alpha1 » 7.x-2.x-dev
Status: Patch (to be ported) » Postponed (maintainer needs more info)

Is this still needed in the 7.x-2.x branch?

aangel’s picture

aangel commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (outdated)