summaryrefslogtreecommitdiff
path: root/program
diff options
context:
space:
mode:
authorAleksander Machniak <alec@alec.pl>2013-03-17 09:59:48 +0100
committerAleksander Machniak <alec@alec.pl>2013-03-17 10:01:41 +0100
commitc60f58f9fdcc82574db341cc093eb9644f333edf (patch)
tree2d156451553f5b2d46145d1dab5efc78677bff78 /program
parent4c0c248dd9eb2f928a8e2468927307b2c8dbc375 (diff)
Fix "rcmail is undefined" error in HTML attachment preview - regression
from commit d30460ad2fc0f78ce44d474fa2c466d660596d27, small improvements
Diffstat (limited to 'program')
-rw-r--r--program/include/rcmail_html_page.php2
-rw-r--r--program/include/rcmail_output_html.php20
2 files changed, 13 insertions, 9 deletions
diff --git a/program/include/rcmail_html_page.php b/program/include/rcmail_html_page.php
index 1d8391580..5d07b8d04 100644
--- a/program/include/rcmail_html_page.php
+++ b/program/include/rcmail_html_page.php
@@ -30,7 +30,7 @@ class rcmail_html_page extends rcmail_output_html
{
public function write($contents = '')
{
- self::reset();
+ self::reset(true);
// load embed.css from skin folder (if exists)
if ($embed_css = $this->get_skin_file('/embed.css')) {
diff --git a/program/include/rcmail_output_html.php b/program/include/rcmail_output_html.php
index 2babe1cbb..ec32c11a7 100644
--- a/program/include/rcmail_output_html.php
+++ b/program/include/rcmail_output_html.php
@@ -307,17 +307,19 @@ class rcmail_output_html extends rcmail_output
/**
* Delete all stored env variables and commands
+ *
+ * @param bool $all Reset all env variables (including internal)
*/
- public function reset()
+ public function reset($all = false)
{
- $env = array_intersect_key($this->env, array('extwin'=>1, 'framed'=>1));
+ $env = $all ? null : array_intersect_key($this->env, array('extwin'=>1, 'framed'=>1));
parent::reset();
// let some env variables survive
$this->env = $this->js_env = $env;
- $this->js_labels = array();
- $this->js_commands = array();
+ $this->js_labels = array();
+ $this->js_commands = array();
$this->script_files = array();
$this->scripts = array();
$this->header = '';
@@ -362,7 +364,7 @@ class rcmail_output_html extends rcmail_output
$this->parse($templ, false);
}
else {
- $this->framed = $templ == 'iframe' ? true : $this->framed;
+ $this->framed = true;
$this->write();
}
@@ -396,9 +398,11 @@ class rcmail_output_html extends rcmail_output
$this->set_env('request_token', $this->app->get_request_token());
// write all env variables to client
- $js = $this->framed ? "if(window.parent) {\n" : '';
- $js .= $this->get_js_commands() . ($this->framed ? ' }' : '');
- $this->add_script($js, 'head_top');
+ if ($commands = $this->get_js_commands()) {
+ $js = $this->framed ? "if (window.parent) {\n" : '';
+ $js .= $commands . ($this->framed ? ' }' : '');
+ $this->add_script($js, 'head_top');
+ }
// send clickjacking protection headers
$iframe = $this->framed || !empty($_REQUEST['_framed']);