[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-24 05:21) wrote:

> if $_prefs['default_identity']['locked'] is set to true. I don't get  
> a Sent folder.

This was the key piece of information.  If default_identity was  
locked, we were taking the current value of the various identity  
properties and temporarily saving them to the prefs cache.  However,  
this saved information was written directly to the prefs backend,  
without first calling setValue(), so any necessary changes (like  
converting from object -> string) were not done.

I don't see why we even need to save to the prefs cache.  We should  
just let subsequent calls to getValue() grab the default values from  
the preferences backend as needed.

So try this patch and see if it fixes things.