In drupal 7 this is not an issue due to the $http_response_code being sent out via drupal_alter in drupal_goto. Drupal 6's drupal_goto doesn't have this option and from my experiments with php header functions, it is impossible to get the $http_response_code from the header.

So I'm proposing that we pass http_response_code along. So this

module_invoke_all('exit', $url);

would become this

module_invoke_all('exit', $url, $http_response_code);
CommentFileSizeAuthor
#3 drupal-1003838-D6.patch813 bytesmikeytown2
#1 drupal-1003838.patch813 bytesmikeytown2

Comments

mikeytown2’s picture

Status: Active » Needs review
StatusFileSize
new813 bytes

Status: Needs review » Needs work

The last submitted patch, drupal-1003838.patch, failed testing.

mikeytown2’s picture

Status: Needs work » Needs review
StatusFileSize
new813 bytes

d6 patch

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.