[documentation] [Documentation bug] Add query to the menu items.

[Morbus Iff]

Issue status update for 
http://drupal.org/node/69270
Post a follow up: 
http://drupal.org/project/comments/add/69270

 Project:      Documentation
 Version:      <none>
 Component:    Documentation in CVS
 Category:     bug reports
 Priority:     normal
 Assigned to:  gordon
 Reported by:  gordon
 Updated by:   Morbus Iff
 Status:       patch (code needs review)

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!




Morbus Iff



Previous comments:
------------------------------------------------------------------------

Fri, 16 Jun 2006 01:12:00 +0000 : gordon

Attachment: http://drupal.org/files/issues/menu_query.patch.txt (770 bytes)

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.




------------------------------------------------------------------------

Sat, 17 Jun 2006 15:22:35 +0000 : Dries

Committed to CVS HEAD.  


(Looks like a stupid function to me.)




------------------------------------------------------------------------

Tue, 20 Jun 2006 03:41:24 +0000 : gordon

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




------------------------------------------------------------------------

Tue, 20 Jun 2006 03:52:20 +0000 : gordon

Attachment: http://drupal.org/files/issues/menu_doc.patch.txt (814 bytes)

Here is the changes to the api for hook_menu()