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

* Your Email Address
* Spam protection	Enter the letters below: .__. __ .__ . ..__ [__]/ `\| \\| \|\| \ \| \|\__.\|__/\|/\\|\|__/
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. > >
Attachment
Watch this ticket