Closed (fixed)
Project:
Custom Breadcrumbs
Version:
6.x-1.3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jun 2008 at 16:18 UTC
Updated:
21 May 2009 at 02:10 UTC
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.
| Comment | File | Size | Author |
|---|---|---|---|
| multiple_breadcrumbs_for_node_type.patch | 564 bytes | kittipat |
Comments
Comment #1
kittipat commentedComment #2
MGN commentedThis was fixed in #296166: Only first row checked and committed to 6.x.1.x-dev.