Menu system regression: Local tasks no longer passing through theme('menu_item_link')

merlinofchaos - October 3, 2007 - 19:08
Project:Drupal
Version:6.5
Component:menu system
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

Previously, local tasks (Tabs) passed through theme('menu_item_link'). Currently they go directly through l(). It is necessary for themes to be able to theme the link in order to do pretty tabs, as it often requires putting markup inside the <a> tag in order to achieve the proper look. This worked in Drupal 5.

This patch restores this functionality.

AttachmentSize
local-task-item-theme.patch2.22 KB

#1

merlinofchaos - October 3, 2007 - 19:09
Status:needs work» needs review

#2

chx - October 8, 2007 - 17:05
Status:needs review» reviewed & tested by the community

It's okay.

#3

Gábor Hojtsy - October 8, 2007 - 18:49
Status:reviewed & tested by the community» fixed

Thanks, committed.

#4

Anonymous - October 22, 2007 - 18:51
Status:fixed» closed

#5

adpo - October 12, 2008 - 08:06
Version:6.x-dev» 6.5

It is not working for me.
I received a message:

patching file includes/menu.inc
Hunk #1 FAILED at 908
Hunk #2 FAILED at 1121
Hunk #3 FAILED ar 1153

 
 

Drupal is a registered trademark of Dries Buytaert.