docs for drupal_goto() don't say what happens if $path is empty

It looks like we should probably say on $path ", which will be passed to url() to compute the redirect for the URL.", and we should add something to url() that says that '' is equivalent to '<front>' (which you can tell from reading the code, but it should be in the documentation).

The reason I suggest this is that is that there is a lot of documentation already on url() that I don't think we want to duplicate. Better just to refer to reader there.

• Implemented the suggestion from #2.
• Additionally, I've added "(optional)" to all optional parameters in drupal_goto() and url(), and an extra sentence and default for the $http_response_code in url().

That looks really good! Tow small things I think should be fixed:

a)

  *   @link http://tools.ietf.org/html/draft-reschke-http-status-308-07 draft for the new HTTP status codes: @endlink
+ *   Defaults to 302.
(list comes next)

This will end up reading as:

(link)...status codes: (endlink) Defaults to 302.
- list item

That isn't good. The : needs to be at the end of the line before the list, so the "Defaults to 302." should go somewhere else.

b)

+ *   "node/34" or "http://example.com/foo". The default value is equivalent to
+ *   passing '<front>'. A few notes:

passing -> passing in

Can't believe I missed the ':'+list. Would this work:

The HTTP status code to use for the redirection, defaults to 302. The valid values for 3xx redirection status codes are defined in...

status change

That looks perfect, thanks! I'll get it committed shortly.

Committed to 8.x. The patch also applied to 7.x with a small amount of fuzz, so I committed it there too. Thanks!

I think we should probably backport it to 6.x as well, since I think it is all correct there (might be a good idea to verify this).

Back-ported to D6, verified that the logic is the same (i.e. passing NULL or '' to url is equivalent to pointing to the home page).

That looks good for 6.x -- thanks!

I've been given permission to commit docs patches for Drupal 6, so this is in! Thanks again!