6.0.0-beta1
7/6/25

[#5842] Sent mail folder setting in identity settings wierd behaviour
Summary Sent mail folder setting in identity settings wierd behaviour
Queue IMP
Queue Version 4.1.5
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester szymon (at) juraszczyk (dot) com
Created 10/30/2007 (6459 days ago)
Due
Updated 11/23/2007 (6435 days ago)
Assigned 10/30/2007 (6459 days ago)
Resolved 11/23/2007 (6435 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
11/23/2007 05:34:55 PM Jan Schneider Comment #4
State ⇒ Resolved
Reply to this comment
This has been fixed in IMP.
10/30/2007 05:50:35 PM Chuck Hagenbuch Assigned to Jan Schneider
State ⇒ Assigned
 
10/30/2007 05:20:14 PM szymon (at) juraszczyk (dot) com Comment #3 Reply to this comment
I know you guys are busy, so here's a patch that did the trick for me:



--- templates/prefs/identityselect.inc.orig     2007-08-03 
06:48:56.000000000 +0200

+++ templates/prefs/identityselect.inc  2007-10-30 18:09:15.286498175 +0100

@@ -61,7 +61,7 @@

      }

      for (var i = 0; i < identities[id].length; i++) {

          field = document.prefs[identities[id][i][0]];

-        if (!field) {

+        if (!field && identities[id][i][1] != "implicit") {

              continue;

          }

          val = identities[id][i][2];





It turned out that it had nothing to with Cyrus or namespaces - it was 
just a design bug. Somehow the sent mail folder select field is named 
"sent_mail" and the corresponding setting in identities[] table is 
named "sent_mail_folder", so there won't be any 
document.prefs[identities[id][i][0]] and newChoice_sent_mail_folder() 
JavaScript function will never be called.



My patch basically alters newChoice() JavaScript function so that it 
won't require the existence of a form field for settings of type 
"implicit". If you don't like it, I'm sure you're come up with another 
solution.


10/30/2007 04:29:11 PM szymon (at) juraszczyk (dot) com Comment #2 Reply to this comment
Sorry for generating traffic, but after I have just observed something 
that may be relevant. In the source of services/prefs.php page there is



identities[1][12] = new Array("sent_mail_folder", "implicit", 
decodeURIComponent("INBOX.sent_mail"));



On another installation of good old IMP 4.0.5 which I have at hand and 
where this setting works correctly, this differes significantly:



identities[0][12] = new Array("sent_mail_folder", "implicit", 
decodeURIComponent("sent_mail"));



Apparently there was no "INBOX." prefix in older version, but there I have



       'namespace' => 'INBOX.'



set in IMAP server definition. If I understand correctly, semantic of 
this setting was altered in IMP 4.1.x as some namespace auto-detection 
magic has been implemented. I therefore have no 'namespace' set in the 
affected 4.1.5 installations, following $servers['cyrus'] example. I 
do not connect to IMAP server using TLS or SSL (I guess this matters 
in some cases). Anyway folders in general seem to be working fine.
10/30/2007 04:05:35 PM szymon (at) juraszczyk (dot) com Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Sent mail folder setting in identity settings wierd behaviour
Queue ⇒ IMP
State ⇒ Unconfirmed
Reply to this comment
Hello everyone



I have two IMP 4.1.5 installations and I experience some weird 
behaviour of "Sent mail folder" setting in identity preferences. I can 
select any folder I like, save options and it works fine and dandy, 
but whenever I go back to identity settings, the "Sent mail folder" 
setting is always displayed as if it was set to "Use Default Value". 
When I select any defined identity in the 'Select the identity you 
want to change' field (I have a few of them set up), all other 
settings seem to be populated correctly with their previously assigned 
values (at least the settings I use, for example the signature, email 
and name), but "Sent mail folder" will adamantly remain set to "Use 
Default Value".



My IMAP server is Cyrus, in one case 2.1.18, and 2.2.13 in another. 
Tested with Firefox 2.0.0.8 and MSIE 7.0.

Saved Queries