diff --git a/core/modules/overlay/overlay.module b/core/modules/overlay/overlay.module index 197d006..446d56c 100644 --- a/core/modules/overlay/overlay.module +++ b/core/modules/overlay/overlay.module @@ -572,51 +572,17 @@ function overlay_preprocess_page(&$variables) { } /** - * Stores and returns whether an empty page override is needed. - * - * This is used to prevent a page request which closes the overlay (for - * example, a form submission) from being fully re-rendered before the overlay - * is closed. Instead, we store a variable which will cause the page to be - * rendered by a delivery callback function that does not actually print - * visible HTML (but rather only the bare minimum scripts and styles necessary - * to trigger the overlay to close), thereby allowing the dialog to be closed - * faster and with less interruption, and also allowing the display of messages - * to be deferred to the parent window (rather than displaying them in the - * child window, which will close before the user has had a chance to read - * them). - * - * @param $value - * By default, an empty page will not be displayed. Set to TRUE to request - * an empty page display, or FALSE to disable the empty page display (if it - * was previously enabled on this page request). - * - * @return - * TRUE if the current behavior is to display an empty page, or FALSE if not. - * - * @see overlay_page_delivery_callback_alter() - */ -function overlay_display_empty_page($value = NULL) { - $display_empty_page = &drupal_static(__FUNCTION__, FALSE); - if (isset($value)) { - $display_empty_page = $value; - } - return $display_empty_page; -} - -/** - * Implements hook_page_delivery_callback_alter(). - */ -function overlay_page_delivery_callback_alter(&$callback) { - if (overlay_display_empty_page()) { - $callback = 'overlay_deliver_empty_page'; - } -} - -/** * Prints an empty page. * * This function is used to print out a bare minimum empty page which still has * the scripts and styles necessary in order to trigger the overlay to close. + * + * It can be used to prevent a page request which closes the overlay (for + * example, a form submission) from being fully re-rendered before the overlay + * is closed, thereby allowing the dialog to be closed faster and with less + * interruption, and also allowing the display of messages to be deferred to + * the parent window (rather than displaying them in the child window, which + * will close before the user has had a chance to read them). */ function overlay_deliver_empty_page() { $empty_page = '' . drupal_get_css() . drupal_get_js() . ''; @@ -749,7 +715,7 @@ function overlay_overlay_child_initialize() { } /** - * Requests that the overlay closes when the page is displayed. + * Immediately returns HTML to to the browser and closes the overlay. * * @param $redirect * (optional) The path that should open in the parent window after the @@ -759,6 +725,9 @@ function overlay_overlay_child_initialize() { * @param $redirect_options * (optional) An associative array of options to use when generating the * redirect URL. + * + * @todo This function should only request that the overlay close when the page + * is displayed (as it did in Drupal 7), not immediately end the request. */ function overlay_close_dialog($redirect = NULL, $redirect_options = array()) { $settings = array( @@ -776,7 +745,7 @@ function overlay_close_dialog($redirect = NULL, $redirect_options = array()) { // Since we are closing the overlay as soon as the page is displayed, we do // not want to show any of the page's actual content. - overlay_display_empty_page(TRUE); + overlay_deliver_empty_page(); } /**