Go to admin/menu/item/add -- then add a new menu item with a query in its path; something like:

chatbox?autolaunch=1

After submitting and clicking on the new link, it expands to the following, which gives a 404 error.

chatbox%3Fautolaunch%3D1

This is with 4.7 menu.module v 1.70, with clean urls. I thought it had something to do with http://drupal.org/node/53857 but that doesn't seem to be the case.

Comments

Uwe Hermann’s picture

Uwe Hermann commented

Confirmed in HEAD.

seanr’s picture

seanr
he/him
commented
Priority: Normal » Critical

It's passing the menu link through check_plain() which fubars the links. That also breaks #, so I can't use any named anchors. I'm bumping this up to critical since it makes it impossible to link to certain pages.

chx’s picture

chx commented
Priority: Critical » Normal
Status: Active » Closed (won't fix)

The path this menu item links to. This can be an internal Drupal path such as node/add or an external URL such as http://drupal.org. Enter to link to the front page.

chx’s picture

chx commented

If you want quote string or fragment in a menu item, make it an external URL.

seanr’s picture

seanr
he/him
commented

This was already said in the chatroom, but noting it here for the sake of posterity:

The above is a kludgy workaround that is not ideal in many situations. We should still be looking into a better long-term fix for this.

NaX’s picture

NaX commented

look here for possible menu system patch http://drupal.org/node/90570