Better handle log/temp directory misconfiguration

Aleksander Machniak

2015-02-03 e17decc4c2b175e1b2e3cd79f7c8b4c39c1f70f1

 program/js/editor.js

@@ -36,11 +36,13 @@
function rcube_text_editor(config, id)
{
  var ref = this,
    abs_url = location.href.replace(/[?#].*$/, '').replace(/\/$/, ''),
    conf = {
      selector: '#' + ($('#' + id).is('.mce_editor') ? id : 'fake-editor-id'),
      cache_suffix: 's=4010700',
      theme: 'modern',
      language: config.lang,
      content_css: 'program/js/tinymce/roundcube/content.css?v1',
      content_css: rcmail.assets_path('program/js/tinymce/roundcube/content.css'),
      menubar: false,
      statusbar: false,
      toolbar_items_size: 'small',
@@ -82,7 +84,7 @@
      toolbar: 'bold italic underline | alignleft aligncenter alignright alignjustify'
        + ' | bullist numlist outdent indent ltr rtl blockquote | forecolor backcolor | fontselect fontsizeselect'
        + ' | link unlink table | emoticons charmap image media | code searchreplace undo redo',
      spellchecker_rpc_url: '../../../../../?_task=utils&_action=spell_html&_remote=1',
      spellchecker_rpc_url: abs_url + '/?_task=utils&_action=spell_html&_remote=1',
      spellchecker_language: rcmail.env.spell_lang,
      accessibility_focus: false,
      file_browser_callback: function(name, url, type, win) { ref.file_browser_callback(name, url, type); },
@@ -194,7 +196,7 @@
  };

  // switch html/plain mode
  this.toggle = function(ishtml)
  this.toggle = function(ishtml, noconvert)
  {
    var curr, content, result,
      // these non-printable chars are not removed on text2html and html2text
@@ -211,11 +213,12 @@
      content = input.val();

      // replace current text signature with temp mark
      if (is_sig)
        content = content.replace(signature.text, sig_mark);
      if (is_sig) {
        content = content.replace(/\r\n/, "\n");
        content = content.replace(signature.text.replace(/\r\n/, "\n"), sig_mark);
      }

      // convert to html
      result = rcmail.plain2html(content, function(data) {
      var init_editor = function(data) {
        // replace signature mark with html version of the signature
        if (is_sig)
          data = data.replace(sig_mark, '<div id="_rc_sig">' + signature.html + '</div>');
@@ -231,7 +234,16 @@
            ref.tabindex(true);
          }
        }, 500);
      });
      };

      // convert to html
      if (!noconvert) {
        result = rcmail.plain2html(content, init_editor);
      }
      else {
        init_editor(content);
        result = true;
      }
    }
    else if (this.editor) {
      if (is_sig) {
@@ -250,8 +262,7 @@
      // get html content
      content = this.editor.getContent();

      // convert html to text
      result = rcmail.html2plain(content, function(data) {
      var init_plaintext = function(data) {
        tinymce.execCommand('mceRemoveEditor', false, ref.id);
        ref.editor = null;

@@ -260,7 +271,16 @@
          data = data.replace(sig_mark, "\n" + signature.text);

        input.val(data).focus();
      });
      };

      // convert html to text
      if (!noconvert) {
        result = rcmail.html2plain(content, init_plaintext);
      }
      else {
        init_plaintext(input.val());
        result = true;
      }

      // bring back current signature
      if (!result && curr)
@@ -375,29 +395,33 @@
  };

  // get selected text (if no selection returns all text) from the editor
  this.get_content = function(selected, plain)
  this.get_content = function(args)
  {
    // apply spellcheck changes if spell checker is active
    this.spellcheck_stop();
    var sigstart, ed = this.editor, text = '', strip = false,
      defaults = {refresh: true, selection: false, nosig: false, format: 'html'};

    var sigstart, ed = this.editor,
      format = plain ? 'text' : 'html',
      text = '', strip = false;
    args = $.extend(defaults, args);

    // apply spellcheck changes if spell checker is active
    if (args.refresh) {
      this.spellcheck_stop();
    }

    // get selected text from tinymce editor
    if (ed) {
      ed.getWin().focus(); // correct focus in IE & Chrome
      if (selected)
        text = ed.selection.getContent({format: format});
      if (args.selection)
        text = ed.selection.getContent({format: args.format});

      if (!text) {
        text = ed.getContent({format: format});
        strip = true;
        text = ed.getContent({format: args.format});
        // @todo: strip signature in html mode
        strip = args.format == 'text';
      }
    }
    // get selected text from compose textarea
    else if (ed = rcube_find_object(this.id)) {
      if (selected && $(ed).is(':focus')) {
      if (args.selection && $(ed).is(':focus')) {
        text = rcmail.get_input_selection(ed).text;
      }

@@ -408,7 +432,8 @@
    }

    // strip off signature
    if (strip) {
    // @todo: make this optional
    if (strip && args.nosig) {
      sigstart = text.indexOf('-- \n');
      if (sigstart > 0) {
        text = text.substring(0, sigstart);