Fix opener check in extwin (avoid JS errors in IE when opener is gone); alw...

Thomas Bruederli

2013-01-11 7cc1d3ae170b63c5f477db4eda09b94ab42d1bb5

 program/js/app.js

@@ -361,7 +361,7 @@

        if (this.gui_objects.editform) {
          this.enable_command('save', true);
          if (this.env.action == 'add' || this.env.action == 'edit')
          if (this.env.action == 'add' || this.env.action == 'edit' || this.env.action == 'search')
              this.init_contact_form();
        }

@@ -441,10 +441,11 @@

        this.enable_command('login', true);
        break;
    }

      default:
        break;
      }
    // unset contentframe variable if preview_pane is enabled
    if (this.env.contentframe && !$('#' + this.env.contentframe).is(':visible'))
      this.env.contentframe = null;

    // prevent from form submit with Enter key in file input fields
    if (bw.ie)
@@ -651,13 +652,13 @@
        break;

      case 'expunge':
        if (this.env.messagecount)
        if (this.env.exists)
          this.expunge_mailbox(this.env.mailbox);
        break;

      case 'purge':
      case 'empty-mailbox':
        if (this.env.messagecount)
        if (this.env.exists)
          this.purge_mailbox(this.env.mailbox);
        break;

@@ -990,7 +991,7 @@
        if (uid = this.get_single_uid()) {
          url = {_reply_uid: uid, _mbox: this.env.mailbox};
          if (command == 'reply-all')
            // do reply-list, when list is detected and popup menu wasn't used 
            // do reply-list, when list is detected and popup menu wasn't used
            url._all = (!props && this.commands['reply-list'] ? 'list' : 'all');
          else if (command == 'reply-list')
            url._all = 'list';
@@ -1386,8 +1387,8 @@
      // over the folders
      for (k in this.env.folder_coords) {
        pos = this.env.folder_coords[k];
        if (mouse.x >= pos.x1 && mouse.x < pos.x2 && mouse.y >= pos.y1 && mouse.y < pos.y2){
          if ((check = this.check_droptarget(k))) {
        if (mouse.x >= pos.x1 && mouse.x < pos.x2 && mouse.y >= pos.y1 && mouse.y < pos.y2) {
          if (check = this.check_droptarget(k)) {
            li = this.get_folder_li(k);
            div = $(li.getElementsByTagName('div')[0]);

@@ -1401,7 +1402,8 @@
                rcmail.command('collapse-folder', rcmail.folder_auto_expand);
                rcmail.drag_start(null);
              }, 1000);
            } else if (this.folder_auto_timer) {
            }
            else if (this.folder_auto_timer) {
              clearTimeout(this.folder_auto_timer);
              this.folder_auto_timer = null;
              this.folder_auto_expand = null;
@@ -1411,9 +1413,10 @@
            this.env.folder_coords[k].on = 1;
            this.env.last_folder_target = k;
            layerclass = 'draglayer' + (check > 1 ? 'copy' : 'normal');
          } else { // Clear target, otherwise drag end will trigger move into last valid droptarget
            this.env.last_folder_target = null;
          }
          // Clear target, otherwise drag end will trigger move into last valid droptarget
          else
            this.env.last_folder_target = null;
        }
        else if (pos.on) {
          $(this.get_folder_li(k)).removeClass('droptarget');
@@ -1523,16 +1526,16 @@
    if (this.preview_read_timer)
      clearTimeout(this.preview_read_timer);

    var selected = list.get_single_selection() != null;
    var selected = list.get_single_selection();

    this.enable_command(this.env.message_commands, selected);
    this.enable_command(this.env.message_commands, selected != null);
    if (selected) {
      // Hide certain command buttons when Drafts folder is selected
      if (this.env.mailbox == this.env.drafts_mailbox)
        this.enable_command('reply', 'reply-all', 'reply-list', 'forward', 'forward-attachment', 'forward-inline', false);
      // Disable reply-list when List-Post header is not set
      else {
        var msg = this.env.messages[list.get_single_selection()];
        var msg = this.env.messages[selected];
        if (!msg.ml)
          this.enable_command('reply-list', false);
      }
@@ -1640,27 +1643,31 @@

  this.check_droptarget = function(id)
  {
    var allow = false, copy = false;

    if (this.task == 'mail')
      allow = (this.env.mailboxes[id] && this.env.mailboxes[id].id != this.env.mailbox && !this.env.mailboxes[id].virtual);
    else if (this.task == 'settings')
      allow = (id != this.env.mailbox);
    else if (this.task == 'addressbook') {
      return (this.env.mailboxes[id] && this.env.mailboxes[id].id != this.env.mailbox && !this.env.mailboxes[id].virtual) ? 1 : 0;

    if (this.task == 'settings')
      return id != this.env.mailbox ? 1 : 0;

    if (this.task == 'addressbook') {
      if (id != this.env.source && this.env.contactfolders[id]) {
        // droptarget is a group - contact add to group action
        if (this.env.contactfolders[id].type == 'group') {
          var target_abook = this.env.contactfolders[id].source;
          allow = this.env.contactfolders[id].id != this.env.group && !this.env.contactfolders[target_abook].readonly;
          copy = target_abook != this.env.source;
          if (this.env.contactfolders[id].id != this.env.group && !this.env.contactfolders[target_abook].readonly) {
            // search result may contain contacts from many sources
            return (this.env.selection_sources.length > 1 || $.inArray(target_abook, this.env.selection_sources) == -1) ? 2 : 1;
          }
        }
        else {
          allow = !this.env.contactfolders[id].readonly;
          copy = true;
        // droptarget is a (writable) addressbook - contact copy action
        else if (!this.env.contactfolders[id].readonly) {
          // search result may contain contacts from many sources
          return (this.env.selection_sources.length > 1 || $.inArray(id, this.env.selection_sources) == -1) ? 2 : 0;
        }
      }
    }

    return allow ? (copy ? 2 : 1) : 0;
    return 0;
  };

  this.open_window = function(url, width, height)
@@ -2600,7 +2607,7 @@
  {
    var uid, i, len, trash = this.env.trash_mailbox,
      list = this.message_list,
      selection = list.get_selection();
      selection = list ? list.get_selection() : [];

    // exit if no mailbox specified or if selection is empty
    if (!this.env.uid && !selection.length)
@@ -2710,7 +2717,7 @@
    data._mbox = this.env.mailbox;

    if (!data._uid) {
      var uids = this.env.uid ? this.env.uid : this.message_list.get_selection();
      var uids = this.env.uid ? [this.env.uid] : this.message_list.get_selection();
      data._uid = this.uids_to_list(uids);
    }

@@ -2971,7 +2978,7 @@
  // test if purge command is allowed
  this.purge_mailbox_test = function()
  {
    return (this.env.messagecount && (this.env.mailbox == this.env.trash_mailbox || this.env.mailbox == this.env.junk_mailbox
    return (this.env.exists && (this.env.mailbox == this.env.trash_mailbox || this.env.mailbox == this.env.junk_mailbox
      || this.env.mailbox.match('^' + RegExp.escape(this.env.trash_mailbox) + RegExp.escape(this.env.delimiter))
      || this.env.mailbox.match('^' + RegExp.escape(this.env.junk_mailbox) + RegExp.escape(this.env.delimiter))));
  };
@@ -3030,7 +3037,7 @@
      ac_props;

    // close compose step in opener
    if (window.opener && opener.rcmail && opener.rcmail.env.action == 'compose') {
    if (window.opener && !window.opener.closed && opener.rcmail && opener.rcmail.env.action == 'compose') {
      setTimeout(function(){ opener.history.back(); }, 100);
      this.env.opened_extwin = true;
    }
@@ -3700,9 +3707,10 @@
  {
    this.display_message(msg, type);

    if (this.env.extwin && window.opener && opener.rcmail) {
    if (this.env.extwin) {
      this.lock_form(this.gui_objects.messageform);
      opener.rcmail.display_message(msg, type);
      if (window.opener && !window.opener.closed && opener.rcmail)
        opener.rcmail.display_message(msg, type);
      setTimeout(function(){ window.close() }, 1000);
    }
    else {
@@ -4082,19 +4090,24 @@
    else if (this.env.contentframe)
      this.show_contentframe(false);

    // no source = search result, we'll need to detect if any of
    // selected contacts are in writable addressbook to enable edit/delete
    if (list.selection.length) {
      // no source = search result, we'll need to detect if any of
      // selected contacts are in writable addressbook to enable edit/delete
      // we'll also need to know sources used in selection for copy
      // and group-addmember operations (drag&drop)
      this.env.selection_sources = [];
      if (!source) {
        for (n in list.selection) {
          sid = String(list.selection[n]).replace(/^[^-]+-/, '');
          if (sid && this.env.address_sources[sid] && !this.env.address_sources[sid].readonly) {
            writable = true;
            break;
          if (sid && this.env.address_sources[sid]) {
            writable = writable || !this.env.address_sources[sid].readonly;
            this.env.selection_sources.push(sid);
          }
        }
        this.env.selection_sources = $.unique(this.env.selection_sources);
      }
      else {
        this.env.selection_sources.push(this.env.source);
        writable = !source.readonly;
      }
    }
@@ -4245,22 +4258,35 @@
  // copy a contact to the specified target (group or directory)
  this.copy_contact = function(cid, to)
  {
    var n, dest = to.type == 'group' ? to.source : to.id,
      source = this.env.source,
      group = this.env.group ? this.env.group : '';

    if (!cid)
      cid = this.contact_list.get_selection().join(',');

    if (to.type == 'group' && to.source == this.env.source)
      this.group_member_change('add', cid, to.source, to.id);
    else if (to.type == 'group' && !this.env.address_sources[to.source].readonly) {
      var lock = this.display_message(this.get_label('copyingcontact'), 'loading'),
        post_data = {_cid: cid, _source: this.env.source, _to: to.source, _togid: to.id,
          _gid: (this.env.group ? this.env.group : '')};
    if (!cid || !this.env.address_sources[dest] || this.env.address_sources[dest].readonly)
      return;

      this.http_post('copy', post_data, lock);
    // search result may contain contacts from many sources, but if there is only one...
    if (source == '' && this.env.selection_sources.length == 1)
      source = this.env.selection_sources[0];

    // tagret is a group
    if (to.type == 'group') {
      if (dest == source)
        this.group_member_change('add', cid, dest, to.id);
      else {
        var lock = this.display_message(this.get_label('copyingcontact'), 'loading'),
          post_data = {_cid: cid, _source: source, _to: dest, _togid: to.id, _gid: group};

        this.http_post('copy', post_data, lock);
      }
    }
    else if (to.id != this.env.source && cid && this.env.address_sources[to.id] && !this.env.address_sources[to.id].readonly) {
    // target is an addressbook
    else if (to.id != source) {
      var lock = this.display_message(this.get_label('copyingcontact'), 'loading'),
        post_data = {_cid: cid, _source: this.env.source, _to: to.id,
          _gid: (this.env.group ? this.env.group : '')};
        post_data = {_cid: cid, _source: source, _to: to.id, _gid: group};

      this.http_post('copy', post_data, lock);
    }
@@ -4371,10 +4397,11 @@
  {
    var ref = this, col;

    this.set_photo_actions($('#ff_photo').val());

    for (col in this.env.coltypes)
      this.init_edit_field(col, null);
    if (this.env.coltypes) {
      this.set_photo_actions($('#ff_photo').val());
      for (col in this.env.coltypes)
        this.init_edit_field(col, null);
    }

    $('.contactfieldgroup .row a.deletebutton').click(function() {
      ref.delete_edit_field(this);
@@ -6104,14 +6131,14 @@
    // send request
    this.log('HTTP GET: ' + url);

    // reset keep-alive interval
    this.start_keepalive();

    return $.ajax({
      type: 'GET', url: url, data: { _unlock:(lock?lock:0) }, dataType: 'json',
      success: function(data){ ref.http_response(data); },
      error: function(o, status, err) { ref.http_error(o, status, err, lock, action); }
    });

    // reset keep-alive interval
    this.start_keepalive();
  };

  // send a http POST request to the server
@@ -6139,14 +6166,14 @@
    // send request
    this.log('HTTP POST: ' + url);

    // reset keep-alive interval
    this.start_keepalive();

    return $.ajax({
      type: 'POST', url: url, data: postdata, dataType: 'json',
      success: function(data){ ref.http_response(data); },
      error: function(o, status, err) { ref.http_error(o, status, err, lock, action); }
    });

    // reset keep-alive interval
    this.start_keepalive();
  };

  // aborts ajax request
@@ -6228,7 +6255,7 @@
      case 'purge':
      case 'expunge':
        if (this.task == 'mail') {
          if (!this.env.messagecount) {
          if (!this.env.exists) {
            // clear preview pane content
            if (this.env.contentframe)
              this.show_contentframe(false);
@@ -6248,7 +6275,8 @@
        this.env.qsearch = null;
      case 'list':
        if (this.task == 'mail') {
          this.enable_command('show', 'expunge', 'select-all', 'select-none', (this.env.messagecount > 0));
          this.enable_command('show', 'select-all', 'select-none', this.env.messagecount > 0);
          this.enable_command('expunge', this.env.exists);
          this.enable_command('purge', this.purge_mailbox_test());
          this.enable_command('expand-all', 'expand-unread', 'collapse-all', this.env.threading && this.env.messagecount);