Mega menu strips query string from menus link [#1234516]

If you add a link to your mega menu with a path like candidates?gendre=M, Marinelli's Mega menu will only display candidates making your link inaccurate.

After a bit of research, I found that this issue already existed in Megamenu modules, and a patch has been released:
http://drupal.org/node/1167776

I do not know how to write patches, so the lines missing in theme.inc are:

$options = array();
/* missing */ options += $value['link']['localized_options'];
...
$options = array('class' => array('menu-section-link'));
/* missing */ options += $value2['link']['localized_options'];
...
$options = array('class' => array('menu-leaf-link'));
/* missing */ options += $value3['link']['localized_options'];

Comment	File	Size	Author
#1	1234516.patch	2.12 KB	kadimi
#2	1234516.patch	1.41 KB	kadimi

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

kadimi CreditAttribution: kadimi commented 13 January 2012 at 23:28

Status:

Reviewed & tested by the community

» Active

File	Size
1234516.patch	2.12 KB

~~Here is the patch~~
Corrected on the next post

File	Size
1234516.patch	1.41 KB

Comment #3

mavimo CreditAttribution: mavimo commented 26 August 2011 at 13:15

Status:

Active

» Reviewed & tested by the community

Comment #4

salapataras CreditAttribution: salapataras commented 3 November 2011 at 08:32

Thera are one more problem that the mega menu take all link without check the language and dispaly all the link regardless there language. Can anyone help my please???

Comment #5

kadimi CreditAttribution: kadimi commented 6 November 2011 at 08:53

I suggest you start a separate issue, it will probably get more attention.

Thanks.

Comment #6

salapataras CreditAttribution: salapataras commented 6 November 2011 at 09:42

I have : http://drupal.org/node/1329180

Comment #7

Akshita CreditAttribution: Akshita commented 23 February 2012 at 17:54

Project:	Marinelli	» Drupal core
Version:	7.x-3.0-beta11	» 7.0
Component:	Code	» menu system
Priority:	Major	» Critical

Hi Kadimi

I have an issue with Mega menu. I have a redirect script (customized) which works fine on the right nav elements of any node.

But from the mega menu the query string disappears.

For example : Mega Menu (Main menu from the home page)

Divisions(Main Tab) and on click I have drop down

Divisions->D1 -D2 (If I use redirect script on d2 like (redirect?path=abcd) then when I try to access the link D2 then URL shows only http://mydomain.com/redirect)

Here is my code:

/* Add a system path that we can use to redirect content */

function my_custom_menu() {
$items['redirect'] = array(
'title' => 'Perform Redirect',
'description' => '',
'page callback' => 'my_custom_redirect',
'page arguments' => array(''),
'access arguments' => array('access content')
);
return $items;
}

function my_custom_redirect() {
$path = $_GET['path'];
header("Location: $path");
}

Any Help is appreciated.

Thanks in advance

Comment #8

marcingy CreditAttribution: marcingy commented 23 February 2012 at 23:00

Priority:

Critical

» Normal

This is not critical

Comment #9

marcingy CreditAttribution: marcingy commented 23 February 2012 at 23:01

Project:	Drupal core	» Marinelli
Version:	7.0	» 7.x-3.0-beta11
Component:	menu system	» Code
Priority:	Normal	» Major
Status:	Active	» Reviewed & tested by the community