Integration with Content Menu [#1581840]

Patch included that add support for menu_firstchild

Comment	File	Size	Author
#1	content_menu-1581840-1-support_menu_first_child.patch	1.33 KB	Simon Georges
	content_menu_menu_firstchild.patch	1.27 KB	attiks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Simon Georges CreditAttribution: Simon Georges commented 12 February 2013 at 20:58

File	Size
content_menu-1581840-1-support_menu_first_child.patch	1.33 KB

Hi,
Your patch wouldn't apply on current -dev version. And I didn't manage to make it work after applying it manually.
I ended using the following one (with the addition of $item['module'] = 'menu';). Could you confirm it works?

This patch is part of the #1day1patch initiative.

Comment #2

attiks CreditAttribution: attiks commented 12 February 2013 at 21:46

Status:

Needs review

» Reviewed & tested by the community

Looks good, thanks for the follow up

Comment #3

Simon Georges CreditAttribution: Simon Georges commented 13 February 2013 at 11:35

Title:	Support for menu_firstchild	» Integration with Content Menu
Project:	Content Menu	» Menu Firstchild
Status:	Reviewed & tested by the community	» Needs work

Actually, it should be the other way around: Menu First Child could insert itself into Content Menu using the hook. It's totally possible, and would prevent Content Menu from being required to follow Menu Firstchild eventual implementation change... What do you think?