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
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal_goto-query-documentation-1432708-2.patch. Unable to apply patch. See the log in the details link for more information. View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

jhodgdon’s picture

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

Status: Active » Needs review
FileSize
939 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal_goto-query-documentation-1432708-2.patch. Unable to apply patch. See the log in the details link for more information. View

Here's a patch for this one.

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Looks OK to me. Thanks!

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, committed, pushed.

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