Theme not applicable on all list items

tklawsuc - April 5, 2007 - 16:14
Project:Lucid menu
Version:5.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Not sure if this was intentional but in the lucid_menu_tree function if the item has children, the item is built using:

while items without children are built using the theming option:

<?php
$output
.= theme('lucid_menu_item', $pid);
?>

So if you try to override theme_lucid_menu_item the changes will only be applied to items that have no children. I made the change (to use theme_lucid_menu_item($pid); for parent items) in the module for my work (no sure how to create a patch...still new to drupal). So if this is intentional then ignore this bug...otherwise spare future users hours of frustration trying to override the item function and change accordingly.

#1

tklawsuc - April 5, 2007 - 16:16

Sorry not sure what happened but the original post should say...the item with children is built using:

<?php
 
if (!$skip_parent)
         
$output .= theme_lucid_menu_item($pid);
?>

 
 

Drupal is a registered trademark of Dries Buytaert.