Skip retrieval of menu path on front page [#492572]

I have a multilingual site, which has French as the default language. After installing this module, I noticed that my primary links were no longer visible on the English home page. The attached patch fixes the problem for me, it ensures that menu functions don't get called in menu_breadcrumb_init() if the current page is the front page. This patch has the added benefit of ensuring that the Home link on the front page always points to the front page, which would fix issues like this #278148: menu breadcrumb leads to wrong "home" breacrumb on startpage.

Comment	File	Size	Author
#6	menu_breadcrumb-HEAD_front_page2.patch	1.41 KB	dboulet
#2	menu_breadcrumb-HEAD_front_page.patch	1.37 KB	dboulet
	menu_breadcrumb_HEAD_front_page.patch	1.34 KB	dboulet

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

niff CreditAttribution: niff commented 23 June 2009 at 06:21

I had a similar problem with the primary menu, but on a Swedish site. Clicking any home link (link or logo) with the Swedish language active made the primary menu change language to English, but with the rest of the site in Swedish. This patch fixed that problem.

Comment #2

dboulet CreditAttribution: dboulet commented 8 October 2009 at 15:31

Title:

Using menu functions in hook_init() confuses menu system

» Skip retrieval of menu path on front page

File	Size
menu_breadcrumb-HEAD_front_page.patch	1.37 KB

Improved patch, rolled against latest HEAD.

Comment #3

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 9 October 2009 at 02:28

Looks like this patch will need re-rolling against #595282: Menu weights (unless that fix fixes this one).

Comment #4

dboulet CreditAttribution: dboulet commented 9 October 2009 at 03:15

If #595282: Menu weights gets committed, I can re-roll the last patch, but until then it still applies to the latest development snapshot, and is still valid. Since that issue is still under heavy development, I'll assume that there is still a possibility that this patch gets committed first.

Comment #5

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 9 October 2009 at 03:59

You're right. I've been cleaning up a bunch of older issues today, and didn't spot that your patch was submitted quite recently. Sorry for missing that.