API page: http://api.drupal.org/api/drupal/includes%21common.inc/function/drupal_g...

This is the description of the 'query' paramter:
$query: A query string component, if any.

Due to the fact that the value is simply handed over to the url() function, the description should be the same as on page http://api.drupal.org/api/drupal/includes!common.inc/function/url/6

'query': A URL-encoded query string to append to the link, or an array of query key/value-pairs without any URL-encoding

CommentFileSizeAuthor
#2 drupal_goto-query-documentation-1432708-2.patch939 bytesscorchio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Issue tags: +Novice

Probably it would be even better if it just referred to url() for details. Sounds like a good project for a novice patch contributor. Note that this is 6.x only (7.x/8.x have different arguments and already refer to url() for details).

scorchio’s picture

scorchio CreditAttribution: scorchio commented
Status: Active » Needs review
FileSize
drupal_goto-query-documentation-1432708-2.patch939 bytes

Here's a patch for this one.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

Looks OK to me. Thanks!

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed, pushed.

Automatically closed -- issue fixed for 2 weeks with no activity.