When creating multiple custom breadcrumbs for a node type, only the first custom breadcrumb is tested. Says, you have two custom breadcrumbs for odd pages and even pages, only the one defined first will be show up correctly but the other won't.

After looking through the code, I found that this is caused by the wrong use of a variable in _custom_breadcrumbs_load_for_type(). The variable '$result' is already used to hold breadcrumbs for the type, but it is used again to hold the result of visibility evaluation. This works fine if the evaluation return TRUE. However, when FALSE is returned, there would be nothing to fetch, and the function returns nothing.

CommentFileSizeAuthor
multiple_breadcrumbs_for_node_type.patch564 byteskittipat

Comments

kittipat’s picture

Assigned: kittipat » Unassigned
MGN’s picture

Status: Needs work » Fixed

This was fixed in #296166: Only first row checked and committed to 6.x.1.x-dev.

Status: Fixed » Closed (fixed)

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