From ad335b239539c47ac498ca65bf03629bb2c2795a Mon Sep 17 00:00:00 2001 From: Dries Buytaert Date: Fri, 17 Nov 2006 17:22:45 +0000 Subject: - Patch #84250 by yched and Moshe: providing a path with query string breaks the confirmation form. --- modules/system/system.module | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'modules/system/system.module') diff --git a/modules/system/system.module b/modules/system/system.module index 9bc448f1e..e296f8098 100644 --- a/modules/system/system.module +++ b/modules/system/system.module @@ -2075,7 +2075,7 @@ function system_theme_settings($key = '') { * @param $question * The question to ask the user (e.g. "Are you sure you want to delete the * block foo?"). - * @param $path + * @param $destination * The page to go to if the user denies the action. * @param $description * Additional text to display (defaults to "This action cannot be undone."). @@ -2089,8 +2089,9 @@ function system_theme_settings($key = '') { * @return * The form. */ -function confirm_form($form, $question, $path, $description = NULL, $yes = NULL, $no = NULL, $name = 'confirm') { +function confirm_form($form, $question, $destination, $description = NULL, $yes = NULL, $no = NULL, $name = 'confirm') { $description = ($description) ? $description : t('This action cannot be undone.'); + extract(parse_url($destination)); drupal_set_title($question); $form['#attributes'] = array('class' => 'confirmation'); $form['description'] = array('#value' => $description); @@ -2098,7 +2099,7 @@ function confirm_form($form, $question, $path, $description = NULL, $yes = NULL, $form['actions'] = array('#prefix' => '
', '#suffix' => '
'); $form['actions']['submit'] = array('#type' => 'submit', '#value' => $yes ? $yes : t('Confirm')); - $form['actions']['cancel'] = array('#value' => l($no ? $no : t('Cancel'), $path)); + $form['actions']['cancel'] = array('#value' => l($no ? $no : t('Cancel'), $path, null, $query, $fragment)); $form['#base'] = 'confirm_form'; return $form; } -- cgit v1.2.3