diff options
author | Dries Buytaert <dries@buytaert.net> | 2009-12-06 23:56:47 +0000 |
---|---|---|
committer | Dries Buytaert <dries@buytaert.net> | 2009-12-06 23:56:47 +0000 |
commit | 864408cefa3cecf2b4b3a7ef951a5612101cc190 (patch) | |
tree | 464c6ec3f3663ea32eea41745a3dad91c4cb35ff /modules | |
parent | f5b178083ec7a933c426b518920f3e99c6f080dd (diff) | |
download | brdo-864408cefa3cecf2b4b3a7ef951a5612101cc190.tar.gz brdo-864408cefa3cecf2b4b3a7ef951a5612101cc190.tar.bz2 |
- Patch #652420 by sun: fixed various failures.
Diffstat (limited to 'modules')
-rw-r--r-- | modules/contact/contact.pages.inc | 9 | ||||
-rw-r--r-- | modules/system/system.test | 2 |
2 files changed, 8 insertions, 3 deletions
diff --git a/modules/contact/contact.pages.inc b/modules/contact/contact.pages.inc index f3827083d..fcf3d6e2e 100644 --- a/modules/contact/contact.pages.inc +++ b/modules/contact/contact.pages.inc @@ -20,7 +20,8 @@ function contact_site_form($form, &$form_state) { $window = variable_get('contact_threshold_window', 3600); if (!flood_is_allowed('contact', $limit, $window) && !user_access('administer contact forms')) { drupal_set_message(t("You cannot send more than %limit messages in @interval. Please try again later.", array('%limit' => $limit, '@interval' => format_interval($window))), 'error'); - return drupal_access_denied(); + drupal_access_denied(); + drupal_exit(); } // Get an array of the categories and the current default category. @@ -39,7 +40,8 @@ function contact_site_form($form, &$form_state) { drupal_set_message(t('The contact form has not been configured. <a href="@add">Add one or more categories</a> to the form.', array('@add' => url('admin/structure/contact/add'))), 'error'); } else { - return drupal_not_found(); + drupal_not_found(); + drupal_exit(); } } @@ -177,7 +179,8 @@ function contact_personal_form($form, &$form_state, stdClass $recipient) { $window = variable_get('contact_threshold_window', 3600); if (!flood_is_allowed('contact', $limit, $window) && !user_access('administer contact forms') && !user_access('administer users')) { drupal_set_message(t("You cannot send more than %limit messages in @interval. Please try again later.", array('%limit' => $limit, '@interval' => format_interval($window))), 'error'); - return drupal_access_denied(); + drupal_access_denied(); + drupal_exit(); } drupal_set_title(t('Contact @username', array('@username' => format_username($recipient))), PASS_THROUGH); diff --git a/modules/system/system.test b/modules/system/system.test index 9ecebd9ec..522f96b0a 100644 --- a/modules/system/system.test +++ b/modules/system/system.test @@ -528,6 +528,7 @@ class AccessDeniedTestCase extends DrupalWebTestCase { function testAccessDenied() { $this->drupalGet('admin'); $this->assertText(t('Access denied'), t('Found the default 403 page')); + $this->assertReponse(403); $edit = array( 'title' => array(LANGUAGE_NONE => array(array('value' => $this->randomName(10)))), @@ -557,6 +558,7 @@ class AccessDeniedTestCase extends DrupalWebTestCase { $this->drupalGet('admin'); $this->assertText(t('Access denied'), t('Found the default 403 page')); + $this->assertReponse(403); $this->assertText(t('User login'), t('Blocks are shown on the default 403 page')); // Log back in, set the custom 403 page to /user and remove the block |