Menu breadcrumb can create duplicate ID attribute

Reg - April 10, 2009 - 09:04
Project:Menu Breadcrumb
Version:6.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

This is strictly an interaction with another module in that the other module allows you to add ID's and other attributes to menu items. When breadcrumb picks up a menu item it displays it with all it's HTML attributes. As a quick fix I created the following patch.

AttachmentSize
patch_menu_breadcrumb_1.txt665 bytes

#1

Reg - April 10, 2009 - 09:36

Oops, small error in the patch, here's an update.

AttachmentSize
patch_menu_breadcrumb_2.txt 665 bytes

#2

xurizaemon - October 9, 2009 - 03:27
Status:active» won't fix

IMO this is not a bug in menu breadcrumb. If the other module (which you don't name?) adds an ID to a menu item, and the menu item appears more than once in the same page, then that is a bug in the other module.

The ID should be applied to the HTML element, not to the menu entry.

Sorry, I don't think we'll accept this patch.

 
 

Drupal is a registered trademark of Dries Buytaert.