#145 Take Address not backward for INBOX.Sent (with fix)

open
nobody
None
5
2009-05-29
2006-05-02
Keith Brown
No

When taking addresses ("Add to Addressbook") the code
should look at the "To:" field, not the "From:" field,
which is, obviously, always the user.

Discussion

  • Keith Brown

    Keith Brown - 2006-05-02

    Logged In: YES
    user_id=1514889

    This is in plugins/address_add/classes/...

    If the mailbox is a "Sent" type box, then the utility is in
    athering "To:" addresses rather than "From:", "CC:", or
    "Received for:" fields.

     
  • Keith Brown

    Keith Brown - 2006-05-02

    Logged In: YES
    user_id=1514889

    Here's the fix... .../classes/getemail.php
    added three lines and uses $mailbox global...

    global $message, $mailbox;
    $this->decodedfrom = "";
    if (function_exists('sqgetGlobalVar')) {
    $header = $message->rfc822_header;
    if ($header) {
    if (isset($mailbox) && $mailbox == 'INBOX.Sent')
    $this->decodedfrom = $header->getAddr_s('to');
    else
    $this->decodedfrom = $header->getAddr_s('from');
    if ($this->decodedfrom == "")
    $this->decodedfrom =
    $header->getAddr_s('sender');
    }
    } else {
    $this->decodedfrom = $message->header->from;
    }
    $this->decodedfrom = decodeHeader($this->decodedfrom);
    $this->decodedfrom = clean($this->decodedfrom);

     
  • Keith Brown

    Keith Brown - 2006-05-02
    • summary: Take Address not backward for INBOX.Sent --> Take Address not backward for INBOX.Sent (with fix)
     
  • Tomas Kuliavas

    Tomas Kuliavas - 2006-05-08

    Logged In: YES
    user_id=225877

    address_add plugin is not bundled with SquirrelMail.

     
  • Tomas Kuliavas

    Tomas Kuliavas - 2006-05-08
    • assigned_to: nobody --> cigamit
     
  • Fredrik Jervfors

    • assigned_to: cigamit --> nobody
     

Log in to post a comment.