[Tickets #5842] Re: Sent mail folder setting in identity settings wierd behaviour

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=5842
-----------------------------------------------------------------------
 Ticket             | 5842
 Updated By         | szymon at juraszczyk.com
 Summary            | Sent mail folder setting in identity settings wierd behaviour
 Queue              | IMP
 Version            | 4.1.5
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


szymon at juraszczyk.com (2007-10-30 10:20) wrote:

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.