[Tickets #10098] Re: save_sent_mail pref gets corrupted

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/10098
------------------------------------------------------------------------------
  Ticket             | 10098
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | save_sent_mail pref gets corrupted
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2011-08-17 04:27) wrote:

>> Maybe this commit will help - it will do sanity checking on saved
>> identities data.
>
> no, this is worth. I don't get any Sent folder now.

Now I am just convinced that your PHP is just badly broken.  As  
previously mentioned, there is only **ONE** place where we set the  
value of sent_mail_folder -> in IMP_Prefs_Identity::setValue().  If we  
are saving the 'sent_mail_folder' key, the value is ALWAYS passed  
through IMP_Mailbox::prefTo($val).  IMP_Mailbox::prefTo() ALWAYS  
returns a string:

         if (($ns = self::get($mbox)->namespace_info) !== null) {
              if ($ns['name'] == $def_ns['name']) {
                  /* From personal namespace => strip namespace. */
                  return substr(strval($mbox),  
strlen($def_ns['name'])); // STRING RETURN
              } elseif ($ns['name'] == $empty_ns['name']) {
                  /* From empty namespace => prefix with delimiter. */
                  return $empty_ns['delimiter'] . strval($mbox); //  
STRING RETURN
              }
          }

         return strval($mbox);  // STRING RETURN

You will have to figure out exactly why this isn't working on your  
system, or I am going to mark this bug bogus (because nobody else can  
reproduce).