Add query to the menu items.

Posted by gordon on June 16, 2006 at 1:12am

Issue Summary

Here is a small patch that allows modules to add to the querystring of an item.

Case: I have used this to allow CiviCRM use standard menu entries for it's functions instead of the custom block that is currently being used. I am going to submit a patch for the next version of CiviCRM which will remove the current blocks and use the Drupal menu structure which is more powerful.

The attached patch was created on 4.7.2 but should apply to CVS.

Attachment	Size
menu_query.patch.txt	770 bytes

Comments

#1

Posted by Dries on June 17, 2006 at 3:22pm

Status:

needs review

» fixed

Committed to CVS HEAD.

(Looks like a stupid function to me.)

#2

Posted by gordon on June 20, 2006 at 3:41am

Just out of interest, what is the stupid function. adding the query parameter, or the theme_menu_item() function.

#3

Posted by gordon on June 20, 2006 at 3:52am

Project:	Drupal core	» Documentation
Version:	x.y.z	» <none>
Component:	menu system	» Documentation in CVS
Status:	fixed	» needs review

Here is the changes to the api for hook_menu()

Attachment	Size
menu_doc.patch.txt	814 bytes

#4

Posted by Morbus Iff on June 20, 2006 at 4:36pm

Gordon - thanks for this! I was just remarking to myself how annoying it was that I couldn't use query strings (for, yes, the reset=1 of a CiviCRM menu item that I was manually making into my own Drupal block). Wunderbar!

#5

Posted by gordon on June 21, 2006 at 1:55am

NP, my client also found this annoying. I know that this is not really related, but for interest here is that patch to CiviCRM 14. which puts all the menus in the menu.

It does remove from the blocks, but they can be another change.

Attachment	Size
civicrm_menu.patch.txt	9.14 KB

#6

Posted by Amazon on June 28, 2007 at 6:56pm

I have been testing out query strings with CiviCRM in Drupal 5. I am unable to use query strings with menu items. Could someone shed some history on what was committed and if there was another patch that removed this ability.