- Fixed selecting identity on reply/forward (#1487981)

alecpl

2011-08-19 4436b434800e5b4094be22a053dfce6c14a68fa7

 program/steps/mail/compose.inc

@@ -111,7 +111,7 @@
$OUTPUT->add_label('nosubject', 'nosenderwarning', 'norecipientwarning', 'nosubjectwarning', 'cancel',
    'nobodywarning', 'notsentwarning', 'notuploadedwarning', 'savingmessage', 'sendingmessage', 
    'messagesaved', 'converting', 'editorwarning', 'searching', 'uploading', 'uploadingmany',
    'fileuploaderror', 'autocompletechars');
    'fileuploaderror');

$OUTPUT->set_env('compose_id', $COMPOSE_ID);

@@ -124,7 +124,6 @@
$OUTPUT->set_env('mailbox', $IMAP->get_mailbox_name());
$OUTPUT->set_env('sig_above', $CONFIG['sig_above']);
$OUTPUT->set_env('top_posting', $CONFIG['top_posting']);
$OUTPUT->set_env('autocomplete_min_length', $CONFIG['autocomplete_min_length']);

// get reference message and set compose mode
if ($msg_uid = $_SESSION['compose']['param']['draft_uid']) {
@@ -226,9 +225,11 @@
$MESSAGE->identities = $USER->list_identities();
if (count($MESSAGE->identities))
{
  foreach ($MESSAGE->identities as $idx => $sql_arr) {
    $email = mb_strtolower(rcube_idn_to_utf8($sql_arr['email']));
    $MESSAGE->identities[$idx]['email_ascii'] = $sql_arr['email'];
  foreach ($MESSAGE->identities as $idx => $ident) {
    $email = mb_strtolower(rcube_idn_to_utf8($ident['email']));

    $MESSAGE->identities[$idx]['email_ascii'] = $ident['email'];
    $MESSAGE->identities[$idx]['ident']       = format_email_recipient($ident['email'], $ident['name']);
    $MESSAGE->identities[$idx]['email']       = $email;
  }
}
@@ -243,7 +244,7 @@
else if (count($MESSAGE->identities)) {
  // extract all recipients of the reply-message
  $a_recipients = array();
  if ($compose_mode == RCUBE_COMPOSE_REPLY && is_object($MESSAGE->headers))
  if (is_object($MESSAGE->headers) && in_array($compose_mode, array(RCUBE_COMPOSE_REPLY, RCUBE_COMPOSE_FORWARD)))
  {
    $a_to = $IMAP->decode_address_list($MESSAGE->headers->to);
    foreach ($a_to as $addr) {
@@ -261,45 +262,46 @@
  }

  $from_idx         = null;
  $default_identity = 0;
  $default_identity = null;
  $return_path      = $MESSAGE->headers->others['return-path'];

  // Select identity
  foreach ($MESSAGE->identities as $idx => $sql_arr) {
  foreach ($MESSAGE->identities as $idx => $ident) {
    // save default identity ID
    if ($sql_arr['standard']) {
    if ($ident['standard']) {
      $default_identity = $idx;
    }
    // we need ascii here
    $email = $sql_arr['email_ascii'];
    $ident = format_email_recipient($email, $sql_arr['name']);

    // select identity
    // use From header
    if (in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT))) {
      if ($MESSAGE->headers->from == $ident) {
      if ($MESSAGE->headers->from == $ident['email_ascii']) {
        $from_idx = $idx;
        break;
      }
    }
    // reply to self, force To header value
    else if ($compose_mode == RCUBE_COMPOSE_REPLY && $MESSAGE->headers->from == $ident) {
    else if ($compose_mode == RCUBE_COMPOSE_REPLY && $MESSAGE->headers->from == $ident['email_ascii']) {
      $from_idx = $idx;
      $MESSAGE->compose['to'] = $MESSAGE->headers->to;
      break;
    }
    // set identity if it's one of the reply-message recipients
    else if (in_array($email, $a_recipients) && ($from_idx === null || $sql_arr['standard'])) {
      $from_idx = $idx;
    }
    // set identity when replying to mailing list
    else if (strpos($return_path, str_replace('@', '=', $email).'@') !== false) {
    // use reply-message recipients
    else if (in_array($ident['email_ascii'], $a_recipients)) {
      $from_idx = $idx;
    }
  }

  // Still no ID, use first identity
  // Fallback using Return-Path
  if ($from_idx === null && $return_path) {
    foreach ($MESSAGE->identities as $idx => $ident) {
      if (strpos($return_path, str_replace('@', '=', $ident['email_ascii']).'@') !== false) {
        $from_idx = $idx;
        break;
      }
    }
  }

  // Still no ID, use default/first identity
  if ($from_idx === null) {
    $from_idx = $default_identity;
    $from_idx = $default_identity !== null ? $default_identity : key(reset($MESSAGE->identities));
  }

  $ident   = $MESSAGE->identities[$from_idx];
@@ -336,13 +338,13 @@
      $mailfollowup = $MESSAGE->headers->others['mail-followup-to'];
      $mailreplyto  = $MESSAGE->headers->others['mail-reply-to'];

      if ($MESSAGE->compose['to'])
        $fvalue = $MESSAGE->compose['to'];
      else if ($MESSAGE->reply_all == 'list' && $mailfollowup)
      if ($MESSAGE->reply_all == 'list' && $mailfollowup)
        $fvalue = $mailfollowup;
      else if ($MESSAGE->reply_all == 'list'
        && preg_match('/<mailto:([^>]+)>/i', $MESSAGE->headers->others['list-post'], $m))
        $fvalue = $m[1];
      else if ($MESSAGE->reply_all && $mailfollowup)
        $fvalue = $mailfollowup;
      else if ($mailreplyto)
        $fvalue = $mailreplyto;
      else if (!empty($MESSAGE->headers->replyto))
@@ -386,7 +388,7 @@
      $mailto = mb_strtolower(rcube_idn_to_utf8($addr_part['mailto']));

      if (!in_array($mailto, $a_recipients)
        && (empty($MESSAGE->compose['from_email']) || $mailto != $MESSAGE->compose['from_email'])
        && ($header == 'to' || empty($MESSAGE->compose['from_email']) || $mailto != $MESSAGE->compose['from_email'])
      ) {
        if ($addr_part['name'] && $addr_part['mailto'] != $addr_part['name'])
          $string = format_email_recipient($mailto, $addr_part['name']);
@@ -397,7 +399,7 @@
        $a_recipients[] = $addr_part['mailto'];
      }
    }
    

    $fvalue = implode(', ', $fvalue);
  }

@@ -466,9 +468,12 @@
    $input = new $field_type($field_attrib);
    $out = $input->show($MESSAGE->compose[$param]);
  }
  

  if ($form_start)
    $out = $form_start.$out;

  // configure autocompletion
  rcube_autocomplete_init();

  return $out;
}
@@ -636,26 +641,26 @@
        $body);
    }
  }
  

  $HTML_MODE = $isHtml;
  

  return $body;
}

function rcmail_compose_body($attrib)
{
  global $RCMAIL, $CONFIG, $OUTPUT, $MESSAGE, $compose_mode, $LINE_LENGTH, $HTML_MODE, $MESSAGE_BODY;
  

  list($form_start, $form_end) = get_form_tags($attrib);
  unset($attrib['form']);
  

  if (empty($attrib['id']))
    $attrib['id'] = 'rcmComposeBody';

  $attrib['name'] = '_message';

  $isHtml = $HTML_MODE;
  

  $out = $form_start ? "$form_start\n" : '';

  $saveid = new html_hiddenfield(array('name' => '_draft_saveid', 'value' => $compose_mode==RCUBE_COMPOSE_DRAFT ? str_replace(array('<','>'), "", $MESSAGE->headers->messageID) : ''));
@@ -688,7 +693,7 @@

  // include HTML editor
  rcube_html_editor();
  

  // include GoogieSpell
  if (!empty($CONFIG['enable_spellcheck'])) {

@@ -749,7 +754,7 @@
    $OUTPUT->add_label('checking');
    $OUTPUT->set_env('spellcheck_langs', join(',', $editor_lang_set));
  }
 

  $out .= "\n".'<iframe name="savetarget" src="program/blank.gif" style="width:0;height:0;border:none;visibility:hidden;"></iframe>';

  return $out;
@@ -762,8 +767,13 @@

  // build reply prefix
  $from = array_pop($RCMAIL->imap->decode_address_list($MESSAGE->get_header('from'), 1, false));
  $prefix = sprintf("On %s, %s wrote:",
    $MESSAGE->headers->date, $from['name'] ? $from['name'] : rcube_idn_to_utf8($from['mailto']));
  $prefix = rcube_label(array(
    'name' => 'mailreplyintro',
    'vars' => array(
      'date' => format_date($MESSAGE->headers->date, $RCMAIL->config->get('date_long')),
      'sender' => $from['name'] ? $from['name'] : rcube_idn_to_utf8($from['mailto']),
    )
  ));

  if (!$bodyIsHtml) {
    $body = preg_replace('/\r?\n/', "\n", $body);
@@ -858,7 +868,7 @@

    $prefix .= "</tbody></table><br>";
  }
    

  return $prefix.$body;
}

@@ -866,7 +876,7 @@
function rcmail_create_draft_body($body, $bodyIsHtml)
{
  global $MESSAGE, $OUTPUT;
  

  /**
   * add attachments
   * sizeof($MESSAGE->mime_parts can be 1 - e.g. attachment, but no text!
@@ -881,7 +891,7 @@
    if ($cid_map)
      $body = str_replace(array_keys($cid_map), array_values($cid_map), $body);
  }
  

  return $body;
}

@@ -1061,7 +1071,7 @@
  } else if ($path) {
    @unlink($path);
  }
  

  return false;
}

@@ -1084,7 +1094,7 @@
    unset($attachment['data'], $attachment['status'], $attachment['content_id'], $attachment['abort']);
    return $attachment;
  }
  

  return false;
}

@@ -1205,20 +1215,11 @@
  if (!$attrib['id'])
    $attrib['id'] = 'rcmUploadbox';

  // Enable upload progress bar
  rcube_upload_progress_init();
  // Get filesize, enable upload progress bar
  $max_filesize = rcube_upload_init();

  // find max filesize value
  $max_filesize = parse_bytes(ini_get('upload_max_filesize'));
  $max_postsize = parse_bytes(ini_get('post_max_size'));
  if ($max_postsize && $max_postsize < $max_filesize)
    $max_filesize = $max_postsize;

  $OUTPUT->set_env('max_filesize', $max_filesize);
  $max_filesize = show_bytes($max_filesize);
  
  $button = new html_inputfield(array('type' => 'button'));
  

  $out = html::div($attrib,
    $OUTPUT->form_tag(array('name' => 'uploadform', 'method' => 'post', 'enctype' => 'multipart/form-data'),
      html::div(null, rcmail_compose_attachment_field(array('size' => $attrib['attachmentfieldsize']))) .
@@ -1229,7 +1230,7 @@
      )
    )
  );
  

  $OUTPUT->add_gui_object('uploadbox', $attrib['id']);
  return $out;
}