Aleksander Machniak
2013-03-17 c60f58f9fdcc82574db341cc093eb9644f333edf
Fix "rcmail is undefined" error in HTML attachment preview - regression
from commit d30460ad2fc0f78ce44d474fa2c466d660596d27, small improvements
2 files modified
22 ■■■■■ changed files
program/include/rcmail_html_page.php 2 ●●● patch | view | raw | blame | history
program/include/rcmail_output_html.php 20 ●●●●● patch | view | raw | blame | history
program/include/rcmail_html_page.php
@@ -30,7 +30,7 @@
{
    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')) {
program/include/rcmail_output_html.php
@@ -307,17 +307,19 @@
    /**
     * 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 @@
            $this->parse($templ, false);
        }
        else {
            $this->framed = $templ == 'iframe' ? true : $this->framed;
            $this->framed = true;
            $this->write();
        }
@@ -396,9 +398,11 @@
          $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']);