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

[gordon]

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:   gordon
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/civicrm_menu.patch.txt (9.14 KB)

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.




gordon



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()




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

Tue, 20 Jun 2006 16:36:59 +0000 : Morbus Iff

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!